查看: 10222|回复: 62
收起左侧

[带新人 Mentoring others] 同事代码实在太烂,我review,该怎么办呀?

  |只看干货
匿名用户-C76  发表于 2021-10-28 02:32:00 |阅读模式
本楼: 👍   28% (9)
 
 
71% (23)   👎

注册一亩三分地论坛,查看更多干货!

您需要 登录 才可以下载或查看附件。没有帐号?注册账号

x
一个博士女生加入我们组,她第一份工作,代码写的真是太烂了,一个method巨长,try catch 满天飞,catch 里面跑business logic,catch 完啥也不做,直接ignore,constructor 里各种不相关的东西,variable 命名毫无意义,啥都是session, api, log, 之类的
各种奇怪的写代码方法,比如把不相关的东西pass 到更低层的object里,太无语了,哎

这种情况下,该怎么办呀!! 他是上边老大直接雇的,我以前反应过问题给我们小老板,小老板没理我,看来小老板也不想得罪上面的人。以前有个不干活的烙印,小老板直接把他从我们组移走了,但是这个女生就一直放着留着。平常小老板也不理这个女生,有啥需要交流的都跟我说,让我去管她。

现在code review了,怎么办呀?我不想打击女博士自尊心,也不想得罪上面的人,更不想得罪我的小老板,又不想让她code review直接过,不然显得我太无能。这种事怎么处理好呀?求指教!!万分感谢!!





评分

参与人数 2大米 +2 收起 理由
MiNiMiMi + 1 赞一个
GTrepublic + 1 赞一个

查看全部评分


上一篇:该不该quit现在公司去google
下一篇:求助:被Facebook recruiter威胁了
pubu666 2021-10-28 02:54:53 来自APP | 显示全部楼层
本楼: 👍   100% (120)
 
 
0% (0)   👎
全局: 👍   96% (5049)
 
 
3% (159)    👎
我要是你的小老板我也不会理会你的抱怨。这些听起来都是很容易改善的代码风格问题,只是刚工作的人没有这些意识。你就认真code review,老板让你管她不就是让你干这个的。review时注意一下说话方式的话,又怎么会伤及她的自尊心呢。

评分

参与人数 2大米 +3 收起 理由
whdawn + 1
Lastheart + 2 给你点个赞!

查看全部评分

回复

使用道具 举报

qwerte 2021-10-28 03:08:31 | 显示全部楼层
本楼: 👍   100% (92)
 
 
0% (0)   👎
全局: 👍   96% (1553)
 
 
3% (52)    👎
这种你就直接在code review 时comment不就行了?你所列出的那些问题都是刚开始工作的新人容易犯的错误,而且都是很容易改正的。谁不是从新人开始做起呢?

评分

参与人数 1大米 +3 收起 理由
jacksterling + 3 给你点个赞!

查看全部评分

扫码关注一亩三分地求职与职场公众号
更多干货内容等你发现
回复

使用道具 举报

stonefish 2021-10-28 03:16:47 | 显示全部楼层
本楼: 👍   100% (55)
 
 
0% (0)   👎
全局: 👍   98% (2615)
 
 
1% (29)    👎
第一份工作的人哪有什么好的coding习惯,都是review的时候看comments学来的
学完以后再回馈社会,继续去review新人的code
回复

使用道具 举报

johnpang 2021-10-28 02:47:48 | 显示全部楼层
本楼: 👍   100% (14)
 
 
0% (0)   👎
全局: 👍   96% (1189)
 
 
3% (40)    👎
为啥不能打击,写的不好难道还要夸她吗? 不经历过打击代码就一直这么烂
回复

使用道具 举报

tonylhy2008 2021-10-28 03:12:54 | 显示全部楼层
本楼: 👍   100% (4)
 
 
0% (0)   👎
全局: 👍   90% (289)
 
 
9% (30)    👎
就一行一行的review,把你的suggestion都写在上面,然后把你老板也加进reviewer里面。
回复

使用道具 举报

本楼: 👍   100% (2)
 
 
0% (0)   👎
全局: 👍   90% (6566)
 
 
9% (699)    👎
让他把CR分成多个小份,一个一个的review,然后留comments
回复

使用道具 举报

kafkagre 2021-10-28 03:20:04 来自APP | 显示全部楼层
本楼: 👍   100% (14)
 
 
0% (0)   👎
全局: 👍   97% (138)
 
 
2% (3)    👎
我觉得楼主可以认真地review. 往好的方向说,楼主你帮新人onboard 算是leadership技能之一,之后可以写进去你的performance docs吧

评分

参与人数 1大米 +1 收起 理由
Mr.酒蒙子 + 1 赞一个

查看全部评分

回复

使用道具 举报

地里的匿名用户
匿名用户-985  发表于 2021-10-28 03:29:53
本楼: 👍   100% (4)
 
 
0% (0)   👎
这种经验很涨leadership experience啊
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 注册账号
职场达人
  • ↑ 本版用于讨论职场各种干货话题,闲聊请去🔗聊聊或者🔗匿名版
  • ❌ 本版严禁水贴,引战,发布广告,拉群,贴个人联系方式,扣分无警告
  • ☑ 求职、面经等去 🔗北美求职和 🔗回国求职大区,刷题和学习请去 🔗终身学习大区
  • ☑ 请去专版发布 🔗内推, 🔗招聘信息,和讨论 🔗创业内容

本版积分规则

>
快速回复 返回顶部 返回列表