I do return the value in the second version I posted. It was an error in the
code. In my original code I had:

if (c1.size() > c2.size()) {
return 1;

if (c1.size() < c2.size() {
return -1;

But I thought that since the contract only requires the value to be a
positive or negative, then the simple subtraction would do.

So it should be:
int c = c1.size() - c2.size();
if (c != 0)
return c;

Just forgot the most important 2 lines of code. I appreciate you pointing
that out though. Sill mistake, but could have consequences if it was left

I have actually no heard of Test Driven Design before. I will have to read
about it a bit more, but my normal testing procedure seems to follow the
basics I read about it. My first goal is to make sure it runs, then see if I
can break it :-)

As always, thanks for the input!

James R. Perkins

On Wed, Sep 2, 2009 at 23:11, Thorbjoern Ravn Andersen <ravn@xxxxxxxxxx>wrote:

James Perkins skrev:

* Compares the first collection against the second collection.
* negative integer, zero or positive integer.
* <p>
* If the size of the first collection is greater than the size of
* second collection, a positive integer is return. If the size of
* collection is less than the size of the second collection a
* number is returned.
* </p>

int c = c1.size() - c2.size();

The javadoc does not conform to the code :) You do not return a value
if the size is unequal.

Have you considered using Test Driven Design to write such a library? I
usually do this because it works well for me focussing on a given
detail, while the tests catch if I mess something up.

Thorbjørn Ravn Andersen " Tubular Bells!"

This is the Java Programming on and around the iSeries / AS400 (JAVA400-L)
mailing list
To post a message email: JAVA400-L@xxxxxxxxxxxx
To subscribe, unsubscribe, or change list options,
or email: JAVA400-L-request@xxxxxxxxxxxx
Before posting, please take a moment to review the archives

As an Amazon Associate we earn from qualifying purchases.

This thread ...


Follow On AppleNews
Return to Archive home page | Return to MIDRANGE.COM home page

This mailing list archive is Copyright 1997-2022 by and David Gibbs as a compilation work. Use of the archive is restricted to research of a business or technical nature. Any other uses are prohibited. Full details are available on our policy page. If you have questions about this, please contact [javascript protected email address].

Operating expenses for this site are earned using the Amazon Associate program and Google Adsense.