吐槽一下,同事把java写的像C

2013-06-17 11:11:13 +08:00
 moroumo
公司内基本都是用java开发的。前段时间我转入一个新项目组,这个项目组建立几年了。和几个已在项目组的老员工一起开发了新项目,现在code进入到review阶段。写核心代码的两个同事,一个资深的,一个毕业一年的。我在review代码的时候,发现这两个同事基本不讲究OO设计之类的。代码就是几个大函数,基本没有抽象。要说是C风格,也是比较差的C风格,模块划分也是比较差的。
项目至少会持续两年左右,迭代开发,会不停的增加feature。所以后续再增加代码可能非常痛苦。

我虽是老员工,但是相对这个项目组,还很新。有几次旁巧側击的提过代码的问题,回答就是项目紧,或者refactor也就一两天的事,不着急。按照多年的经验,代码一旦完成,老板很难给时间去refactor的。
7916 次点击
所在节点    程序员
50 条回复
kfc315
2013-06-17 20:21:04 +08:00
@ivenvd 哈哈哈……这
leegorous
2013-06-17 20:58:14 +08:00
连个风格都要给人吐槽,写代码真不容易啊
jasontse
2013-06-17 21:01:32 +08:00
@ivenvd 无力吐槽。。。
davepkxxx
2013-06-17 21:24:14 +08:00
果然厉害
ivenvd
2013-06-18 01:38:55 +08:00
@wang2191195 @kfc315 @jasontse 你们啊,Too Simple,这只是我随手找来的而已…… &_&

我们想用 Python 发个 POST 请求,都是 fork 出一个 curl,然后解析输出的…… &_&
stranbird
2013-06-18 12:40:45 +08:00
@ivenvd 代码最少的实现,挺好的啊。
zhwei
2013-06-18 12:42:28 +08:00
@chemhack 厉害!
bradlee
2013-06-18 22:21:44 +08:00
@clowwindy 这个正解 写的像C我理解为像linux kernel 或nginx那样的
luikore
2013-06-18 22:23:58 +08:00
这是赞美吧...
colincat
2013-06-18 23:31:12 +08:00
我无力吐槽我自己,我写的代码别人都能看懂,难道我也是c风格?要怪就怪你们的架构师吧
pepsin
2013-06-19 07:07:07 +08:00
@railgun 想象一下Rails的Controller写了4000多行
alexrezit
2013-06-19 07:31:49 +08:00
hanf
2013-06-19 08:39:25 +08:00
@moroumo 我表示非常同意,这表面上是风格问题,其实深层是流程和规范问题。要么不专业,要么偷懒不动脑。
victor
2013-06-19 16:07:22 +08:00
@pepsin 跪了,如何写出4000行,求指教。
pepsin
2013-06-19 17:17:39 +08:00
ivenvd
2013-06-20 00:09:51 +08:00
@alexrezit 哈哈,你这个只是不会循环而已,我这个可是不会数组啊……显然实现你那个功能难度更高……
Abruzzi
2013-07-02 15:54:39 +08:00
@ivenvd 之前看了那个switch的代码愕然了,然后仔细一想好像也不是那么容易改,最后用反射写出来:
public void testSwitch(Integer i){
try {
Field field = Constants.class.getDeclaredField("AGENT_" + i);
agentType = (String)field.get("AGENT_" + i);
} catch (NoSuchFieldException | SecurityException | IllegalArgumentException | IllegalAccessException e ) {
e.printStackTrace();
}
}
觉得还不如之前那样安全。


今天又看到你回复的数组,瞬间想捡砖拍死自己。
tolbkni
2013-07-02 19:18:28 +08:00
@Abruzzi 跪了!瞬间高端了
Hashell
2013-07-02 19:43:51 +08:00
@ivenvd ..真巧,我今天看到同事用delphi的代码,是sql语句的查询变量.看着一行行的复制修改变量名中的数字..我好奇,工作量好大.他说,PHP的话我也知道变量名可以改的.我不忍心轻声说了一句,PHP循环输出这段代码吧.他说快完成了.我不忍心,用数组的应该好点,不知delphi有没有数组的说法.他说,这个要试试..然后就用大括号把这段代码括起来,好像用数组去弄了..
Hashell
2013-07-02 19:45:02 +08:00
@ivenvd 补充一句..也是switch...case...case..

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/72617

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX