Re: [討論] 因為空格~我離開了一間公司

[littlethe (東周小星星)]

我說說我對code review的想法,

我認為code review就像法律一樣,

這社會要有制度的運作,一定要有法律,

但若是這個法律是總統隨他高興說了就算的話,

那這個法律會是很爛的法律,

說穿了也不是什麼法律,根本就是命令,

那還不如不要有,

因為這種隨個人喜好而制定出的"法律"往往最後只是成為整人的工具,

一個好的法律是要一群人討論後制定出的,

那code review也是這樣,

在review之前,

先大家一起制定好code standard,

說好要尊守這個standard,也了解狀況後,

再依據standard做code review,

只要符合standard的話,

裡面怎麼寫那就是coder的自由,

不會造成"問題"的話,

reviewer不應該去干涉coder的寫法,

畢竟程式這種東西還是有些地方是主觀的,

管太細會引起不必要的爭執,

我本身是很認同要有code review,

但我發現很多人,好像是想要表現出自己很強,

就會把code review弄得很像是批鬥大會...

code review對我來說主要目的就兩個,

一個是確認code有沒有符合"大家約定好的"standard,

二個了解對方怎麼寫,而不是要逼對方怎麼寫,

若在code review時刁別人,

並不會讓別人覺得reviewer很強,

只會讓別人覺得這個reviewer很閒或存心找碴,



我是曾經遇過2種不好的code review,

第一種是老鳥沒有遵守code standard,

也不用被code review,

而我呢,是新人,就被code review啦,

我原本程式寫成A,被code review後,

被要求改成B,再被code review後,

被reviewer指責說為什麼要寫成B,

要我改回A...

我心裡就覺得這reviewer是來亂的...


第二種是reviewer程式功力很差,

他不會用function,

而我遇到重覆的code會寫成function,

他看到function就要我把function拿掉,

唸說寫function別人會"看不懂"..


原PO真的不用難過啦,

我自己寫case 1的時候,

我的寫法也是和你一樣的,

你前主管沒看過的話,

那真的程式看得太少了...

我自己是遇過一種狀況,

我刮號會習慣打成

if(){
  XXX;
}

但我有個前同事看到我這樣寫,就罵我亂寫,他只看得懂

if()
{
  XXX;
}

我也好奇大家會怎麼看待這件事,

我對code review這件事也是有些感慨,

本來是一件很好的事,

結果常常被扭曲成"政治工具"


※ 引述《workworkwork (Miyada vv)》之銘言：
: 這是我"前"公司的經驗了
: 一開始以為公司內有嚴格的coding style規定是件好事
: 我也贊成公司要有一致的coding style
: (像我以前看過apache的C code
:  全部CODE都像同一人寫出來的一樣)
: 而公司內也會有人code review你的部份
: 一切聽起來都很完美
: 一開始聽到有規定coding style和code reviewer也很開心
: 但因這一切都因為公司裡有一個奇怪的規定而毀了
: "code不可以用code formatter去掃"
: 我承認自己寫程式常會漏勾
: 所以寫完會花很多心力在檢查有沒有BUG 是否會被攻擊 資安問題等等....
: 但在這間公司發現一個很奇怪的事情
: "有資安漏洞的CODE大家會很有耐心的教 空格沒空好會被罵的狗頭淋頭"
: 搞到最後一段程式寫完我只知道檢查空格....
: 最後的最後我決定離職的原因是出在reviewer
: 和reviewer的code觀念差太多 跟本無法共事
: 例如：
: 1.
: 有時為了避免太多層出現===>
: if(a)
: {
:   //do a things
:   if(b)
:   {
:     //do b things
:     if(c)
:     {
:       //do c things
:     }
:   }
: }
: 會改成====>
: if(!a)
: {
:   return ;
: }
: //do a things
: if(!b)
: {
:   return ;
: }
: //do b things
: if(!c)
: {
:   return ;
: }
: //do c things
: 但因為這寫法code reviewer沒看過
: 她直接在辨公室裡開飆
: 2.
: reviewer說出了我聽說最莫名奇妙的一句話
: "這function以前沒人用過 所以你不能用這function"
: (jquery的trigger)
: 不是因為效能 不是因為安全問題 而是因為公司前人沒用過....
: 我第一次遇到會規定coding style的公司
: 一開始以為是檢到寶
: 後來沒想到工作沒滿試用期就離職了....

--
寫程式是一種信仰，
寫得出來是一種藝術，
寫不出來是一種哲學。

