Printing Information in a node

[Tenn]

Solved.
The access that was being used to get to things was backasswards.
To many .nexts and not enough veriables.
I'll be sure to tell the people in the class about it.

Ok, We've got an assignment, where you have to print names in a particular month.

Yes we've done the assignment with arrays and I got that program working fine. Now we're using a linked list. and I have a basic code someone else wrote and have been using it as a jumping base and help to set up my own algorithem and such things.

Now, I have sucessfully gotten most of the program working. The problem is this...

public void printMonth(int Month)
{
UnorderedLinkedList<ExtPerson>.LinkedListIterator< ExtPerson> i = list.iterator();
//using month make a loop
while(i.hasNext())
{
if(i.next().getMonth()== Month)
{
System.out.println(i.next().getLastName()+ " " + i.next().getFirstName());
}
}

What happens is insted of printing the first and last name of the person who's birthday is in that month, its printing four different names from four different nodes

and I have tried changing .next to .info sense info is the name of the half of the node that has the actuall information in it but then it doesn't complie.

So I'm asking for any suggestions as to what to do insted. I realize part of what its doing is it's going to the information in the link beyond the one its supposed to be doing, I figured that out with the straight up print method. This one though has me stumped.

[OrangeDog]

Firstly this

UnorderedLinkedList<ExtPerson>.LinkedListIterator< ExtPerson> i = list.iterator();
while(i.hasNext()) {
    //do stuff with i.next()
}

is equivalent to the much simpler

for (ExtPerson ep : list) {
    //do stuff with ep
}

The problem you have is that every time you call i.next() it will advance the iterator by one. So the next() you print with is not the next() you tested for the date. Using the for-each loop above will solve that problem. To fix it while still using iterators:

while(i.hasNext()) {
    ExtPerson ep = i.next();
    if(ep.getMonth() == Month) {
        System.out.println(ep.getLastName()+ " " + ep.getFirstName());
    }
}

[Tenn]

Mmm Thanks, I'll try that and see if it creates the needed result.

Alright, I put the for each loop in there insted of the crazy blurb. And JGrasp gave me foreach not applicable to expression type --

Its a linked list, though I dont' see why that should change anything. save Perhaps make the classes that more unwilling to work... I've been fighting with this all day so Its really begining to grate on my last nerve. and at this point all begining to blur togeather.

[Tenn]

That worked. Thanks

[OrangeDog]

for-each loops should work with any array and anything that implements Iterable