📄 extremereviews.html
字号:
<head><title>Extreme Reviews</title></head><body><h1><img src="logo.gif"> Extreme Reviews</h1><a href="ExtremeProgramming.html">ExtremeProgramming</a> does not include formal reviews. <a href="PairProgramming.html">PairProgramming</a>, <a href="http://c2.com/cgi/wiki?CollectiveCodeOwnership">CollectiveCodeOwnership</a>, and impromptu informal reviews as part of team play are all part of XP.
<p><a href="UnitTests.html">UnitTests</a> are run all the time, reviewing the code with machine precision, and helping the developers go very rapidly and confidently.
<p><a href="FunctionalTests.html">FunctionalTests</a> review the entire system with machine precision, providing confidence to the customers.
<p><a href="http://c2.com/cgi/wiki?CollectiveCodeOwnership">CollectiveCodeOwnership</a> means that over time all eyes see most all the code.
<p><a href="http://c2.com/cgi/wiki?NoSecondChance">NoSecondChance</a> means that in troubled parts of the system, a new team takes over.
<p>These, and the many other XP practices, have been sufficient so far to provide the quality that has been needed. XP is based on simplicity and we never add additional process elements unless we need them.
<p>If quality defects of any kind were slipping through the process, an <a href="ExtremeProgrammingMaster.html">ExtremeProgrammingMaster</a> would then (and only then) devise something to catch them. She would follow the steps in <a href="ImprovingExtremeProgramming.html">ImprovingExtremeProgramming</a>.
<hr>
<a href="http://c2.com/cgi/wiki?BradAppleton">BradAppleton</a> points out that <a href="http://c2.com/cgi/wiki?InformalReviews">InformalReviews</a> could facilitate more people seeing more of the code. <a href="PairProgramming.html">PairProgramming</a> and <a href="http://c2.com/cgi/wiki?CollectiveCodeOwnership">CollectiveCodeOwnership</a> <em>are</em> a very active form of <a href="http://c2.com/cgi/wiki?InformalReviews">InformalReviews</a>. <a href="ExtremeProgramming.html">ExtremeProgramming</a> projects have observed no need to improve this area.
<p><hr>
<p>On the whole I'm a big fan of reviews. I push them with most clients I work with. From my observation C3 is full of reviews, through its practices of group design reviews with CRC cards, <a href="PairProgramming.html">PairProgramming</a>, and Refactoring. It does not seem to me that adding further reviews would add much.
<p><em>I haven't seen C3 in person, but the <a href="http://c2.com/cgi/wiki?DoIt">DoIt</a> team uses these practices aggressively. Back in the days when I worked on that team <sentimental sniff> we used to think of <a href="PairProgramming.html">PairProgramming</a> as 'real time code review' -- <a href="http://c2.com/cgi/wiki?BillBarnett">BillBarnett</a></em>
<p>However Kent did do reviews in the earlier stages. I'd be interesting in knowing what people (Ron, Kent, other C3ers) think of that now. Do they feel it was not worthwhile, or worthwhile then but not now?
<p>However this observation does not dim my liking of reviews. I usually work with non-XP clients. Without a strong use of <a href="PairProgramming.html">PairProgramming</a> and Refactoring I wouldn't be comfortable without some form of review process. To me it isn't a case of reviews vs XP, it's more a case of <em>how</em> XP does reviews.
<p>See also my thoughts on <a href="http://c2.com/cgi/wiki?RefactoringReviews">RefactoringReviews</a>
<p>--<a href="http://c2.com/cgi/wiki?MartinFowler">MartinFowler</a>
<p><hr>
<p>The place (or, rather, lack of one) for inspections in XP came up at a recent <a href="http://c2.com/cgi/wiki?ExtremeTuesdayClub">ExtremeTuesdayClub</a>, see <a href="http://www.xpdeveloper.com/cgi-bin/wiki.cgi?CodeInspection">http://www.xpdeveloper.com/cgi-bin/wiki.cgi?CodeInspection</a><hr><a href="http://c2.com/cgi/wiki?edit=ExtremeReviews">EditText</a> of this page (last edited July 10, 2000)<br><a href="http://c2.com/cgi/wiki?FindPage&value=ExtremeReviews">FindPage</a> by browsing or searching<p><font color=gray size=-1>This page mirrored in <a href="index.html">ExtremeProgrammingRoadmap</a> as of March 31, 2001</font></body>⌨️ 快捷键说明
复制代码
Ctrl + C
搜索代码
Ctrl + F
全屏模式
F11
切换主题
Ctrl + Shift + D
显示快捷键
?
增大字号
Ctrl + =
减小字号
Ctrl + -