java - searching value inside a multi dimensional array

Question

I need to search an array and find a certain value, if it exists returns true, and if it doesn't exist returns false.

The array:

private String zonaDeInternamento[][][] = {
    {
        {"Livre", "Livre", "Livre", "Livre"},
        {"Livre", "Livre", "Livre", "Livre"},
        {"Livre", "Livre", "Livre", "Livre"},
        {"Livre", "0012", "Livre", "Livre"},
        {"Livre", "Livre", "Livre", "Livre"}
    },
    {
        {"Livre", "0013", "Livre", "Livre"},
        {"Livre", "Livre", "Livre", "Livre"},
        {"Livre", "Livre", "Livre", "Livre"},
        {"Livre", "Livre", "Livre", "Livre"},
        {"Livre", "Livre", "Livre", "Livre"}
    }
};

So, If the search finds one of the numbers, it returns true, if it finds "Livre", returns false;

public boolean isPacienteInternado(String numeroProcesso) {
    if (isNumeroProcessoValido(numeroProcesso)) {
        for (int i = 0; i < zonaDeInternamento.length; i++) {
            for (int j = 0; j < zonaDeInternamento[i].length; j++) {
                for (int h = 0; h < zonaDeInternamento[i][j].length; h++) {
                    if ((zonaDeInternamento[i][j][h].equals(numeroProcesso))) {
                        System.out.println("O paciente com número de processo " + numeroProcesso + " está internado!");
                        return true;
                    } else {
                        System.out.println("O paciente com número de processo " + numeroProcesso + " não está internado!");
                        return false;
                    }
                }
            }
        }
    }
    return false;
}

It keeps returning false, never goes inside the if statement, returns the else message and false. Running debug if ((zonaDeInternamento[i][j][h].equals(numeroProcesso))) returns false when numeroProcesso is 13 or 12. What am I missing?

You need to iterate all the items before deciding if the item is present or not. Remove the else loop inside the nested for. — BatScream
– BatScream, Commented Dec 4, 2014 at 23:59
I need this statement if (!((zonaDeInternamento[i][j][h].equals(numeroProcesso)))) to be returned as false, and need it to be inside the nested for — Nuno Teixeira
– Nuno Teixeira, Commented Dec 5, 2014 at 0:12

kamoor · Accepted Answer · 2014-12-05 00:11:59Z

1

public boolean isPacienteInternado(String numeroProcesso) {
if (isNumeroProcessoValido(numeroProcesso)) {
    for (int i = 0; i < zonaDeInternamento.length; i++) {
        for (int j = 0; j < zonaDeInternamento[i].length; j++) {
            for (int h = 0; h < zonaDeInternamento[i][j].length; h++) {
                if ((zonaDeInternamento[i][j][h].equals(numeroProcesso))) {
                    System.out.println("O paciente com número de processo " + numeroProcesso + " está internado!");
                    return true;
                }
            }
        }
    }
}
return false;}

answered Dec 5, 2014 at 0:11

kamoor

2,9493 gold badges21 silver badges34 bronze badges

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

Comments

hiimjames · Accepted Answer · 2014-12-05 00:12:26Z

1

Your for loop makes only one itaration because you return true or false in first step. So delete the else statement.

if ((zonaDeInternamento[i][j][h].equals(numeroProcesso))) {
   System.out.println("O paciente com número de processo " + numeroProcesso + " está internado!");
   return true;
}

answered Dec 5, 2014 at 0:12

hiimjames

5381 gold badge5 silver badges11 bronze badges

2 Comments

Nuno Teixeira Over a year ago

I know that removing the else makes it work, but I need the else statement to print a message.

hiimjames Over a year ago

So keep your else statement only with System.out.println and remove the return statement from here.

chitraketu Pandey · Accepted Answer · 2014-12-05 00:14:17Z

1

You have given return false at the end of the block which is causing the error remove the return false at the bottom.

public boolean isPacienteInternado(String numeroProcesso) {
   bool value;
   if (isNumeroProcessoValido(numeroProcesso)) {
      for (int i = 0; i < zonaDeInternamento.length; i++) {
         for (int j = 0; j < zonaDeInternamento[i].length; j++) {
             for (int h = 0; h < zonaDeInternamento[i][j].length; h++) {
                 if ((zonaDeInternamento[i][j][h].equals(numeroProcesso))) {
                     System.out.println("O paciente com número de processo " + numeroProcesso + "                 está internado!");
                    bool = true;
                } else {
                    System.out.println("O paciente com número de processo " + numeroProcesso + " não está internado!");
                    bool = false;
                }
            }
        }
    }
}
return bool;

}

edited Dec 5, 2014 at 0:14

answered Dec 5, 2014 at 0:02

chitraketu Pandey

1531 gold badge1 silver badge8 bronze badges

1 Comment

Nuno Teixeira Over a year ago

I need that return statement, else it gives an error when running

CBredlow · Accepted Answer · 2014-12-05 00:17:27Z

You are only checking one item in the entire array and if it is not equal to numeroProcesso then you are exiting the loop. Instead you need to continue looping until either 1. You found a match or 2. You iterated over every item in the array. Instead you want something like this:

public boolean isPacienteInternado(String numeroProcesso) {
if (isNumeroProcessoValido(numeroProcesso)) {
    for (int i = 0; i < zonaDeInternamento.length; i++) {
        for (int j = 0; j < zonaDeInternamento[i].length; j++) {
            for (int h = 0; h < zonaDeInternamento[i][j].length; h++) {
                if ((zonaDeInternamento[i][j][h].equals(numeroProcesso))) {
                    System.out.println("O paciente com número de processo " + numeroProcesso + " está internado!");
                    return true;
                } 
            }
        }
    }
    // let the user know that a match was not found
    System.out.println("Combinar não encontrado");
}
return false;

}

Collectives™ on Stack Overflow

java - searching value inside a multi dimensional array

4 Answers 4

Comments

2 Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

2 Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related