楼主: lga2011_3
跳转到指定楼层
上一主题 下一主题
收起左侧

[同事协作] 组里的人代码质量差怎么委婉的说

   
全局:
菜元培kiwicai 发表于 2021-10-04 10:27:52
参加过这种讲代码的会,代码逻辑很复杂,新兵自己看多少遍还是会懵,单步调试调着调着就迷了,此时只能求助同事了
有道理,新人就直接接这种硬骨头嘛?见过很大坨儿又复杂又高级的legacy code,从来没有交给新人修改,都是随着项目有机会每次啃一部分这么学习的。当然也因为代码已经很smart,项目需要加需求的机会也不多。有能力的人知道新需求需要在哪加,看不懂的人根本不理解要加的新需求是什么意思😂  旧人凭实力理解,然后画high level chart讲解,但从来没有一行一行带人去读
回复

使用道具 举报

🔗
小蓝之歌 2021-10-5 02:40:54 | 只看该作者
全局:
噼里啪啦梆梆响 发表于 2021-10-4 01:56
典型的学生思想,公司雇你写代码是为了赚钱,不是提高代码质量,你要是沉迷于代码质量,可以去刷leetcode

看过clean code吗?代码质量高维护起来更容易……ops成本低
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-G4V9B  | 添加认证 | 2021-10-5 03:34:19 来自APP
你一个实习生还要judge别人?
回复

使用道具 举报

全局:
lulunini 发表于 2021-10-4 10:20
路过问一下。。。从来没见过谁专门安排讲解代码的会,真有人这样嘛?document 不足需要大家读代码理解逻 ...

我参加过这样的会,就是code design之类的会,或者可以和对方安排时间1对1讲解一下,或者refer其他人问。
主要看人看组里的culture,有的地方还比较互助,也有不喜欢share的。
confluence 的document多多少少会写一些,最主要还是靠自己琢磨
回复

使用道具 举报

🔗
lulunini 2021-10-5 04:04:25 | 只看该作者
全局:
每只猫都是天使 发表于 2021-10-4 15:45
我参加过这样的会,就是code design之类的会,或者可以和对方安排时间1对1讲解一下,或者refer其他人问。 ...

哦哦,code design 那种我就没算在里面。。比如前两天我就和人一起看了一些remodel的代码,他讲了一下改进的地方和思路,方便我把我的代码融进去
回复

使用道具 举报

🔗
brtt13 2021-10-5 04:35:59 来自APP | 只看该作者
全局:
噼里啪啦梆梆响 发表于 2021-10-04 01:56:44
典型的学生思想,公司雇你写代码是为了赚钱,不是提高代码质量,你要是沉迷于代码质量,可以去刷leetcode
不认同这个观点,我发而觉得这是一个很好的体现leadership的机会
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-ORSLY  | 添加认证 | 2021-10-5 07:35:18 来自APP
brtt13 发表于 2021-10-04 13:35:59
不认同这个观点,我发而觉得这是一个很好的体现leadership的机会
It never works in that way. 一个intern试图去纠正别人的代码,如果owner是个老油条大概率找个legacy reason顶回去,然后把这个intern的return offer搅黄

作为manager来说,code这个样子已经很久了,一个intern进组啥也没干就先开始指责别人,没必要保他不
回复

使用道具 举报

🔗
debuger 2021-10-5 08:20:50 | 只看该作者
全局:
和你的mentor/manager 沟通一下看看
回复

使用道具 举报

🔗
charles8star 2021-10-5 09:06:39 | 只看该作者
全局:
没事,你喜欢的话可以和你manager说,你去专门帮人改这些。感觉这是一个很大的impact的活
回复

使用道具 举报

🔗
plich 2021-10-5 09:13:50 | 只看该作者
全局:
委婉肯定不行。
他要是自己意识得到话你怎么委婉他都可以装听不懂. .и
他要是自己意识不到的话你委婉他就真听不懂
回复

使用道具 举报

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

本版积分规则

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