<?xml version="1.0" encoding="UTF-8"?><rss version="2.0"
	xmlns:content="http://purl.org/rss/1.0/modules/content/"
	xmlns:dc="http://purl.org/dc/elements/1.1/"
	xmlns:atom="http://www.w3.org/2005/Atom"
	xmlns:sy="http://purl.org/rss/1.0/modules/syndication/"
		>
<channel>
	<title>Comments on: Article Series: &#8220;Automated code reviews with Checkstyle&#8221; on JavaWorld</title>
	<atom:link href="http://blog.xebia.com/2008/11/26/article-series-automated-code-reviews-with-checkstyle-on-javaworld/feed/" rel="self" type="application/rss+xml" />
	<link>http://blog.xebia.com/2008/11/26/article-series-automated-code-reviews-with-checkstyle-on-javaworld/</link>
	<description></description>
	<lastBuildDate>Mon, 26 Jul 2010 20:40:47 +0200</lastBuildDate>
	<generator>http://wordpress.org/?v=2.8.4</generator>
	<sy:updatePeriod>hourly</sy:updatePeriod>
	<sy:updateFrequency>1</sy:updateFrequency>
		<item>
		<title>By: Shrikant Vashishtha</title>
		<link>http://blog.xebia.com/2008/11/26/article-series-automated-code-reviews-with-checkstyle-on-javaworld/comment-page-1/#comment-64588</link>
		<dc:creator>Shrikant Vashishtha</dc:creator>
		<pubDate>Wed, 26 Nov 2008 16:40:05 +0000</pubDate>
		<guid isPermaLink="false">http://blog.xebia.com/?p=821#comment-64588</guid>
		<description>Thanks Vikas for the appreciation.

@Peter. If you read the articles in detail, I focus on using Checkstyle custom checks to avoid severe design issues. I also hate using such tools for trivial stuff (brackets, whitespace stuff). But the same tool can be used for writing some cool checks. For instance, in a project where it is completely a NO-NO to write any singleton if they are using Spring, a custom Checkstyle check does the trick. You will not be able to compile that code. Similarly the example I mentioned in the article, you can avoid accessing a method on potentially NULL object. It&#039;s also useful for code-analysis reasons for audit/performance tuning assignments. At the end of it, it&#039;s always upto the project team to abuse or use a tool. However in my view Checkstyle custom checks and its proactive implementation is something a lot of projects (especially big or Waterfall model based or with a lot of inexperienced deevelopers) look for.</description>
		<content:encoded><![CDATA[<p>Thanks Vikas for the appreciation.</p>
<p>@Peter. If you read the articles in detail, I focus on using Checkstyle custom checks to avoid severe design issues. I also hate using such tools for trivial stuff (brackets, whitespace stuff). But the same tool can be used for writing some cool checks. For instance, in a project where it is completely a NO-NO to write any singleton if they are using Spring, a custom Checkstyle check does the trick. You will not be able to compile that code. Similarly the example I mentioned in the article, you can avoid accessing a method on potentially NULL object. It&#8217;s also useful for code-analysis reasons for audit/performance tuning assignments. At the end of it, it&#8217;s always upto the project team to abuse or use a tool. However in my view Checkstyle custom checks and its proactive implementation is something a lot of projects (especially big or Waterfall model based or with a lot of inexperienced deevelopers) look for.</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: Vikas Hazrati</title>
		<link>http://blog.xebia.com/2008/11/26/article-series-automated-code-reviews-with-checkstyle-on-javaworld/comment-page-1/#comment-64464</link>
		<dc:creator>Vikas Hazrati</dc:creator>
		<pubDate>Wed, 26 Nov 2008 08:56:27 +0000</pubDate>
		<guid isPermaLink="false">http://blog.xebia.com/?p=821#comment-64464</guid>
		<description>You always have an option of configuring the checkstyle ruleset to take away what you consider as noise, this includes whitespaces and other formatting checks. In my view some of the things like RFC, DIT, NOC, Coupling are things which you should be able to check with a tool to see if your design is good. Of course you would not be able to take out the human factor completely but automation helps, metrics as a tool would help in this case or CAP for that matter. Also various other SCA metrics like size, CC, Npath, boolean expression complexity etc would help you keep your code clean, understandable, testable, reliable etc etc</description>
		<content:encoded><![CDATA[<p>You always have an option of configuring the checkstyle ruleset to take away what you consider as noise, this includes whitespaces and other formatting checks. In my view some of the things like RFC, DIT, NOC, Coupling are things which you should be able to check with a tool to see if your design is good. Of course you would not be able to take out the human factor completely but automation helps, metrics as a tool would help in this case or CAP for that matter. Also various other SCA metrics like size, CC, Npath, boolean expression complexity etc would help you keep your code clean, understandable, testable, reliable etc etc</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: Peter Veentjer</title>
		<link>http://blog.xebia.com/2008/11/26/article-series-automated-code-reviews-with-checkstyle-on-javaworld/comment-page-1/#comment-64459</link>
		<dc:creator>Peter Veentjer</dc:creator>
		<pubDate>Wed, 26 Nov 2008 08:04:17 +0000</pubDate>
		<guid isPermaLink="false">http://blog.xebia.com/?p=821#comment-64459</guid>
		<description>Personally I think that checkstyle introduces a lot of noise. There are a few useful metrics like method complexity, but there are other tools for that as well. 

And I have seen too many projects that focus on the very low level style issues (brackets, whitespace, etc), but where there are big differences in design. The problem is that static analysis tools to detect these problems, are much harder to find. So the human factor is still invaluable.</description>
		<content:encoded><![CDATA[<p>Personally I think that checkstyle introduces a lot of noise. There are a few useful metrics like method complexity, but there are other tools for that as well. </p>
<p>And I have seen too many projects that focus on the very low level style issues (brackets, whitespace, etc), but where there are big differences in design. The problem is that static analysis tools to detect these problems, are much harder to find. So the human factor is still invaluable.</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: Vikas Hazrati</title>
		<link>http://blog.xebia.com/2008/11/26/article-series-automated-code-reviews-with-checkstyle-on-javaworld/comment-page-1/#comment-64400</link>
		<dc:creator>Vikas Hazrati</dc:creator>
		<pubDate>Wed, 26 Nov 2008 03:12:08 +0000</pubDate>
		<guid isPermaLink="false">http://blog.xebia.com/?p=821#comment-64400</guid>
		<description>Excellent articles. The concept of proactive code reviews before the code hits the repository is very good to know.</description>
		<content:encoded><![CDATA[<p>Excellent articles. The concept of proactive code reviews before the code hits the repository is very good to know.</p>
]]></content:encoded>
	</item>
</channel>
</rss>
