Why is this for loop not initiating?

[Scopic]

Hello, I'm new here and had a quick question. I have set up output checks and it looks like my program stops right at the for loop declaration and does not progress into the loop, instead it seems to skip over it.

Anyone have any clue why this would be? Thank you!

       System.out.println("Enter Subscriber's name: ");
                subSearch = keyboard.nextLine().toUpperCase();

                for(int i = 0; i < set.size(); i++)
                {
                    anElement = set.getCurrent();
                    classId = anElement.getClassName();
                    
                    if(classId.equals("Subscriber"))
                    {
                        aSub = (Subscriber) anElement;
                        if((aSub.getName()).equals(subSearch))
                        {
                            set.flagIt(aSub);
                        }
                    }
                }

[JosAH]

An excellent opportunity for System.out.println( ... ) the Poor Man's Debugger (tm). Sprinkle in some of those statements and print out everything that is of interest to you in in he body of the for loop and see for yourself.

kind regards,

Jos

[Scopic]

I tried this:

//flag a specified subscriber
                System.out.println("Enter Subscriber's name: ");
                subSearch = keyboard.nextLine().toUpperCase();
                System.out.println("Here.");
                for(int i = 0; i < set.size(); i++)
                {
                    System.out.println("Here.");
                    anElement = set.getCurrent();
                    System.out.println("Here.");
                    classId = anElement.getClassName();
                    System.out.println("Here.");
                    
                    if(classId.equals("Subscriber"))
                    {
                        System.out.println("Here.");
                        aSub = (Subscriber) anElement;
                        System.out.println("Here.");
                        if((aSub.getName()).equals(subSearch))
                            System.out.println("Here.");
                        {
                            System.out.println("Here.");
                            set.flagIt(aSub);
                            System.out.println("Here.");
                        }
                    }
                }

Result:

Here.
Here.
Here.
Here.

It looks like this is the culprit:

if(classId.equals("Subscriber"))
                    {
                        aSub = (Subscriber) anElement;
                        if((aSub.getName()).equals(subSearch))
                        {
                            set.flagIt(aSub);
                        }
                    }

Hmmm, now I'm even more confused. Our instructor provided us with the getClassName function. So I am unsure if it is doing it's job correctly.

Here it is:

public String getClassName()
    { 
        String resultStr;
        
        int location;
        
        resultStr = this.toString();
        location = resultStr.indexOf('@');
        return resultStr.substring(0, location);
    }

[Scopic]

I figured it out. The if statement was causing issues and was not even necessary. Here's the revised code:

System.out.println("Enter Subscriber's name: ");
                subSearch = keyboard.nextLine().toUpperCase();

                for(int i = 0; i < set.size(); i++)
                {
                    anElement = set.getCurrent();

                    aSub = (Subscriber) anElement;
                        if((aSub.getName()).equals(subSearch))
                        {
                            set.flagIt(aSub);
                        }
                        else
                        {
                            System.out.println("Member not found.");
                        }
                }

[JosAH]

See? The Poor Man's Debugger (tm) is of great value.

kind regards,

Jos