這是我"前"公司的經驗了
一開始以為公司內有嚴格的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的公司
一開始以為是檢到寶
後來沒想到工作沒滿試用期就離職了....