アカウント名:
パスワード:
きっかけはこのおもしろい(オブラートに包んだ表現)ツイートではないかな
https://twitter.com/fnya/status/1192036095820615680 [twitter.com]
return (条件)? true : false; なんて書くくらいなら return 条件; だよなあそりゃレビューで落とすでしょ感
return ok ? true : false;なら俺でも怒るがreturn b*b-4*a*c >= 0 ? true : false;なら怒らない
この手の三項演算子結果をboolにしたくなるケースの本質は型が分かりにくいという点にあるのではと最近は思っている自分ならばこうするかなint d = b*b-4*a*c;return d >= 0;
式の中で型が変化しまくる場合、型が変わるタイミングで変数に入れてから、新しい式を作っていけば見通しが良くなるのではと思っている。
それで良いなら普通に return (b*b-4*a*c >= 0); とする方が評価式を返してるのがより明らかに思えるけどなあ。
自分が書くのにどちらがよいかではなく、他人が書いたのに怒るかどうか、許容できるかどうかの話をしているのですプログラマーは論理思考ができないのか
「怒る」「許容できる」なんて主観的な基準を論じてるんだから「自分が書くのにどちらがよいか」の話だろお前の論理思考では何がどう違うんだ
これはコードレビューの話だから、怒るとか許容できないというのは自分一人だけの問題ではなく、チームの総意である「おまえはこんな書き方はやめろ」と禁止するなら、禁止する理由が必要だ和ゴミを嫌っていても、禁止する理由がなければ禁止できないのと同じだ
俺の書き込みはそのチームの一員と仮定してだした意見であり、個人的に好む書き方とは限らない
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
弘法筆を選ばず、アレゲはキーボードを選ぶ -- アレゲ研究家
発端? (スコア:3, 参考になる)
きっかけはこのおもしろい(オブラートに包んだ表現)ツイートではないかな
https://twitter.com/fnya/status/1192036095820615680 [twitter.com]
Re: (スコア:0)
return (条件)? true : false; なんて書くくらいなら return 条件; だよなあ
そりゃレビューで落とすでしょ感
Re: (スコア:0)
return ok ? true : false;
なら俺でも怒るが
return b*b-4*a*c >= 0 ? true : false;
なら怒らない
Re: (スコア:0)
この手の三項演算子結果をboolにしたくなるケースの本質は型が分かりにくいという点にあるのではと最近は思っている
自分ならばこうするかな
int d = b*b-4*a*c;
return d >= 0;
式の中で型が変化しまくる場合、型が変わるタイミングで変数に入れてから、新しい式を作っていけば見通しが良くなるのではと思っている。
Re: (スコア:0)
それで良いなら普通に return (b*b-4*a*c >= 0); とする方が
評価式を返してるのがより明らかに思えるけどなあ。
Re: (スコア:0)
自分が書くのにどちらがよいかではなく、他人が書いたのに怒るかどうか、許容できるかどうかの話をしているのです
プログラマーは論理思考ができないのか
Re:発端? (スコア:0)
「怒る」「許容できる」なんて主観的な基準を論じてるんだから「自分が書くのにどちらがよいか」の話だろ
お前の論理思考では何がどう違うんだ
Re: (スコア:0)
これはコードレビューの話だから、怒るとか許容できないというのは自分一人だけの問題ではなく、チームの総意である
「おまえはこんな書き方はやめろ」と禁止するなら、禁止する理由が必要だ
和ゴミを嫌っていても、禁止する理由がなければ禁止できないのと同じだ
俺の書き込みはそのチームの一員と仮定してだした意見であり、個人的に好む書き方とは限らない