等等，我原本以為只是一個簡單的問題
居然歪樓了
推動coding conventions 可以從你我做起
像原原po的問題是
if
if
if
if
；
；
；
；
把判斷式改過來變成
if
return;
if
return;
即可
這個就簡單起草一份coding conventions
拿給長官review, 以後code review 看到這個問題
就直接貼連結請junior 改就好了
這種東西很多學生時期根本沒碰過
自然就會波動拳出現
跟頂不頂大沒關系
反而是senior 不知道怎麼幫助junior
才是問題
: 我從上面的文章只看到原po說有很多if...else跟function用原本的copy過來，改一下自
: 己想修改的code
: 但卻沒看到任何提到效率問題，而且if...else是O(1)，並不會拖垮速度。
: 每個人寫code的習慣不一樣，
: 有的人喜歡這樣寫
: if() {
: }
: 有的人喜歡這樣寫
: if()
: {
: }
: 有的人喜歡程式碼短就連在一起
: if(...) cout << "xxx";
: else cout << "bbb";
: 也有人喜歡短的程式碼連在一起
: cout << "請輸入數字月份(1~12):"; cin >> month;
: 有的人喜歡命名用底線分開，如:month_arr
: 有些人喜歡用小寫大寫分開，如:monthArr
: 有些人不喜歡程式碼跟程式碼之間有空一行
: while {
: ....
: }
: if() {
: ....
: }
: for(int i = 0; i < N; i++) {
: ....
: }
: 但有些人喜歡有空一行
: while {
: ....
: }
: if() {
: ....
: }
: for(int i = 0; i < N; i++) {
: ....
: }
: 有人程式碼喜歡有空格分開
: for(int i = 0; i < N; i++)
: 有人不喜歡太多空格
: for(int i=0; i<N; i++)
: 以上這些都沒有錯，沒有誰的才是對的，誰才是錯的，重點流程有沒有錯，有沒有bug，
: 執行會不會慢，巢狀迴圈幾層。
: 執著在那些格式很沒有意義，或誰誰誰寫code格式不符合我意的，就把別人弄走。
: 你不能說你就是標準，全部人都要跟你的寫法一模一樣，很多人寫程式想的是這個問題要
: 怎麼寫才巧妙解決，而不是十分在乎格式，太執著就有強迫症或太龜毛，合作起來也很痛
: 苦。
: 放過別人也放過自己，互相尊重。
--
https://i.imgur.com/QDN9AhN.jpeg
紫楓創作：
https://portaly.cc/tbpfs
我是AI王紫楓
你可以叫我AI王
也可以叫我AI王子
也可以叫我AI王子瘋
--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 123.205.130.61 (臺灣)
※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1721717892.A.838.htmllabbat1樓遇到if-else完整陳述語法就沒辦法這樣偷吃步了 07/23 15:21
chihlee55662樓Conventions? 07/23 15:55
→ stepnight3樓senior 最大的問題是 知識的詛咒 07/23 15:58
→ brucetu4樓每個都有else就不能像你說的這樣改 07/23 16:14
→ brucetu5樓真的遇到這種狀況只能把條件參數化再寫成其他形式，就像r 07/23 16:17
→ brucetu6樓oute一樣，但也有可能到最後你發現還是if else最好維護， 07/23 16:17
→ brucetu7樓而且在某些很在意延遲的場景，if else更好 07/23 16:17
→ shooter5558樓conversation 07/23 16:44
→ shooter5559樓太多層就是 邏輯不夠明確 接手的人很痛苦 07/23 16:46
→ ma72110樓沒事找事，太閒 07/23 16:53
→ 本人11樓靠邊 手指太肥了點到conversation XD 07/23 17:27
→ acer1832a12樓我記得一個func裡多個return，這種方式不是不建議使用? 07/23 17:28
→ 本人13樓你說的可能是很久以前的寫法不建議使用 07/23 17:29
→ 本人14樓但現在為了readability, 都是用這種寫法 07/23 17:29
→ a381700115樓early return現在還滿常見的 07/23 17:45
f82102716樓一個func多個return滿常在leetcode most vote 看到 07/23 17:46
→ atst217樓early return 還是要看一下返回的理由是什麼比較好. 07/23 18:09
→ atst218樓一般的建議還是用在檢視輸入的資料有沒有符合規則. 07/23 18:10
atst219樓然後，下面的例子有點極端 kubernetes裡的pc_controller.go 07/23 18:18
→ atst220樓1714行，充滿if-else，作者一開頭就告訴你:別亂改，每個 07/23 18:19
→ atst221樓if-else都有意義. 07/23 18:19
→ atst222樓以原發文的例子而言，有沒有可能，就是因為junior，為了避 07/23 18:24
→ atst223樓免犯錯，才大量使用if-else去描述每一個路徑? 07/23 18:25
gino071724樓如果不能return必須繼續做下去怎麼辦 07/23 19:14
→ gino071725樓我在c++裡面要解析json我都會先給一個if看欄位在不在 07/23 19:15
→ gino071726樓然後再一個if看到底是array還是object 再一個if看到底是 07/23 19:15
→ gino071727樓字串還是數字 最後才開始做事 07/23 19:15
abccbaandy28樓C++ json還要手寫parser？ 07/23 19:40
→ DrTech29樓沒看過程式碼真的別太篤定對錯。 07/23 19:51
karst1060730樓early return 感覺還比較適合多數情境，防呆機制比起 07/23 22:12
