同事的代码写得不太规范,读起来有点痛苦,如何委婉地向同事表达自己的想法?

2015-08-28 11:02:23 +08:00
 batman2010
入职新公司快一周了,一直在读同事的代码,但是同事的代码写得不规范,驼峰命名、行内空格这些都不常见到。想和同事说说,又不想伤了和气,毕竟大家刚认识,相处也还不错。 LZ 不太会说话,所以请教下各位 v 友,对于 LZ 这种情况,如何表达自己的想法为好?
5084 次点击
所在节点    问与答
35 条回复
konakona
2015-08-28 11:04:24 +08:00
最好不要说话.
banri
2015-08-28 11:08:24 +08:00
说呗,互联网从业者大多都比较好说话,大家都是追求技术的,藏着掖着多不好
我刚来就觉得哪哪哪不太好,开会直接说,现在大家照样打成一片,有的也在慢慢改进

怕啥
harry890829
2015-08-28 11:11:22 +08:00
新招了个同事,命名一点规范都没有,教都教不会,头一个月被我训了好几次
sinxccc
2015-08-28 11:18:16 +08:00
你们项目组有编码规范么?没有的话跟你们主管提…
10iii
2015-08-28 11:23:27 +08:00
LZ. 你想和同事说是要达到什么效果呢?

你是希望让他去重写现有代码么?重写需要多少时间呢?他会乐意么?你为他多付出的时间埋单么?

或者你是希望作为善意的建议,提醒他从现在开始改变编程风格,多注意编程规范么?那你要考虑你有没有资格给同事建议,你们关系很近么?你资历比他高么?你负责审核他的工作么?
batman2010
2015-08-28 11:30:30 +08:00
@harry890829 我是新入职的,不能训人家啊,况且相处得还不错。
god
2015-08-28 11:37:51 +08:00
在注释里面卖萌并严厉的斥责你同事的行为的同时辅以纠正
batman2010
2015-08-28 11:46:09 +08:00
@10iii 我的目的是想让同事的代码可读性更好一些,这样,将来和同事一起维护这份代码会容易很多。我不能为他付出的时间买单,更没有资格建议人家,但对于我来说,维护这份代码的成本实在有点高。
ljbha007
2015-08-28 11:47:48 +08:00
直说 但是不要用高人一等的语气说
ljbha007
2015-08-28 11:49:19 +08:00
比如:“这个命名有点难读懂 我帮你改一下行不?以后合作会容易很多”
fwrq41251
2015-08-28 12:02:46 +08:00
跟 tech leader 提代码规范的问题,别指明哪个同事不规范,也别直接跟那个同事说
batman2010
2015-08-28 12:05:21 +08:00
@ljbha007 如果改起来的话,就要改好多了。不仅仅是命名,代码结构也有很多不合理之处。
mhycy
2015-08-28 12:16:43 +08:00
这事本应该向领导提出,就怕领导不把这事当成问题不去解决。。。
alore
2015-08-28 12:17:59 +08:00
比如:“这个命名有点难读懂 你帮我看看?”多问几次,懂得自然就懂,不懂的你直说人家也未必懂.
blacklee
2015-08-28 12:19:12 +08:00
不用说
因为他读你的代码页是一样的感受
batman2010
2015-08-28 12:32:37 +08:00
@blacklee 大家都不说的结果就是大家都痛苦着。
Mrun
2015-08-28 12:34:37 +08:00
放宽心,程序员读他人的代码都是一种折磨
blacklee
2015-08-28 12:35:05 +08:00
@batman2010 这个问题随着一个人写的代码越来越多水平越来越好才能解决,别人说的话,双方都会觉得是双方的习惯不同导致的
bk201
2015-08-28 12:41:11 +08:00
等你和他混熟了再说吧,不混熟你说的话人家根本不会当你回事。或者你请人家吃顿饭,然后谈谈你想法,说明是为了你们 2 个将来维护起来更轻松。
zerh925
2015-08-28 12:45:01 +08:00
这些话好好说就行了呗,为什么 it 从业者总要觉得自己真的交流困难一样?

这次采取什么方式,看别人怎么反应,你才能知道哪种处理方式好,哪种不好,锻炼出来了才是自己做人的方式。

吸取别人意见是可以的,但是最后做决定的还是你自己。
做自己,而不是做大多数。

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

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

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

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

© 2021 V2EX