想吐槽一下经理的code review

001596 发表于 2021-04-27 20:07:58
别的不说，int vs size_t那个，你认错吧。如果用int，越界了你怎么办？production code直接crash吗？

非常同意，最近某个组有个sev1, int越界了就是其中一大原因。。

size_t 有个问题是如果参与了运算产生负值，会产生很难debug的错误。
所以在有可能int overflow的情况下我一般用long long.当然了我承认这个不是很标准的做法。

只要经理看CR的组，我第一时间走人。为啥经理能有闲功夫来看CR？只能说明他能力不足干不了经理的活，或者你们组压根就不需要经理。无论哪点，都是sinking ship

匿名用户 发表于 2021-04-30 16:32:20
只要经理看CR的组，我第一时间走人。为啥经理能有闲功夫来看CR？只能说明他能力不足干不了经理的活，或者你们组压根就不需要经理。无论哪点，都是sinking ship

我们组加我就4个人，经理是从senior sde转过去的 干了一年半，我的code都是她review，正常吗？

油冬冬 发表于 2021-05-11 04:30:44
我们组加我就4个人，经理是从senior sde转过去的 干了一年半，我的code都是她review，正常吗？

而且经理也oncall 我们组还有一个principal也oncall