[PR]上記の広告は3ヶ月以上新規記事投稿のないブログに表示されています。新しい記事を書く事で広告が消えます。
ただいまコメントを受けつけておりません。
最近は JavaScript 以外を評価することが多い。
後輩のコードをレビューしての観点 その1 その2 その3 その4
例外はあるのだが、単語を略さない。 ZabbixApi を zapi 等と略しているのを見たが、素直に ZabbixApi と書くべき。どうせ IDE 等が助けてくれる。昔は変数名に長さ制限がある言語もあったので癖でやる人も結構いるようだ。
for, if, while 等が三重・四重になったらあまりよくない書き方をしていることがおおい。書き方を再考すべき。ガード節を使ったり、 map, filter, each 等を使うことで減らせるケースが多い。
関数の引数に "mode" といったものがあり、文字列型を取る。この値が "create" とか "delete" といった文字列を値をとる……というのはあとでたいへんなことになりがちなので避けるべき。どうしてもやりたい場合は文字列ではなく enum や固定値などをとる。そして、その引数を取る関数は他の関数へのルーティングにただただ特化させる。
ただいまコメントを受けつけておりません。