[seek-dev] Query Builder Code Review

[Rod Spears]

I have put together an overview document <http://129.237.201.166/QB/> to 
help with the code review.

At Ricardo's suggestion, I used the "Maven" tool to generate a source 
code listing, a cross-reference, metrics and JavaDoc 
<http://129.237.201.166/QB/index.html#Query_Builder_DesignArchitecture>.

The links in that section should be enough to do the code review. When 
looking at the metrics make sure you familiarize yourself with the 
various terms, some mean the opposite that you might think.

Also, there seems to be some varying feelings and opinions about the 
coding style for Kepler. My approach which I am sure differs from Kepler 
is as follows:

1) All method arguments start with a lowercase "a"
2) All class data members all start with "m" (unless they are final)
3) All local variables have no prefix.

It is my belief that you should always be able to look at a variable in 
the code and understand it's scope. I disagree with the approach where 
both the method arguments and the local variables have no prefix and are 
often distinguished by a "this.", which is rather verbose at times.


Rod

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mercury.nceas.ucsb.edu/ecoinformatics/pipermail/seek-dev/attachments/20040812/78655f31/attachment.htm