程度による (#1593575) | コードレビューって意味あるの ? | スラド
VB.NETで文字数をカウントする処理を書かせたら、 1.配列に1文字ずつ文字を切り出して入れて 2.配列の内容をFor文で1文字ずつ読み出してループ回数を取得 3.ループ回数を結果として返す みたいなのを書いてくるような組織（実話）に対しては、最初にガツンと釘を刺すという意味で有効だとおもいました。 あぁ、あと、同じくVB.NETでフラグ的な位置づけの変数をStringで宣言して「"True"」とか突っ込むようなチームとか。 ＃いずれも実話 時間は掛かりますが、将来のことを考えるとレベルアップの一手段として有効だと思います 具体例がいろいろ挙がっていますが、 一般常識不足（いわゆるお約束、定石から外れている） もっと簡単に記述できる（ライブラリにあるのに、等） ラッパを何重にもかぶせていて実体に到達しない（不要だし、却ってわからない） 無駄に計算資源、メモリ資源を使いすぎ（それでもできる
プログラミングテクニックのまとめ - プログラミング日記
とりあえず思いついたもののまとめ。 まずは、ベーシックなものから。 変数のスコープをなるべく狭くしろ 他はグローバル変数を使うなとか、モジュール化と界面を意識せよなど。とにかくスコープは重要かつ意外と奥が深い。スコープに関係する機能は、モジュール(パッケージ)、クロージャ、ローカル関数、ローカルクラス、変数の種類、アクセス制御など。 同じロジックのコードを2度以上書くな 他はDRY原則、コピペをするななど。自分の場合、2度書く方がシンプルになる場合、2度書くこともある。特に、ifやswitchなどのロジックの中で同じコードが2度現れる場合、ちょっとしたコードでわざわざ別のところで関数やブロックにまとめて、それを参照するのは面倒。但し3度以上現れる場合は関数などにまとめるケースが多いかも。 汎用コード内で条件分岐コードを減らせ 他はifをポリモーフィズムによりなくせなど。条件分岐は汎用性を損
1
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
処理を実行中です
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
