Cryptic S6
2022-07-17T09:20:23+00:00
背景:cs毕业7年,在国内主要负责一个中厂的中间件部分。卷不动了,就出来找找机会,最近入职了一家欧洲的公司。
第一个月被code review搞的很挫败。原公司写java
,现公司也是写java。code review被老大提了很多点,包括但不限于:
1.方法命名一定要见名知意。比如testThrowExceptionForGenrratingXXXWithUserNameAndPassword。这只是一个非常短的了,还有更长的。
2.非常依赖lombok和java 8的stream,可读性很强。我说这个不好调试啊。我们老大回复说,调试属于暴力法。你的代码不能运行/运行错误的时候才需要调试。
3.不可变的变量要声明为final(包括临时变量)
4.optional不可以作为方法参数。
和原公司的一些区别(非代码部分)
1.没有统一的测试环境,每人一个namesapce,用kubernetes+aws sandbox管理每个人的测试环境
2.对代码测试覆盖率有严格的要求
3.基本上每周都要升级依赖库(安全检查)
我这才写了不到一百行代码………给我找出来很多问题。。。。。
第一个月被code review搞的很挫败。原公司写java
,现公司也是写java。code review被老大提了很多点,包括但不限于:
1.方法命名一定要见名知意。比如testThrowExceptionForGenrratingXXXWithUserNameAndPassword。这只是一个非常短的了,还有更长的。
2.非常依赖lombok和java 8的stream,可读性很强。我说这个不好调试啊。我们老大回复说,调试属于暴力法。你的代码不能运行/运行错误的时候才需要调试。
3.不可变的变量要声明为final(包括临时变量)
4.optional不可以作为方法参数。
和原公司的一些区别(非代码部分)
1.没有统一的测试环境,每人一个namesapce,用kubernetes+aws sandbox管理每个人的测试环境
2.对代码测试覆盖率有严格的要求
3.基本上每周都要升级依赖库(安全检查)
我这才写了不到一百行代码………给我找出来很多问题。。。。。