如何劝说(Ctrl)CV党对自己的代码做精简?

icediv 发布于 2015年12月22日
tinyfool 等1人欣赏。

感觉可以把这次做项目的经历写成故事了,之前一直没有插手后台部分,今天抽空看了一眼后台哥们的代码,真的是抱着学习的心态看的,另外也有review的目的。

可是当我打开route文件的时候,密密麻麻将近2000行的代码立时让我眼前一亮。控制器里在没有注释(真心不敢催别人给自己的代码写注释,得罪不起)代码的情况下随随便便都是500到1000行。

手痒,真想给他的代码减肥,但是想想看这都是他这么多天来啪啪啪啪敲键盘的心血啊,如果真要是能用代码行数给工资,他简直要赚翻了啊,怪不得无论啥时候看他都是忙的不可开交,我还在感慨后台的事务逻辑可真多啊。标题写了CV党,但我感觉,这些劳动成果保不齐真不是复制粘贴的,可是这要怎么说呢?有这方面沟通经验的大哥给点提示啊,如果我真是经理或者班长组长就好说了,可偏偏管不着人家啊,如果直接和领导讲,岂不是成了打小报告。

共3条回复
tinyfool 回复于 2015年12月22日

关键在于是不是理解抽象的意义

icediv 回复于 2015年12月22日

1楼 @tinyfool 我们项目里用的laravel。从本质上说他就是没学着按laravel的方式编码,刚才想举个路由的那个例子,比如原本一句就可以的路由写法被他分拆成了4-7个,结果在他的项目下打了laravel自带的查看路由列表的命令(控制台下),返回了个没有管理员权限的html报错输出,不知道这大哥咋改的。关键要用什么方式和他沟通呢?总不能开门见山就说人家做的不对吧

tinyfool 回复于 2015年12月22日

2楼 @icediv 可以试试看,无所谓的,沟通嘛,心平气和的,如果是一个无可救药的,就不用再理了

登录 或者 注册
相关帖子