Как избежать повторения в заполнении массива

Question

У меня есть следующий код, и я хочу проверить, есть ли у меня значение внутри массива или нет. Проблема в том, что мой код выполняет сравнение только между значением, которое я даю в качестве параметра, и последним значением в массиве, то, что я хочу проверить, - это увидеть, что у меня есть значение, а затем возвращаю значение boolean true, но мой код просто сравнить последнее значение в массиве. Код здесь:

public boolean trueIdTienda(String s) { 
    boolean f = false; 
    for (int x = 0; x < lista.size(); x++) { 
     if (s.equals(ventas.getVenta(x).getIdTienda())) { 
      f = true; 
     } else { 
      f = false; 
     } 
    } 
    return f; 
} 

Answer 1

Это классический алгоритм поиска:

public boolean trueIdTienda(String s) { 
    for (int x = 0; x < lista.size(); x++) { 
     if (s.equals(ventas.getVenta(x).getIdTienda())){ 
      return true; 
     } 
    } 
    return false; 
} 

Answer 2

Эта часть не требуется:

else { 
     f = false; 
    } 

Так, если один из них совпадают, это совпадение, вам не нужно, чтобы установить его в true. Также вы можете вернуться после матча, чтобы немного ускорить код (и более логичный).

Если вы оставите там часть else, она вернет значение до false после того, как оно найдет неравный. Рассмотрим поиск [1, 2] для 1, и вы увидите.

Answer 3

Ваша функция может быть упрощена для немедленного возврата true, если значение найдено.

public boolean trueIdTienda(String s) { 
    for (int x = 0; x < lista.size(); x++) { 
     if (s.equals(ventas.getVenta(x).getIdTienda())) { 
      return true; 
     } 
    } 
    return false; 
} 

Answer 4

Может быть, вы можете использовать коллекцию. что-то вроде Hash должно помочь вам http://www.java2s.com/Tutorial/Java/0140__Collections/Createuniquelistsofitems.htm

Answer 5

Проблема заключается в том, что, хотя вы обнаружите, что значение в массиве равно аргументу установить логическое истинны и продолжить сравнение с другими значениями, которые возвращают логическое значение ложь.

Конечное значение логического значения является результатом сравнения с последним значением, чем решает эту проблему, чтобы добавить break; после f=true;

Try, чтобы понять, что вызвало проблему не просто используя правильный код.