苦逼的程序员

little_sunshine · 发表于 2012-12-6 11:07

Amatista 发表于 2012-12-6 10:46
我们看项目的，有个项目头很看重变量名的，名字起的不好，还会被说的，连行距都要管。

恩我以前公司所有Code 必须过Code Review. 别说是naming convention 就是我的Tab 是4个space 都被说因为他们的是2个

elektopia · 发表于 2012-12-6 11:09

Amatista 发表于 2012-12-6 09:40
就是这样的。但是，他很会说话，把歪理说得头头是道。让他的Betreuer很无奈，经常哭笑不得。不过，他合同 ...

你说的这个人是不是个阿拉伯人？在Wolfsburg工作？

Amatista · 发表于 2012-12-6 11:11

本帖最后由 Amatista 于 2012-12-6 11:20 编辑

elektopia 发表于 2012-12-6 11:09
你说的这个人是不是个阿拉伯人？在Wolfsburg工作？

不是的，是埃塞俄比亚人。我是在斯图这边。

老湿不给力 · 发表于 2012-12-6 11:14

little_sunshine 发表于 2012-12-6 11:07
恩我以前公司所有Code 必须过Code Review. 别说是naming convention 就是我的Tab 是4个space 都被说 ...

这个就看出来差距来了，农村小公司从来就没有review的掩面飘过。。。。。。

Amatista · 发表于 2012-12-6 11:15

老湿不给力发表于 2012-12-6 11:06
你们头的意思就是任何变量名都应该有意义？

对啊

而且如果你写的有含义的变量名，他觉得不够贴切，他还要自己改的。

一次他改了本来有我负责的那部分里的变量名，后来那部分出了个小问题。一开始他还问我，是不是我没把我rechner上的aenderungen都上传到server上啊
后来，我发现是他几个月前改了变量名。正常情况下改变量名是不会出问题的。但是那个部分里面有个结构很十三，是运行期间根据变量名找localizedText。他这名字一改，相应localizedText找不到，自然就出问题了。这个很十三的结构是之前就有的，不是我搞出来的。

Amatista · 发表于 2012-12-6 11:18

little_sunshine 发表于 2012-12-6 11:07
恩我以前公司所有Code 必须过Code Review. 别说是naming convention 就是我的Tab 是4个space 都被说 ...

就是就是这样
空行也要看
kommentar也有人管
你说你coding的时候，很正常会把某些句子auskommentieren来做调试。如果一下子要改很多的话，忘记某个部分其实可以删除也正常吧。反正被要求，已经是auskommentieren的语句就要删除。没删除也会被说。你要找以前写过的code，要去repository找。

老湿不给力 · 发表于 2012-12-6 11:20

Amatista 发表于 2012-12-6 11:15
对啊

而且如果你写的有含义的变量名，他觉得不够贴切，他还要自己改的。

那比如对一个数组进行遍历，那个循环数（一般是i）能有什么意义？还不就是信手拈来，想怎么命名就怎么命名，i,j,k随心所欲呗

Amatista · 发表于 2012-12-6 11:21

老湿不给力发表于 2012-12-6 11:20
那比如对一个数组进行遍历，那个循环数（一般是i）能有什么意义？还不就是信手拈来，想怎么命名就怎么命名 ...

你说的这个情况，i之类的还是可以用的

老湿不给力 · 发表于 2012-12-6 11:23

Amatista 发表于 2012-12-6 11:18
就是就是这样
空行也要看
kommentar也有人管

你们是专业的软件公司吧，感觉写软件好专业啊

little_sunshine · 发表于 2012-12-6 11:36

老湿不给力发表于 2012-12-6 11:20
那比如对一个数组进行遍历，那个循环数（一般是i）能有什么意义？还不就是信手拈来，想怎么命名就怎么命名 ...

不过 i 和 j 太想了要是inner loop 用j 的话 outer loop 我不会用i.. 万一mix up 很难找 (.. 血的教训啊

账号		自动登录	找回密码
密码			注册

萍聚头条

[心情] 苦逼的程序员