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

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

我虽是老员工,但是相对这个项目组,还很新。有几次旁巧側击的提过代码的问题,回答就是项目紧,或者refactor也就一两天的事,不着急。按照多年的经验,代码一旦完成,老板很难给时间去refactor的。
8054 次点击
所在节点    程序员
50 条回复
kalintw
2013-07-02 19:48:09 +08:00
设计和写设计文档的时候不包括类设计和接口设计?设计好了怎么写成大函数?
summic
2013-07-02 19:49:26 +08:00
看到 PHP 写得像Java的,变量夸张的占掉了一行
angelface
2013-07-02 19:49:31 +08:00
这个写法我见的多了, 一个方法2000来行, 都是小意思, 就是用c语言好手也不会这么写, 所以, 和C真的没毛关系 。。。
akira
2013-07-02 20:07:18 +08:00
一个函数的长度尽量控制在25行以内,20年前老师教的。
dorentus
2013-07-02 20:21:48 +08:00
@Abruzzi 不要被那个 switch 代码的作者的逻辑绕进去……

一般这种,语言相关的性能问题暂且不论,我一般是会把 { 1 : Constants.AGENT_1, 13 : Constants.AGENT_13, ... } 这些放到一个字典容器里(或者干脆写成其它格式配置文件,在运行时加载),然后顶多再处理一些 Constants.AGENT_DEFAULT 的情形,就可以了。
013231
2013-07-02 20:49:42 +08:00
@stranbird 你确定fork出curl再解析會比直接用requests代碼少?
ygtq
2013-07-02 21:38:03 +08:00
@akira 25行 有点夸张吧……
angelface
2013-07-02 21:47:39 +08:00
@ygtq 还好, 没看是20年前么, 也就是93年, 哈哈。
marginleft
2013-07-02 22:30:16 +08:00
@ivenvd 这是在凑字数?
jasontse
2013-07-02 23:04:43 +08:00
这是V2EX吗?我怎么觉得我走错地方了。。。

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

https://yangjunhui.monster/t/72617

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

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

© 2021 V2EX