wood burning stoves
The moose likes Developer Certification (SCJD/OCMJD) and the fly likes Please review my criteriaFind method Big Moose Saloon
  Search | Java FAQ | Recent Topics | Flagged Topics | Hot Topics | Zero Replies
Register / Login

JavaRanch » Java Forums » Certification » Developer Certification (SCJD/OCMJD)
Bookmark "Please review my criteriaFind method" Watch "Please review my criteriaFind method" New topic

Please review my criteriaFind method

Sergei Simonov

Joined: Dec 20, 2002
Posts: 3
Hi there
Please review and comment my criteriaFind method implementation.
fieldMap - HashMap contains field name as the Key and order number of it as the value.

[ December 20, 2002: Message edited by: Junilu Lacar ]
Mag Hoehme
Ranch Hand

Joined: Apr 07, 2002
Posts: 194
Hi Sergei,
if it works, it is ok.
However, I find your method somewhat long. I stuffed all this parsing and matching into a separate class, thus encapsulating it.
As a rule of thumb: your methods should be so long that you can view them on the screen without scrolling. If your methods are longer than that, consider refactoring them.
Besides, I used two tokenizers, one for separating the key-value pairs, and the other one for separating the key from the value. Instead of putting all possible delimiters into one string you could use the nextToken(String delim) method.
Hope this helps.

I agree. Here's the link: http://aspose.com/file-tools
subject: Please review my criteriaFind method
jQuery in Action, 3rd edition