你们不要以为老外的开发质量就怎样了

sumtec 发布于 2014年01月21日 | 更新于 2014年01月28日
tinyfool leon_lu 等2人欣赏。

这是一位同事的注释:

//It seems too hard to implement bodyshop editing within existing code.
//So, just remove the old part and than add the new one.

part在这里是部件的意思。说白了他的意思就是:

// 哇,原来的代码我读不懂啊。
// 所以,我就把旧记录删了然后把新的加上去就好了。

擦,你读不懂就不要乱改嘛,改出个大bug来。人生最怕遇到这样的猪队友了,自己觉得自己干了件好事屁颠屁颠的,却要别人给你擦屁股。

擦屁股就算了,看在你好歹写了一句注释的份上。不过悲剧的是,就只有这么一句注释,也不知道这个改动到底跟哪些改动有关系,背后的动机是什么,解决了什么问题,尼玛只能靠猜了。而且他的重构规模还不是一般的大,面目全非了只好慢慢猜。

所以说,一定要有单元测试来避免这种猪队友自觉高大上乱改一气。

共28条回复
tinyfool 回复于 2014年01月21日

打破老外都是神开发的神话,不过,说起来,老外当然也有二货了,不是么,不过我们产量可能更大吧

灵感之源 回复于 2014年01月21日

bodyshop是那个出名的bodyshop?

没有神话,外国普遍代码要规范些,中国个体写的代码精彩。

原来公司一个代码文件原来3万多4万行代码的样子,600还是700多个全局变量,后来给建议他改,最后改成2万多行,死活不拆文件,还是很多冗余的,最后我也放弃了。

代码没有规范就不说了,命名看不懂,而且还没注释,这样的代码谁去维护。

tinyfool 回复于 2014年01月21日

2楼 @灵感之源 这样的可以直接开除了,就算是神仙,能写出来这样的代码,也没有神仙可以维护这样的代码啊

minddriven 回复于 2014年01月21日

当前我的项目是我一人 from scratch 写起的。吐槽的话,我只能吐槽 opensource lib 。当然也在 github 上给他们提交过几个 bug。

灵感之源 回复于 2014年01月21日

4楼 @minddriven

以前做非green field项目也遇到过吧

leon_lu 回复于 2014年01月21日

代码规范小日本搞得还是比较好的,第一次打开日方的代码文件时,被那注释给震住了,注释虽然精彩,但是有些代码写的真不咋地啊,同事们修改代码的时候总是边改边骂娘

minddriven 回复于 2014年01月21日

最好的代码就是代码本身就是注释,注释只要两三句话提点一下。

我会想个很好的变量名,很好的方法名,单写注释就比较懒。

minddriven 回复于 2014年01月21日

5楼 @灵感之源 遇到过,并不吃惊。好代码是相对少数,差的代码是相对多数,与肤色无关。

我遇到过最雷人的代码其实是国内,变量名是 a134, b234, csdf 之类的… ,完全不知道那家伙头脑在想神马。

BarryWey 回复于 2014年01月21日

如果能够确定楼主那段代码不是印度程序员写的,那还有点意思。

tinyfool 回复于 2014年01月21日

8楼 @minddriven 人肉混淆器

灵感之源 回复于 2014年01月21日

8楼 @minddriven

中文变量,函数名,类名,文件名,数据库表、字段统统是中文,这才头痛

tinyfool 回复于 2014年01月21日

11楼 @灵感之源 这种人直接让他失业好了

akunamotata 回复于 2014年01月21日

11楼 @灵感之源 这会导致很多问题吧。。。坑爹呢吧。。。

hashmap 回复于 2014年01月21日

做了10年对日外包的路过。 这年头日本的代码都是靠中国人写的。

sumtec 回复于 2014年01月21日

2楼 @灵感之源 显然不是那个Bodyshop。这里Bodyshop就是指修车店的意思。

sumtec 回复于 2014年01月21日

8楼 @minddriven 你过将View命名为Model,然后将Controller命名为View的吗?看的时候要切换到另一种思路。而且同一个实体有两种,哦不对是四五种不同的格式,然后切换来切换去,其中有3种还是xml,名称类似格式“稍有”不同……

minddriven 回复于 2014年01月21日

16楼 @sumtec 这才是真正的人肉混淆器吧?

tinyfool 回复于 2014年01月21日

16楼 @sumtec 赞,你赢了

灵感之源 回复于 2014年01月21日

15楼 @sumtec

干,下次拜托分开写。。。

敏感字符 回复于 2014年01月21日

比如- rm -rf /usr /lib/nvidia-current/xorg/xorg

敏感字符 回复于 2014年01月21日

7楼 @minddriven 写代码的时候很少写注释,但是想变量名和方法名会考虑很久,为的就是以后不看注释光看名字就能知道这是干什么的……

enno 回复于 2014年01月21日

能感受到LZ第一眼看到后的微微一笑和内心的鄙视,接下来的几个小时,苦逼的改bug吧。

sumtec 回复于 2014年01月22日

19楼 @灵感之源 那个是老外写的注释,不是我写的……

saquer0716 回复于 2014年01月22日

看多了就不觉得气怪了。哪都一样,最常见的就是业余的命名和臃肿的源文件结构。

之前看过一个斯里兰卡人的项目代码,维护指数0.5(满分10分),除了无法理解的混乱命名,还有互不相干的源文件集中在一个文件夹。看了几分钟我就开始犯恶心。最奇怪的是这竟然是给American Express做的项目,真不知道怎么通过review的。

其实命名这个东西很能区分程序员,好的文件名展现了程序员对文件的深入理解。菜鸟程序员写第一个文件一般随便起个名字了事,后来写了一个类似的文件,懒得仔细思考,就加上数字后缀来区分,完全不去思考是否需要重构。之前看过一个调查,程序员写代码排名第一的困难就是命名,觉得不困难的,肯定不是好程序员,呵呵~

evildickman 回复于 2014年01月27日

看了这篇文章学习了很多啊。。。

xiongxoy 回复于 2014年01月27日

把注释写成函数名,然后调用。听thoughtbot的podcast说,新手的代码里如果出现大量注释,说明功能抽象的还不够细。

Million 回复于 2014年01月28日

24楼 @saquer0716 命名很困难,说多了都是泪啊。我见过用拼音命名的,这不离谱,离谱的是用拼音首字母命名。。。。。。

灵感之源 回复于 2014年01月28日

24楼 @saquer0716

斯里兰卡就是印度风格了,兄弟国家,原来公司老总就是斯里兰卡人,招的员工大部分是印度人

本帖有28个回复,因为您没有注册或者登录本站,所以,只能看到本帖的10条回复。如果想看到全部回复,请注册或者登录本站。

登录 或者 注册
[顶 楼]
|
|
[底 楼]
|
|
[首 页]