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

※ 引述《saladim (殺拉頂)》之銘言：
: 小魯目前在一家還算大的公司工作 現在有兩三位頂大的junior的同事
: 寫程式的習慣讓我覺得是不是要趕快跑了 舉兩個例子好了
: 他們都喜歡if-else combo, 沒巢狀到波動拳那麼深 但就是動不動就if-else
: 三層 然後三層裡面還會再if-else
: 第二個例子就是如果有function 90%(50~100行)適合他們想要的用途,
: 他們就會copy整個function 然後修改一下後"整段"code插到他們需要的地方
: 光是上面兩個例子已經讓我的頭超大的 原本公司的codebase已經長得像科學怪人
: 了 然後又有他們持續"加持" 根本就沒辦法理解
: 更慘的是 跟他們講他們也不當一回事 又有頂大當紅碩論護體 一副你也不敢砍我的底氣
: 公司也沒人想要當壞人 code review也沒人出聲 而且大家都知道 上市公司每個都喜歡: 有學歷的人當門面 反正真正主力有人會扛 XDDDD
: 每次改到他們經手過的code都很痛苦 若是要幫忙擦屁股根本擦不完阿 因為一直拉....: 自己寶貴的時間也都被吃掉了
: 這樣是不是該走了比較好?
: 我知道爛code到處都是啦 但是至少不能一直拉吧 又是都講不聽的 更資深的也都能閃就?
: 但大家輪流中槍.............
我從上面的文章只看到原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格式不符合我意的，就把別人弄走。
你不能說你就是標準，全部人都要跟你的寫法一模一樣，很多人寫程式想的是這個問題要怎麼寫才巧妙解決，而不是十分在乎格式，太執著就有強迫症或太龜毛，合作起來也很痛苦。
放過別人也放過自己，互相尊重。

--

我個人認為執行效率，bug rate，unit test，開發速度 比起仔細每個程式碼對有幾個空格，程式碼段落之間有沒有空行，大方向來說更有效率， 程式設計師的目標要產出沒有bug執行速度又快，程式碼又簡潔，開發速度又快。 code review是要讓其他人了解你程式的邏輯思維有沒有盲點，順便也是交接的過程，如 果以後離職別人也懂程式碼流程。

二個都很重要。但如果遇到一直挑剃應該多空一格少空一格，大括號這樣括號，不要有段 落空行， 而不是著重在流程，效率計算，使用函式模板 類別模板（c++)，java是泛型<T>，程式碼 是那裡會用到模板，我個人覺得理解思維比較重要