请问各位软件开发大拿，面试中被问到关于code review的问题，怎么回答好？

Keith

比如问题：“如果让你review别人的code，你会看什么（怎么review）？”

在澳洲

看code效率、是否有备注、是否整洁易读、安全性

差不多够了

沧海一帆

代码是否实现需求，
是否简洁易懂，
是否遵循软件设计的基本原则
测试能否覆盖各种情况，
代码风格是否和团队一致

kingsfull123

ding!

aries_kael

可读性
有没有对应的单元测试
有没有实现BR
有没有注释
（代码风格不知道要不要提，因为有些人就是喜欢自己的风格）
Thread Safe

zn7726

学习了

Ponny

里面的逻辑也要看一下

wanming1986

1）method name 是否 meaningful，
2）拼写有没有错误===》》》我处女座强迫症，见谅
3）逻辑 是否准确，先了解BR，先把他的code 从git checkout 你local 然后看算法复杂度，有的要是O(n^2) 可否优化 成 O(N)
4）测试，unit test -> component test-> blackbox text 是否都有，测试方法是否遵守
     methodname_context_returnXXXX（） 格式
5）security defect
6)  test coverage，我们Jenkins 上配置了，如果jenkins pass了，说明没问题
7）检查acceptance criteria 都做到了没
8）异常处理是否得当
9）结构上，内容上都要注意，结构上看代码，内容上，把你对BR的理解和他沟通，确定你们在一个频道上
10）整齐划一，大家都用jdk8，你还在用以前的for loop 提出来，建议整改
11）。。。。

大神补充吧

suke06

sonarQube + Jenkins.