代码审查 例四
可以看出，你非常关心这个类是否被放在了合适的命名空间里。非常好，使用命名空间是个好事。但在这个文件里，你嵌套了6层。看起来你试图在一个小小的地方里装下太多的东西。你要么应该想想不要分那么多层(是的，我可以看到这里有两个辅助类，应该放到它们自己的空间里，但如果把它放到其它地方会有不好吗？)，要么拆分一些代码，放到自己的根空间下。 代码审查 例五
非常详细的注释，佩服！这段代码需要好几个章节的文字来辅助理解，佩服！ 代码审查 例六
仔细看一下这第二个方法。如果一个方法需要8个参数才能让它知道干什么事情、如何干事情，那么，这个 方法有点太累了。请重构它，从它的肩膀上消减一些负担，拿走一些压力。把它一分为二(或更多)，或者，也许将一些参数当成类的初始化参数，可能会更有意义些。这8个参数你会同时都用到吗？所以，不要期望你的方法这样。 所以，这就是当你忘了拿近视眼镜、或直视太阳太久时做代码审查的方法。如果你有丰富的编程经验，我相信你也能举出很多有趣的例子。有人可能会反对，说这些只是一些小事情，还有很多更重要的事情需求考虑。然而，清爽、简洁、良好格式的代码是你需要的，如果你不去用心控制好你的代码结构，它终究会给你带来烦恼。 转自 http://www.vaikan.com/code-review-without-your-eyes/ |
