aspose file tools*
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
Author

Please review my criteriaFind method

Sergei Simonov
Greenhorn

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.


Mag
 
I agree. Here's the link: http://aspose.com/file-tools
 
subject: Please review my criteriaFind method