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

从这个Sr的角度讲，留一堆comment convince别人改代码，既能在组里刷存在感，又能体现自己的技术能力。说白了他的行为完全是为了自己的利益，完全不考虑别人的ddl和时间。Craftmanship这种事情根本没有上限，我要是诚心想挑毛病的话，可以给任何人代码里留一堆comment。

短期最直接的办法，1.单独create ticket track optimize的工作。2，跟老板反映问题，需要在代码质量和时间上取舍。 如果老板支持Sr的话，可以考虑尽快转组止损了。明确告诉你这组文化有毒，不用听一些人胡说八道对你有好处。就和谈恋爱一样，遇到性格或者心理有问题的人，趁早分手止损才是最优解。

donnice 发表于 2021-02-20 14:08:41
就像你在工作的时候有人喊你去健身，你不去人家说有人主动催你去健身不是人生一大幸事吗？怎么还抱怨呢？
.1point3acres
分清主次是关键。“主”指的是赶得上ddl

我觉得那就是你ddl定的不对，在牺牲quality的前提下定的。

asdfjhfEREW 发表于 2021-02-20 11:05:12
不可能改变得了老板的，老板只会觉得你做的很慢，一直错过ddl，组里没有deliver，老板都没有办法跟自己的老板交代，难道老板跟自己老板说因为在改legacy code所以错过ddl？

那你就跟老板还有senior都说一下，我们希望啥啥时候完成，看看有哪一部分的quality improvement可以break down可以defer，哪些不可以，再调整ddl。如果那时候老板还是只管ddl不管quality，那这样的老板，你不需要跟着。如果商量好啦某些quality可以defer，那那个senior也不会再拦着

谢谢LZ 的分享，小弟学习了！

我觉得可以试图和那个senior 沟通。 下次改进.好的senior 会理解和妥协的...
如果是打死不肯罢休的那种，那要看看那个senior平时对自己是不是也这么严格要求自己...如果不是....换个组吧，或者能避免和那个人合作就避免

跑路跑路

匿名者 发表于 2021-2-20 14:34
谢谢这么detailed的回复！
因为senior指出很多别人留下的小问题，我本来也觉得都是小问题可以顺手改一下 ...

所以很多时候，别人留下的小问题，真的不要顺手改，改了 code review 就看到 diff 了，然后这个就是你的任务了。然后你不把这些搞清楚就是没有 ownership...  所以 legacy code 能不动就不动，最好自己开一个 class 或者 method 在里面写自己的逻辑，然后 legacy code 去调用这个 class 或者 method, 这样 unit test 和 git diff 啥的都非常 clean 不用趟之前的浑水,  CR 里面看到你在旧代码上改动的越小越好，对你越安全. 如果 Senior 对这些代码不熟悉，那么你改动只要看不到原来的烂代码的 diff， 就不关你的事，如果 Senior 对原来的 代码很熟，让你改，你就说我要 focus on my current task, but thanks for pointing those out, we should create a new ticket and track it in a separate CR so it's more clean 这样也不算 push back 对面也没啥话说.
(求米求米)

跟我遇到的一模一样。但我知道那个同事不是针对我 而是所有人的pr review在他手里都会很难。我之前一个非常急的项目因为他的很多要求最后slipped。之后我给老板说 希望他在code review时候考虑ddl和priority 而我开pr也会先写一个refractor 在refactor pr里把自己看不爽的 他可能nit的部分先解决完。而他现在也会说 如果我时间紧 他可以帮我refractor 或者follow up里解决。但这的确是个很无奈的事 时常是我的pr已经被approve了 结果被他看到 他又提出很多要求 最后要重写。和他argue他有很多legit的理由 很难push back很难convince。我自己觉得这样的同事不是很好合作

LittleFishBall 发表于 2021-2-21 07:15
所以很多时候，别人留下的小问题，真的不要顺手改，改了 code review 就看到 diff 了，然后这个就是你的 ...

感谢 senior几年前就参与写了这个package 所以他其实对这个package很熟 很容易指出各种问题 要我去fix 下次会push back回去

521197609 发表于 2021-2-21 08:04
跟我遇到的一模一样。但我知道那个同事不是针对我 而是所有人的pr review在他手里都会很难。我之前一个非常 ...

是的！和这种人合作过才知道多难受。。code quality真的是个没有上限的标准。。