--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 61.224.80.33
※ 文章網址: http://www.ptt.cc/bbs/Soft_Job/M.1410275065.A.CAB.html
推 yyhsiu: JAVA我會把{放行尾 C++會換行
→ littlethe: 為什麼呢?
→ yyhsiu: 奇怪的習慣 不過行尾好像符合google的style
→ yyhsiu: 主要是建立習慣時期看他人的code學下來的吧
→ yyhsiu: 在公司是只寫過Java啦 規定是行尾
→ littlethe: 你習慣真特別XD
→ hSATAC: go fmt 下去大家 code 都長一樣 超棒的
推 askacis: {放行尾就 K&R 風格啊，一堆人在用的東西說沒看過...
推 GX90160SS: 比較喜歡換行，這樣括號才有對稱性看起來舒服
→ askacis: 叫那個阿菜reviewer查一下K&R，真的是菜蟲掉滿地XD
→ superpai: 我還以為是javascript
推 PICASSO1: askacis正解!!
推 ayumiayayaai: golang 讚讚
推 doraemon1293: 我的習慣跟一樓一樣耶
推 snaketsai: 我是習慣用第二種，不過我是用Whitesmiths style
→ snaketsai: 不過這點我覺得真的就是有固定、不要混用就好...
→ GoalBased: 整個公司都不會用function的團隊會決定要code r ...?
推 b6byc: 括號兩種都看過,我都用耶.
推 azureblaze: 個人覺得括號只是潔癖而已混用也無所謂在意這個很無聊
→ azureblaze: 如果位置會造成閱讀上的影響那代表你括號太多層了
推 GoalBased: ㄓ音文也看得懂阿，不影響閱讀嗎
推 vampirewalk: 我也支持go fmt
推 bobju: 看樣子還是需要標準化的美化工具, 工具之前人人平等 ;)
→ bobju: 至於風格什麼的, 真的就只是看得順不順眼而已, 影響才沒那
→ bobju: 麼大, code寫得夠久的話, 各種風格都會經歷過, 為這個起爭
→ bobju: 執真的是無事忙而已
→ bobju: code review 的重點應該擺在這 code 是否符合一些資料安全
→ azureblaze: 還可以來戰tab還是space 我認為全形空白以外都是垃圾
→ bobju: 性, 易讀性, 結構性, 演算上的效率這些項目上, 而不是在搞
→ bobju: 形式
→ littlethe: 沒錯，有各種風格，沒什麼好壞，只是從中選一個，但糟
→ littlethe: 糕的是有人的心態變成是去瞧不起別人
→ littlethe: 看了大家的討論，好像愛review 別人的人，coding 就不
→ littlethe: 怎麼樣的感覺，半瓶水響叮噹
→ azureblaze: 因為講不出東西的時候挑剔風格最容易啊XD
→ littlethe: 見過大陣仗看過許多code 的人，只會在意bug 和效能這
→ littlethe: 種比較實際的東西，而非雞皮蒜毛，也沒空管小事
推 bobju: 其實現在很多code都是從網路上抓下來安裝引用, 甚至是直接
→ bobju: 嵌入到自己的code當中, 若真的要求code style的風格一致的
→ bobju: 話, 那天天就改這些code就飽啦 :P
→ bobju: 而這些公開發佈的code已有一定的可信度了, 為了code style
→ bobju: 是否一致而改code, 衍生出一些莫名其妙的bug 又要debug,那
→ bobju: 真的是補鍋師傅在幹的事
→ cha122977: 同一樓 其實就是看現在寫的程式碼通用的習慣是什麼
→ cha122977: 像kernel的function不放尾行 但if放尾行這種也是有呀
→ meowyih: 無法接受 bobju 說的 "目前很多 code 都直接抓下來引用"
→ meowyih: 的說法, 抓下來的東西的 code 長什麼樣子我是管不著, 但
→ meowyih: 要使用這種像 open source 的東西一定都會先 build 成
→ meowyih: library 的型式, 不然出事了怎麼切分問題或責任所在呢?
→ littlethe: 沒build 是還ok ，但要獨立成不同檔案，不要和自己的co
→ littlethe: de 混在一起
→ askacis: 抓下來的code不build要怎麼用XD 一般都直接build library
→ askacis: 在自己的code裡呼叫library，很少數情況下才會抽code出來
→ askacis: 跟自己的Code綁在一起build~
→ askacis: 通常都是嵌入式環境下求快以及減少flash size才會考慮~