PTT評價

Re: [請益] 痾 遇到這種事情 是不是需要趕快離職了?

看板Soft_Job標題Re: [請益] 痾 遇到這種事情 是不是需要趕快離職了?作者
tbpfs
(http://portaly.cc/tbpfs)
時間推噓23 推:23 噓:0 →:54

等等,我原本以為只是一個簡單的問題

居然歪樓了

推動coding conventions 可以從你我做起

像原原po的問題是

if
if
if
if




把判斷式改過來變成
if
return;
if
return;

即可
這個就簡單起草一份coding conventions
拿給長官review, 以後code review 看到這個問題
就直接貼連結請junior 改就好了
這種東西很多學生時期根本沒碰過
自然就會波動拳出現
跟頂不頂大沒關系

反而是senior 不知道怎麼幫助junior
才是問題

※ 引述《purin88 (原來我是憤怒的鄉民)》之銘言:
: 我從上面的文章只看到原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 留言評論
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 123.205.130.61 (臺灣)
PTT 網址

labbat07/23 15:21遇到if-else完整陳述語法就沒辦法這樣偷吃步了

chihlee556607/23 15:55Conventions?

stepnight07/23 15:58senior 最大的問題是 知識的詛咒

brucetu07/23 16:14每個都有else就不能像你說的這樣改

brucetu07/23 16:17真的遇到這種狀況只能把條件參數化再寫成其他形式,就像r

brucetu07/23 16:17oute一樣,但也有可能到最後你發現還是if else最好維護,

brucetu07/23 16:17而且在某些很在意延遲的場景,if else更好

shooter55507/23 16:44conversation

shooter55507/23 16:46太多層就是 邏輯不夠明確 接手的人很痛苦

ma72107/23 16:53沒事找事,太閒

※ 編輯: tbpfs (123.205.130.61 臺灣), 07/23/2024 17:26:54 ※ 編輯: tbpfs (123.205.130.61 臺灣), 07/23/2024 17:26:59

tbpfs07/23 17:27靠邊 手指太肥了點到conversation XD

※ 編輯: tbpfs (123.205.130.61 臺灣), 07/23/2024 17:27:52 ※ 編輯: tbpfs (123.205.130.61 臺灣), 07/23/2024 17:28:15

acer1832a07/23 17:28我記得一個func裡多個return,這種方式不是不建議使用?

tbpfs07/23 17:29你說的可能是很久以前的寫法不建議使用

tbpfs07/23 17:29但現在為了readability, 都是用這種寫法

a381700107/23 17:45early return現在還滿常見的

f82102707/23 17:46一個func多個return滿常在leetcode most vote 看到

atst207/23 18:09early return 還是要看一下返回的理由是什麼比較好.

atst207/23 18:10一般的建議還是用在檢視輸入的資料有沒有符合規則.

atst207/23 18:18然後,下面的例子有點極端 kubernetes裡的pc_controller.go

atst207/23 18:191714行,充滿if-else,作者一開頭就告訴你:別亂改,每個

atst207/23 18:19if-else都有意義.

atst207/23 18:24以原發文的例子而言,有沒有可能,就是因為junior,為了避

atst207/23 18:25免犯錯,才大量使用if-else去描述每一個路徑?

gino071707/23 19:14如果不能return必須繼續做下去怎麼辦

gino071707/23 19:15我在c++裡面要解析json我都會先給一個if看欄位在不在

gino071707/23 19:15然後再一個if看到底是array還是object 再一個if看到底是

gino071707/23 19:15字串還是數字 最後才開始做事

abccbaandy07/23 19:40C++ json還要手寫parser?

DrTech07/23 19:51沒看過程式碼真的別太篤定對錯。

karst1060707/23 22:12early return 感覺還比較適合多數情境,防呆機制比起

karst1060707/23 22:12風格是更順暢的理由

viper970907/23 23:49推這篇

crazwade07/24 00:40推 剛入行也是針對這點有被前輩教育過

crazwade07/24 00:40後來也有像是用枚舉或是switch 來取代 if else

crazwade07/24 00:41只能說原 po可能有盡力 但不是當事人不好評論

saladim07/24 00:51其實若if-else或switch裡面思路清楚 不會是個問題 而且就

saladim07/24 00:51我少少的經驗來說一但思路清楚 大概也不太會形成很深的

saladim07/24 00:52if/switch 很難用例子去闡述那種if顯示出來的思路雜亂跟

saladim07/24 00:53跟理解/修改的困難(更難驗證邏輯正確性)

saladim07/24 00:54@atst2 有可能是避免犯錯 但我也跟其他人一起討論當時案

saladim07/24 00:56例 一旦分析好狀況 可以寫出用少量的if/switch的同樣功能

saladim07/24 00:57也有分享給對方...比較難理解的是 下次遇到同樣需要分析

saladim07/24 00:58時 還是用同樣方式描述路徑是有點意外

saladim07/24 00:59補充:當然也不排除有上面提到的那種極端狀況. 至少不是現

saladim07/24 00:59在遇到的

joeboy07/24 01:30我之前被review也是被教early return

neo527707/24 01:37請愛用策略工廠

yamagishi07/24 06:24你邏輯沒理清楚才會覺得 if else 是 must

yamagishi07/24 06:27怎麼會有人問必須做下去該怎麼辦…OMG…

yamagishi07/24 06:32Guard Clauses 跟 EAFP 有空可以去了解一下

brucetu07/24 07:43三小,就是會有if else must的狀況啊

ssteves07/24 08:21early return能提高程式碼的可讀性、維護性,而且

ssteves07/24 08:21也可以減少不必要的計算資源

bnd032707/24 10:14如果該函式的行為有清楚定義並做好單元測試,裡面if迴圈

bnd032707/24 10:15複雜一點好像也還好。

yamagishi07/24 10:16不可能會有,另外寫一個 function 在那邊 early return

yamagishi07/24 10:16 而已

yamagishi07/24 10:17不然你提出一個例子我們討論看看

yamagishi07/24 10:19這東西你最終只要注意是 Passing By Pointer 還是 Pass

yamagishi07/24 10:19ing By Reference 你就能做出你要的東西了

chal07/24 12:37會寫成巢狀if 也可能是歷史造成的 前人寫 後人不敢動太大

brucetu07/24 14:33你當然可以把內層的if-else拉出去另一個function取個名字

brucetu07/24 14:33對於要trace整個狀況的開發者來說, 邏輯的複雜性沒有降低

brucetu07/24 14:34有時候反而跳來跳去更痛苦

sasoman07/24 14:38guard clause 不是蠻基本的嗎

sharek07/24 15:51guard clause, early return 都看情況不用在那邊文人相輕

Obama1907/24 17:07五十步笑百步 一堆early return有比較好嗎

abccbaandy07/24 18:13不好嗎? 哪個條件不要了就選起來刪掉就好,完全不用

abccbaandy07/24 18:14研究那堆if else

brucetu07/24 19:37if else 還不是一樣選起來刪掉-.-

brucetu07/24 19:39真值表畫出來就知道你early return不會讓複雜度降低只是

brucetu07/24 19:39程式碼語法差異而已

abccbaandy07/24 20:50波動拳你怎麼選...

Csir07/24 23:14我之前也看過一堆goto差點自焚

abraxas07/25 00:07坐等例子

akito11707/25 18:49early return 在做防呆類型好用,可以簡化if else,讓

akito11707/25 18:49邏輯清楚一點。