Java - Searching table for an object with a matching String

Question

I'm searching through a table for a customer with a matching name, if found then I want to return that customer, otherwise return null. The problem is I'm getting null all the time, I think my problem is with the if in the inner for loop.

public class testter {

    public Customer isCustomerInTable(Customer[][] table, String customerName) {
        for ( int r = 0; r < table.length; r++ ) {
            for ( int c = 0; c < table[r].length; c++ ) {
                if ( table[r][c].equals(customerName) ) {
                    return table[r][c];
                }
            }
        }
        return null;
    }
}

My customer class:

class Costumer {

    private String name;

    public Customer()  {
        name = "";
    }

    public void setCostumerName(String name) {
        this.name = name;
    }

    public String getCostumerName(){
        return name;
    }
}

Any ideas? Thanks

Rahul · Accepted Answer · 2013-06-03 06:18:25Z

6

You if condition should be

if (table[r][c].getCostumerName().equals(customerName))

You need to compare the names, but in your case, you are comparing the Customer object with the String customerName.

answered Jun 3, 2013 at 6:18

Rahul

45.2k11 gold badges89 silver badges108 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Jack Osborn Over a year ago

As soon as I finished submitting the question I got the answer by my self, but since yours is exactly what I had in mind. I'm gonna give you credits, Thanks

sarwar026 · Accepted Answer · 2013-06-03 06:18:55Z

1

You are checking Customer object with a string. That's the reason. You should do the following

if ( table[r][c].getCostumerName().equals(customerName) ) {
    return table[r][c];
}

answered Jun 3, 2013 at 6:18

sarwar026

3,8413 gold badges29 silver badges39 bronze badges

Collectives™ on Stack Overflow

Java - Searching table for an object with a matching String

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related