アカウント名:
パスワード:
VB.NETで文字数をカウントする処理を書かせたら、
1.配列に1文字ずつ文字を切り出して入れて2.配列の内容をFor文で1文字ずつ読み出してループ回数を取得3.ループ回数を結果として返す
みたいなのを書いてくるような組織(実話)に対しては、最初にガツンと釘を刺すという意味で有効だとおもいました。あぁ、あと、同じくVB.NETでフラグ的な位置づけの変数をStringで宣言して「"True"」とか突っ込むようなチームとか。
#いずれも実話
で、問題あったの!?基本的なスペック(要求時間や実行サイズ・動作確認)を満たしていれば、問題無いと思うけど。要件として、上記のような手法はとらないと明記されていなければ、受け入れる側に問題あり。<イチャモンや受け入れ側のスキル不足
上記の例では、もっとスマートに書けるのはわかるし、そうするべきであるのも分かるが、やってはイケナイ手法とまでは思えない。(環境によってはそうするのがベターな場合もある<ループ)
コードレビューの意義って、本当は作成者の思い込みによるバグ混入(たまたま動いてた)ってケースを見つけるためにあると思う。<見る(指摘する)側が余程スキルが高くないと、ほとんどの場合見つからないのが問題。<大概重箱の隅をつついて終了。(変数名がおかしいとか、マクロの宣言名が変とか)
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
普通のやつらの下を行け -- バッドノウハウ専門家
程度による (スコア:1)
VB.NETで文字数をカウントする処理を書かせたら、
1.配列に1文字ずつ文字を切り出して入れて
2.配列の内容をFor文で1文字ずつ読み出してループ回数を取得
3.ループ回数を結果として返す
みたいなのを書いてくるような組織(実話)に対しては、最初にガツンと釘を刺すという意味で有効だとおもいました。
あぁ、あと、同じくVB.NETでフラグ的な位置づけの変数をStringで宣言して「"True"」とか突っ込むようなチームとか。
#いずれも実話
Re:程度による (スコア:0)
で、問題あったの!?
基本的なスペック(要求時間や実行サイズ・動作確認)を満たしていれば、問題無いと思うけど。
要件として、上記のような手法はとらないと明記されていなければ、受け入れる側に
問題あり。<イチャモンや受け入れ側のスキル不足
上記の例では、もっとスマートに書けるのはわかるし、そうするべきであるのも分かるが、
やってはイケナイ手法とまでは思えない。(環境によってはそうするのがベターな場合もある<ループ)
コードレビューの意義って、本当は作成者の思い込みによるバグ混入(たまたま動いてた)って
ケースを見つけるためにあると思う。
<見る(指摘する)側が余程スキルが高くないと、ほとんどの場合見つからないのが問題。
<大概重箱の隅をつついて終了。(変数名がおかしいとか、マクロの宣言名が変とか)