Amatista 发表于 2012-12-6 10:46
我们看项目的，有个项目头很看重变量名的，名字起的不好，还会被说的，连行距都要管。
恩 我以前公司所有Code 必须过Code Review. 别说是naming convention就是我的Tab 是4个space 都被说 因为他们的是2个
Amatista 发表于 2012-12-6 09:40 static/image/common/back.gif
就是这样的。但是，他很会说话，把歪理说得头头是道。让他的Betreuer很无奈，经常哭笑不得。不过，他合同 ...
你说的这个人是不是个阿拉伯人？在Wolfsburg工作？
本帖最后由 Amatista 于 2012-12-6 11:20 编辑
elektopia 发表于 2012-12-6 11:09 static/image/common/back.gif
你说的这个人是不是个阿拉伯人？在Wolfsburg工作？
不是的，是埃塞俄比亚人。我是在斯图这边。
little_sunshine 发表于 2012-12-6 11:07 static/image/common/back.gif
恩 我以前公司所有Code 必须过Code Review. 别说是naming convention就是我的Tab 是4个space 都被说...
这个就看出来差距来了，农村小公司从来就没有review的掩面飘过。。。。。。
老湿不给力 发表于 2012-12-6 11:06 static/image/common/back.gif
你们头的意思就是任何变量名都应该有意义？
对啊
而且如果你写的有含义的变量名，他觉得不够贴切，他还要自己改的。
一次他改了本来有我负责的那部分里的变量名，后来那部分出了个小问题。一开始他还问我，是不是我没把我rechner上的aenderungen都上传到server上啊
后来，我发现是他几个月前改了变量名。正常情况下改变量名是不会出问题的。但是那个部分里面有个结构很十三，是运行期间根据变量名找localizedText。他这名字一改，相应localizedText找不到，自然就出问题了。这个很十三的结构是之前就有的，不是我搞出来的。
little_sunshine 发表于 2012-12-6 11:07 static/image/common/back.gif
恩 我以前公司所有Code 必须过Code Review. 别说是naming convention就是我的Tab 是4个space 都被说...
就是就是这样
空行也要看
kommentar也有人管
你说你coding的时候，很正常会把某些句子auskommentieren来做调试。如果一下子要改很多的话，忘记某个部分其实可以删除也正常吧。反正被要求，已经是auskommentieren的语句就要删除。没删除也会被说。你要找以前写过的code，要去repository找。
Amatista 发表于 2012-12-6 11:15 static/image/common/back.gif
对啊
而且如果你写的有含义的变量名，他觉得不够贴切，他还要自己改的。
那比如对一个数组进行遍历，那个循环数（一般是i）能有什么意义？还不就是信手拈来，想怎么命名就怎么命名，i,j,k随心所欲呗
老湿不给力 发表于 2012-12-6 11:20 static/image/common/back.gif
那比如对一个数组进行遍历，那个循环数（一般是i）能有什么意义？还不就是信手拈来，想怎么命名就怎么命名 ...
你说的这个情况，i之类的还是可以用的
Amatista 发表于 2012-12-6 11:18 static/image/common/back.gif
就是就是这样
空行也要看
kommentar也有人管
你们是专业的软件公司吧，感觉写软件好专业啊{:5_336:}
老湿不给力 发表于 2012-12-6 11:20 static/image/common/back.gif
那比如对一个数组进行遍历，那个循环数（一般是i）能有什么意义？还不就是信手拈来，想怎么命名就怎么命名 ...
不过 i 和 j 太想了要是inner loop 用j 的话 outer loop 我不会用i.. 万一mix up 很难找 (.. 血的教训啊