----------- 众仙请进：MVC3 Code Review能说点啥子？-----------

梦呓人

有点搞笑哈，我自己都不是很清楚MVC3到底有什么特点，现在要我给人做code review.... 还请这方面做的比较多的大哥大姐们指点一下


谢谢！(MVC3 + Razor + Windsor)

梦呓人

看来都回家做饭去了，那我也回去了，明天再来哈

梦呓人

老师们早上好！

有知道的吗？

huazhb

我觉得code review主要是关注软件代码本身， 和什么技术语言关系不大。 根据SOLID原则， 主要看看一个class是不是职责太多， 一些可能变化的地方是不是可以抽象出来变成interface， 是否有足够的Unit Test， Unit Test的完备性等等。

无视

开始给人做code review了阿，不错不错！！

如果你愿意全面的作，你可以参考这个帖子的57楼.

http://www.oursteps.com.au/bbs/f ... p;extra=&page=2

但我个人不建议你这么做，除非你们team将这个check list制度化，贸然这样做很容易被人误认为故意刁难。

梦呓人

无视 发表于 2012-12-4 16:47
开始给人做code review了阿，不错不错！！

如果你愿意全面的作，你可以参考这个帖子的57楼.

大哥,谢谢啊,你终于来了!(握着温家宝温暖的手)


如果是关于MVC+Razor的呢？应该如何从结构上去注意？

梦呓人

今天分分没了， 楼上2位高人等我明天回血后再+++++++++

无视

梦呓人 发表于 2012-12-4 17:28
大哥,谢谢啊,你终于来了!(握着温家宝温暖的手)

具体到这个层面我觉得首先取决于你们的coding standard，不符合的都要做修正。

不过有些通常针对controller/action的原则.

1. view中尽量不放logic，所有logic都应该放在controller中，让其unit testable.
2. 架构应该理论上让unit test容易，而不是arrange起来特别难，这样可能会把logic分散到一些supporting class里面。
3. 结构复杂的domain model不适合作view model,从domain model到view model一般要做特别的mapping

剩下的就去看我给你的那个list.

梦呓人

无视 发表于 2012-12-4 17:44
具体到这个层面我觉得首先取决于你们的coding standard，不符合的都要做修正。

不过有些通常针对control ...

大哥，你说得太对了。

1， 一开始做MVC的时候View里面有大量的@{}代码。应该把它们尽量都放到model binding里面。据说这个和razor engine执行效率有关；


3，View model应该和Data model分开。我们用是NuGet里的那个AutoMapper，转换起来很方便。


你说的这2点都是上次review时候提出过的。大哥实在是太牛了，就是CTO的实力啊!

uowzd01

梦呓人 发表于 2012-12-5 08:57
大哥，你说得太对了。

1， 一开始做MVC的时候View里面有大量的@{}代码。应该把它们尽量都放到model bind ...

图片越来越精彩

xkj

学习了，

梦呓人

无视 发表于 2012-12-4 17:44
具体到这个层面我觉得首先取决于你们的coding standard，不符合的都要做修正。

不过有些通常针对control ...

大哥，针对你的第一点，我还有一个问题。


假设一个页面要传递给下一个页面一些参数，我可以通过viewDataDictionary或者querystring，这样在下一个页面的view里就有代码去读取那些参数了。


还有就是把那些参数写进model里传过去，但实际上这些参数和view model又不是特别有关系从OO角度上去。


如果是你，会怎么做？