求问 如果senior要求很高 block住你的进度怎么办

过来人的经验，如果你觉得对方的建议是有道理的，多花点时间照着改了就行。多来几次，你写代码就会越快而且越好，你也就成长了。rush每个人都会，但能写高质量的代码的人就会少很多。如果每次都rush, 你很难进步。

公司不是特别小的话，没什么ddl是大不了的。赶ddl是处理ddl最低级的方法。好好磨练自己的技能和心智。这个比ddl重要很多。

nixingdeqie 发表于 2021-2-21 23:34
过来人的经验，如果你觉得对方的建议是有道理的，多花点时间照着改了就行。多来几次，你写代码就会越快而且 ...

谢谢建议 锻炼技能和磨练心智 我相信是每个sde都要持续去做的事情。如果不是mgr每天都开会催进度并且说怎么一个多月了还没开始load test这种话的话  我也觉得赶ddl很低级。

匿名者 发表于 2021-2-21 08:07
感谢 senior几年前就参与写了这个package 所以他其实对这个package很熟 很容易指出各种问题 要我去fix 下 ...

那就只有第二种方法，就是如果要让你 fix 别的，那你就说我们另开一个 ticket 然后我们另发一个 CR，不然两个都写在一起不是 best practice, 这样但愿是可以 block. 潜台词是: "我把两个分开了，一来显得我有 priority，而且另外一个我们啥时候弄是不是我弄，那个是属于另一个discussion"

先给个小tips：遇到cr上面让你修前人bug的或者做一些extension类的clean up的我建议你可以直接reply类似 added bug：b/xxxx

然后：关于怎么和reviewer interact，我自己的感觉是，大多数的review下来的问题都是稍微改一下就好的，但你如果遇到restructuring类型的comments我建议你可以和senior约个会花30分钟讨论一下，这样效率会快很多，communication也会更有效率。

最后，在1：1和manager聊天的时候不要害怕去给team的norm提建议。比如我们team就会align说：1.cr的scope只在change的部分，如果要refactor应该开新的bug。
2.cr的目的是提升你review的代码的代码质量而不是重写你review的代码。
3. cr的目的从来不应该是block，提出建议的时候最好能提供一个可行的fixing direction。
.. 4. 如果同一个cr comment 的discussion有超过3条的话，可以尝试约个时间或者写个discussion doc。

o彩蝶寻梦o 发表于 2021-02-20 22:08:24
那我觉得你这个时候就应该站在你们team的角度去分析，到底quality重要还是ddl重要啦，如果你都没分析过，只是抱怨人家senior给你提建议的话，我觉得这个心态走不远的。

同意。如果team每个任务定的ddl根本容不下quality那就是team有毒。如果楼主的team不是这样的话，何苦费尽心思各种试图证明sr的comment全是为了他自己。活儿永远干不完，ddl之后还是ddl，只为了ddl干活终究也就是赶在每一个ddl之前拉下一坨坨的shi而已，最后也拿不到sr的好评。

我的建议是不要一刀切，senior给你提comments本身也要花他的时间，既然他觉得这个时间是值得花的，你也应该信任他的comments大部分是有价值的。

具体到是否事无巨细都满足他的要求，我觉得完全可以就事论事，比如改个函数名让它更容易看懂，这需要花你很多时间吗？我觉得在ddl比较紧的情况下，你可以有选择的把一些容易address的问题先解决，困难的问题做backlog记录下来之后有时间再revisit。

如果你一直在赶ddl，而不能有效维护核心代码的工程质量，我觉得离堆出一个屎山也不远了，这种team就算business impact再大，工作体验也是很差的。

donnice 发表于 2021-02-20 00:07:30
曾去过十分类似的组，最后以转组了事。现复盘如下：
1. 回复将会在专门开一个code cleaning的ticket里一并解决，这个ticket ddl就在眼前，不希望延误工期
1.1 可以在we

其他不评论，单纯吐槽最后一段。Senior不靠comment的量，评 performance或升职，太小儿科了。亚麻的senior通常做high level design,协助写road map，主要依赖abstract thinking和strategic thinking。Review CR属于他们工作中非常微不足道的一件小事，很多Senior基本都不参与coding和CR。

匿名用户 发表于 2022-7-30 15:55
其他不评论，单纯吐槽最后一段。Senior不靠comment的量，评 performance或升职，太小儿科了。亚麻的senior ...

从楼主这段来看（如果是楼主的话）
https://www.1point3acres.com/bbs ... 83&pid=15018133
这还真就是亚麻的Senior了。那我就不明白你们谁是对的了