きっかけは、久しぶりに人力検索はてな
q.hatena.ne.jp
で回答した後に、
自分の書いたコードを読み返してみて笑ってしまったので。
※他の回答者のコードに刺激を受けたので、
コードレビューについて少々老害おっさんが小話を行ってみました。
ライブ配信のテストを兼ねて小話をしてみました。
【プログラマー小話】無くて七癖 コードレビュー ほか ライブ解説テスト - YouTube
www.youtube.com
1.範囲チェックで定数＜変数And変数＜定数に疑問？
2.Yes/No や OK/NG は Boolean型で返そうぜ？(文字列で返すのは素人っぽいし)
・これも、私の悪癖なのですか、
リターン値を今回のように文字列で返す関数が多かったり、
お仲間さんたちに言われますが、
Yes/No や OK/NG などは Boolean型で返そうぜ？
True Falseの判断だから、
あと、 If XXXX = True は やめてください。
↑これは、そのまま派と わざわざ True書かない派が居るかなぁ
3.コードレビューで新しいアイデアをもらったりして成長するよ
今回は、自分で自分のコードレビューだったけど、
会社や部内でやると、いろいろな角度の処理を見れて勉強になりますよ。
勉強になると言えば、今回 はてな の 他の人の回答で勉強させてもらったり。
・コードレビューだけじゃなく、
ネットのQA掲示板などで違う切り口の解決方法をみると、
面白かったり、成長しますね
4.老害のおっさんから注意事項
コードレビュー の 注意事項です。※失敗談
ア.コードをキレイにするつもりが失敗してしまう・・・
コードレビューで良い処理・きれいなコードを理解したので、さっそく今あるコードを書き換えてみた。
書き換え後、テストすればいいのだが、書き換えただけで満足してしまい、テストしなかった。。。すると動いていたコードを壊してしまった。そんなことがあったりなかったり・・
イ.あとは、あまり白熱しないで かなぁ
激論もいいけど、ほどほどにね
おわりに
将棋の感想戦 検討 みたいに、プロジェクト終了後に検討会できたらいいね・・・
怒られているうち 注意されているうちが花
なんで、みなさん いろいろと戦いながら頑張ってくださいね。
なんか、出向や派遣が長いと、戦って注意してもなぁ
あと、三か月、半年ガマンすれば次の職場だしなぁ・・と消極的になりそうだけど、まわりからイロイロなものを吸収していってくださいね。
辛口のご意見・感想まってますね。てきたらお手柔らかに・・・
ライブ配信のテストを兼ねて小話をしてみました。
【プログラマー小話】無くて七癖 コードレビュー ほか ライブ解説テスト - YouTube
www.youtube.com
