m****7
发帖数: 69 | 1 还是公司的老人了，C#，代码写得巨烂无比，每次review他的代码都跟吃屎一样恶心。
举几个例子吧：
1. 你见过一个if statement的条件部分占三十多行吗？大概五十个表达式用||和&&连
接在一起的
2. 你见过把所有用过的变量都做成成员变量吗？一百多个成员变量，看着都壮观，还
都没有统一的地方初始化。你看他用某个变量的时候都替他担心，不读他所有的代码，
谁知道这个变量当前有没有合法的值啊
3. 最极品的是，他的所有函数都没有任何reuse的价值，因为都是跟整个逻辑强烈相关
的。后来我发现他是先把所有的逻辑实现在一个函数里，然后按照每二三十行一个函数
改过来，所有用到的变量不是成员变量就是函数参数，还都是by ref的
最后说一说mit这个验证码真是垃圾啊，敲了半天一点发表，验证码不对，再回去，敲
得东西全没了。这是俺第三次输这篇文章了。别说我闲的慌，我实在是宁愿多敲一次也
不愿意review他的代码啊 |
i***0
发帖数: 8469 | |
w********d
发帖数: 1131 | 3 听说过微软的senior dev 有这样的。
【在 m****7 的大作中提到】
: 还是公司的老人了，C#，代码写得巨烂无比，每次review他的代码都跟吃屎一样恶心。
: 举几个例子吧：
: 1. 你见过一个if statement的条件部分占三十多行吗？大概五十个表达式用||和&&连
: 接在一起的
: 2. 你见过把所有用过的变量都做成成员变量吗？一百多个成员变量，看着都壮观，还
: 都没有统一的地方初始化。你看他用某个变量的时候都替他担心，不读他所有的代码，
: 谁知道这个变量当前有没有合法的值啊
: 3. 最极品的是，他的所有函数都没有任何reuse的价值，因为都是跟整个逻辑强烈相关
: 的。后来我发现他是先把所有的逻辑实现在一个函数里，然后按照每二三十行一个函数
: 改过来，所有用到的变量不是成员变量就是函数参数，还都是by ref的
|
m****7
发帖数: 69 | 4 其实这个人senior的地方就在于能够完成需要的功能，但是因为他的代码所带来的后续
维护工作要比重新写一遍代码的cost还要高的多。他的feature想要加点儿新东西能把
人累死。 |
t******t
发帖数: 607 | |
h**k
发帖数: 3368 | 6 这人是director还是VP的哥们？
你要不就不给他过，除非他把结构改过来；要不就什么意见都不给，直接过。你review
他的程序，主动权在你手里。
【在 m****7 的大作中提到】
: 其实这个人senior的地方就在于能够完成需要的功能，但是因为他的代码所带来的后续
: 维护工作要比重新写一遍代码的cost还要高的多。他的feature想要加点儿新东西能把
: 人累死。
|
z********o
发帖数: 4284 | |
