Re: [討論] 寫三元判斷式code review被打槍

作者: lovdkkkk (dk)   2022-12-27 01:42:03
最近公司讀書會在看 Martin Fowler 的 Refactoring,
大概第九或第十章他有用到三元
sample code 大概是這樣
const price = summer()? summerPrice() : commonPrice();
然後我們有看這串文章討論了一下
討論的結果是,我們覺得三元比較適合用在 "單純,且明確是非 A 即 B 的情況"
ex const desc = score > 60? "及格" : "不及格"
而像上面 summer 的情況則比較不適合,
因為一年有四季,需求調整改著改著就變巢狀了,
除非公司有明確的規範及 review 作業流程
而為了避免各種認知差異衍生的麻煩,直接禁用也是合理的
以上,剛好最近也有討論到
※ 引述《a88241050 (再回頭已是百殘身)》之銘言:
: 小弟寫java的
: 以前常常寫三元判斷式
: 就比如說
: String a;
: if(con) {
: a= "aaa";
: } else {
: a="bbb";
: }
: 這樣就要佔掉六行
: 所以我通常都是寫
: String a= con ? "aaa" : "bbb";
: 從五行變成一行
: 在我看來簡潔又方便
: 但最近給一個資深前輩code review的時候
: 他說不要用三元判斷式
: 因為不好閱讀
: 他工作那麼久也從來不用三元判斷式的
: 而且java有很多套件都可以用來取代三元判斷式
: 所以用三元判斷式真的不好嗎?
作者: jason222333 (發呆)   2022-12-27 06:20:00
這是最恰當的用法
作者: Jichang (C.C.Lemon)   2022-12-27 06:47:00
那個就是bool 你舉switch 的case 造你這說法 if 也要禁用
作者: wulouise (在線上!=在電腦前)   2022-12-27 11:40:00
不懂你說的guard是什麼意思,Early return另一個名稱不就是guard clause?把n個early return變一個?沒有profile就做最佳化等於是浪費機會成本..
作者: viper9709 (阿達)   2022-12-27 16:47:00
推明確非A即B的情況+1
作者: justben (BEN)   2022-12-27 18:38:00
不懂這有啥好吵 規定特殊寫法一率註解就好了啊要秀可以 麻煩註解 done
作者: ohmylove347 (米特巴爾)   2022-12-28 09:23:00
樓上不是,你會說的沒啥好吵的,是因為只要有一套「統一做法」那就不會有爭議,但現在的問題是如果要有一套統一做法,那是哪一套?統一做法能解決大家都知道,有爭議的是該用哪個做法來統一

Links booklink

Contact Us: admin [ a t ] ucptt.com