Question

I’m trying to write a method that looks through an array of objects for a certain color, which is also an object.

public Ghost findFirst(Color c){
        for (int i=0;i<ghosts.length;i++) {
            if (ghosts[i].getColor()==c)
            return ghosts[i];
        else
            return null;
            }  
    }

So if the color of a certain ghost matches color c, then return that ghost. However, I’m getting a dead code warning for i++. Whats wrong with my code? Oh also I’m getting a method error saying this function should return a ghost. I thought I am?

Answer 1

Because you return from the loop on the first iteration! So “i” never gets incremented. Either remove the else block completely or change the “return null” to “continue”.

As a separate point, the == is checking referential equality, not object equality. It seems likely you should be using “.equals” instead