1
konakona 2015-08-28 11:04:24 +08:00 2
最好不要说话.
|
2
banri 2015-08-28 11:08:24 +08:00 1
说呗,互联网从业者大多都比较好说话,大家都是追求技术的,藏着掖着多不好
我刚来就觉得哪哪哪不太好,开会直接说,现在大家照样打成一片,有的也在慢慢改进 怕啥 |
3
harry890829 2015-08-28 11:11:22 +08:00
新招了个同事,命名一点规范都没有,教都教不会,头一个月被我训了好几次
|
4
sinxccc 2015-08-28 11:18:16 +08:00 1
你们项目组有编码规范么?没有的话跟你们主管提…
|
5
10iii 2015-08-28 11:23:27 +08:00
LZ. 你想和同事说是要达到什么效果呢?
你是希望让他去重写现有代码么?重写需要多少时间呢?他会乐意么?你为他多付出的时间埋单么? 或者你是希望作为善意的建议,提醒他从现在开始改变编程风格,多注意编程规范么?那你要考虑你有没有资格给同事建议,你们关系很近么?你资历比他高么?你负责审核他的工作么? |
6
batman2010 OP @harry890829 我是新入职的,不能训人家啊,况且相处得还不错。
|
7
god 2015-08-28 11:37:51 +08:00 via iPad 1
在注释里面卖萌并严厉的斥责你同事的行为的同时辅以纠正
|
8
batman2010 OP @10iii 我的目的是想让同事的代码可读性更好一些,这样,将来和同事一起维护这份代码会容易很多。我不能为他付出的时间买单,更没有资格建议人家,但对于我来说,维护这份代码的成本实在有点高。
|
9
ljbha007 2015-08-28 11:47:48 +08:00 via Android
直说 但是不要用高人一等的语气说
|
10
ljbha007 2015-08-28 11:49:19 +08:00 via Android
比如:“这个命名有点难读懂 我帮你改一下行不?以后合作会容易很多”
|
11
fwrq41251 2015-08-28 12:02:46 +08:00 1
跟 tech leader 提代码规范的问题,别指明哪个同事不规范,也别直接跟那个同事说
|
12
batman2010 OP @ljbha007 如果改起来的话,就要改好多了。不仅仅是命名,代码结构也有很多不合理之处。
|
13
mhycy 2015-08-28 12:16:43 +08:00 1
这事本应该向领导提出,就怕领导不把这事当成问题不去解决。。。
|
14
alore 2015-08-28 12:17:59 +08:00 1
比如:“这个命名有点难读懂 你帮我看看?”多问几次,懂得自然就懂,不懂的你直说人家也未必懂.
|
15
blacklee 2015-08-28 12:19:12 +08:00
不用说
因为他读你的代码页是一样的感受 |
16
batman2010 OP @blacklee 大家都不说的结果就是大家都痛苦着。
|
17
Mrun 2015-08-28 12:34:37 +08:00
放宽心,程序员读他人的代码都是一种折磨
|
18
blacklee 2015-08-28 12:35:05 +08:00
@batman2010 这个问题随着一个人写的代码越来越多水平越来越好才能解决,别人说的话,双方都会觉得是双方的习惯不同导致的
|
19
bk201 2015-08-28 12:41:11 +08:00 1
等你和他混熟了再说吧,不混熟你说的话人家根本不会当你回事。或者你请人家吃顿饭,然后谈谈你想法,说明是为了你们 2 个将来维护起来更轻松。
|
20
zerh925 2015-08-28 12:45:01 +08:00 via iPhone 1
这些话好好说就行了呗,为什么 it 从业者总要觉得自己真的交流困难一样?
这次采取什么方式,看别人怎么反应,你才能知道哪种处理方式好,哪种不好,锻炼出来了才是自己做人的方式。 吸取别人意见是可以的,但是最后做决定的还是你自己。 做自己,而不是做大多数。 |
21
FrankFang128 2015-08-28 13:00:58 +08:00 1
1. 只重构你当前项目的代码,只重构交给你维护的代码
2. 没有「用一个星期来重构」这种事情,你必须「每时每刻都在重构」 3. 没有单元测试就不要乱重构。 4. 改代码风格不叫重构! |
22
kneep 2015-08-28 13:03:11 +08:00 via iPhone
应该要被 review 的人拒掉
|
23
muzuiget 2015-08-28 13:20:15 +08:00
稳定压倒一切啊懂不懂。
|
24
Nausicaa 2015-08-28 14:08:29 +08:00
除非很熟,否则别找不自在
|
25
shenqi 2015-08-28 15:42:17 +08:00
弄个 mr ,合上去就行。
|
26
ys0290 2015-08-28 15:50:18 +08:00 via iPhone
辞职
|
27
batman2010 OP @shenqi 请问 mr 是什么啊?
|
28
hpeng 2015-08-28 16:07:40 +08:00
别说,代码都是小事,我也是刚入职,我就不说我们开发都在主干上,今天这周内网构建整天出问题,不想说话了。
|
29
caserest 2015-08-28 16:09:11 +08:00
直接说啊 看到这个我突然想到我上一个公司有人用用的变量名字是$woaibeijingtiananmen
|
30
batman2010 OP @hpeng 是啊,明明知道问题所在,但说也不是,不说也不是,这种感觉很是累心。
|
31
batman2010 OP @caserest 我是新入职的,技术也不是特别强那种,说话没有分量啊,直接说容易让别人有想法。
|
32
osborn 2015-08-28 16:41:41 +08:00 1
评价别人代码,除非有碾压别人的实力,否则,要非常非常非常委婉。
|
33
frozen2013 2015-08-28 18:53:30 +08:00 1
建议你不要直接和当事人说。
如果你指得不规范仅仅是没有“驼峰命名、行内空格”,作为一个新人直接说出来,资历比你深同事十有八九会认为你小题大做出风头,不一定理会你。 如果你强迫症忍不住,要么向组长提代码规范的重要性,要么在团队会议上自告奋勇自己动手改。 做技术都很实在,但自尊心不一定低。 |
34
young 2015-08-28 19:12:33 +08:00
都很随意怎么办, 复制粘贴, 改个参数...
|
35
cxbig 2015-08-29 06:01:47 +08:00 1
这个属于 TL 领导力问题,团队推行一个代码标准,严格执行。
比方说我们团队的 PHP 代码使用 PSR-*规范 http://www.php-fig.org/ 代码提交汇总前会经过同行代码评审,不通过的打回改好再合并。 |