朝礼暮改で申し訳ないですが、という文章がメールの先頭にさらっと書いてある。
こういった4文字熟語をさらっと出されて、さらっと理解されると、うゎすっげーーこいつ。
と、一発で好印象を与えられると思った。
それは大量にある知識の中のほんの1つなんだろうから、もっともっと色々と知っているに違いない。
この人は俺の2倍は言葉を知ってるなと、頭相当いいなと見てて分かる。
で、今日のコードレビューの内容だ。
簡単に言うと、このようにネストしたif文になっていたのが原因だ。
if(A評価式){
if(B評価式){
continue;
}
}
そして、
if(A評価式 && B評価式){
continue
}
まとめろという指摘だったのだ。
さらに、B評価式の中に
null != value
と、あり、nullの判定要らねーだろとか色々と討論されていたのだ。
内容としては低次元で大したことではない。
俺に言わせれば、こんなcontinueを使わないといけないような値の入れ方自体が気に食わないって言ってやりたい。
答えじゃないけど。
こういう、これこれの場合はこれで、あれあれの場合はあれで、みたいな無駄な条件分岐多すぎるぞ。世の中。
もっとシンプルにエレガントにやれよ！！
コーディングとか、答え無数にあると思うかもしれないけど、真の道は1本じゃね？俺の修正したとこは、誰が修正してもこの答えしかないって修正方法したぞ。
まぁ、極端だけど本当に効率のいい書き方が何本もあるわけがない。
まぁ、置いといて。俺は思った。
こんな、指摘してもらえる環境ってメッチャありがたい。
書いたコードが全部、即座に通っちゃうような環境や、コード指摘したら逆切れされるとか。
俺はそんなんばかりだったから。
最後、若干喧嘩腰だったけど、みんなコードを良くしようと思って発言してるのは、いいことだと思う。受け入れる気が無くて切れて会話するとか、そんなアホな奴が居ないってだけで、どんだけ幸せだよ。
これが当たり前なんだ。これ以下の次元で仕事するなんてストレスなだけだ。
俺は誰にも文句を言わさんような、スマートなコードを書こうと思った。