这是我"前"公司的经验了
一开始以为公司内有严格的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的公司
一开始以为是检到宝
后来没想到工作没满试用期就离职了....