程序员,遇到这样的同事怎么办

Meowcolm-avatar

Meowcolm

2021-08-18T14:43:49+00:00

同事原本是外包,现在按正式员工被招进来 我一直当做跟我平级来处

但我经验和能力比他好 平时领导对项目的交代和需求交付也都是直接找我

但目前痛苦就痛苦在这孩子水平理论有,但设计思想上很烂
举几个例子java
1.好好的注释不用,嫌每个注释占三行,自己写个注解当注释用 我看的一愣一愣的,也不知道干啥用的,后来才知道用来注解的
2.用大量的静态方法,明明是类内处理的函数,非得整个静态类,静态方法调用 我跟他说要有面向对象思想,静态方法占内存,他不以为意,这点内存无所谓
3.写的代码完全看不懂,一坨一坨的,一个列表能遍历五六次 还说每次处理不同的东西
4沟通能力非常差,经常是我在询问一个问题,他第二句一个但是 就能给你带偏讲到其他事情上 我必须非常专注的跟他沟通,一走偏好及时把他拉回来

最关键是写代码还很积极 有时候我也想代码能跑就行了
有时候还觉着看到这样代码很难受
处理问题也很积极,遇到问题就编码补漏洞,不想着在设计上兼容新需求,而且直接硬改

哦对了还不改,说跟我开发风格不一样
我说不是这不是开发风格,这是开发规范
他说需求还做不完,就不要拘泥于这些细节了
我说需求排期都是我们自己定的,又没有怎么加班,你现在代码写不好,后面很难维护
他说这个后面再说吧
。。。。

唉我周五跟领导要求一起代码走查一下我们的项目

其实技术方案领导跟我们一起定过了,但最终实现出来的代码又跟设计相去很远
N8 da Bear-avatar

N8 da Bear

难道没有代码review?
Meowcolm-avatar

Meowcolm

[quote][pid=543464085,28170041,1]Reply[/pid] Post by [uid=14489400]Justwen[/uid] (2021-08-21 22:47):

难道没有代码review?[/quote]已经喊领导周一review了 这边团队人起来了

领导带着几十个人 五六个团队 现在代码走查越来越少了
KorbaK-avatar

KorbaK

Code review的时候给他疯狂提comments,让他休,不然不给deliver
EMANATE-avatar

EMANATE

其实国内绝大多数公司对于程序的要求就是实现功能,至于代码质量和日后可能造成的维护成本,往往都是说起来很重要,真的要推动去做就各种客观原因了。
Meowcolm-avatar

Meowcolm

[quote][pid=543464963,28170041,1]Reply[/pid] Post by [uid=60762722]Samuel514[/uid] (2021-08-21 22:51):

Code review的时候给他疯狂提comments,让他休,不然不给deliver[/quote]目前review就是拉会议室过一遍代码

没啥流程 也没这块的考核
Meowcolm-avatar

Meowcolm

[quote][pid=543465316,28170041,1]Reply[/pid] Post by [uid=17336024]神恩之名[/uid] (2021-08-21 22:53):

其实国内绝大多数公司对于程序的要求就是实现功能,至于代码质量和日后可能造成的维护成本,往往都是说起来很重要,真的要推动去做就各种客观原因了。[/quote]唉也没要求多高,设计上多完善
现在就是以功能为主,但也别太过分了。。。
BigSmart-avatar

BigSmart

纠结个啥,吃点亏,正常人都会动脑子想想自己哪里有问题吧。至于思维发散,会跑偏,这个无解[s:ac:瞎]
KorbaK-avatar

KorbaK

[quote][pid=543465604,28170041,1]Reply[/pid] Post by [uid=39440010]yukungood1[/uid] (2021-08-21 22:54):

目前review就是拉会议室过一遍代码

没啥流程 也没这块的考核[/quote]那就有点麻烦了,想办法搞审核机制吧,同事间相互code review
A昊-avatar

A昊

Reply to [pid=543465316,28170041,1]Reply[/pid] Post by [uid=17336024]神恩之名[/uid] (2021-08-21 22:53)

美帝留学生表示美帝印度人多起来以后不是一摸一样吗?[s:ac:哭笑] 这个现象绝对不是国内特有的
window-avatar

window

屎山不就是这么出来的嘛[s:ac:哭笑]我自己写代码也这样,难改啊
Blimps-avatar

Blimps

引入sonar扫描,不合规不给上生产…当然,最好由安全部门提出统一规范要求…
Meowcolm-avatar

Meowcolm

[quote][pid=543466181,28170041,1]Reply[/pid] Post by [uid=60762722]Samuel514[/uid] (2021-08-21 22:57):

那就有点麻烦了,想办法搞审核机制吧,同事间相互code review[/quote]代码我看了跟他提他不听啊

跟我开发风格不一样
我说不是这不是开发风格,这是开发规范
他说需求还做不完,就不要拘泥于这些细节了
我说需求排期都是我们自己定的,又没有怎么加班,你现在代码写不好,后面很难维护
他说这个后面再说吧
。。。。

已经拉领导周一审了,到时候让领导去说他

不过跟领导最初定的技术方案 他都能实现出偏离很多的结果。。。
Hectic Ren-avatar

Hectic Ren

这就是系统逐渐变得辣鸡的原因[s:ac:哭笑]
不光小公司,大公司很多也这样
建议lz还是尽快跑路,去些工程体系比较好的公司吧[s:a2:中枪]
EMANATE-avatar

EMANATE

[quote][pid=543465889,28170041,1]Reply[/pid] Post by [uid=39440010]yukungood1[/uid] (2021-08-21 22:56):

唉也没要求多高,设计上多完善
现在就是以功能为主,但也别太过分了。。。[/quote]其实这个就没办法,基本上一个公司能够保证核心代码质量就已经很好了,功能模块尤其是可以外包的部分一般也就不做要求了。

如果影响到你们组内开发协作,最好还是让领导规范一下基本的编码规范,具体到如何实施,然后组内定期小规模review。
Carlos99-avatar

Carlos99

好办啊,封印住就好了,你写interface,他写impl,他随便他拉屎山[s:ac:哭笑]
Meowcolm-avatar

Meowcolm

[quote][pid=543467721,28170041,1]Reply[/pid] Post by [uid=483252]dreadice[/uid] (2021-08-21 23:05):

好办啊,封印住就好了,你写interface,他写impl,他随便他拉屎山[s:ac:哭笑][/quote]他要听我的 我加班都给他重构了也行
Pablo5759-avatar

Pablo5759

别把打工当事业做。得过且过得了。模块职能分清楚,让他自己拉的屎自己擦屁股。设计的太好没用。能把这种人招进来,说明是业务主导的公司(也是国内绝大多数公司),而不是(世界上也很少的)工程师文化的公司。在这种公司要站高一级看问题,抓住业务本质,才能尽快升职加薪当领导。 如果实在对技术有执念,建议跳槽。
TylerJenkss-avatar

TylerJenkss

既然你跟他是平级,那他代码写的好不好跟你有啥关系,管得宽[s:a2:doge][s:a2:doge][s:a2:doge]
!1cyy-avatar

!1cyy

sonar+jenkins

过不了不能持续集成