java - зарегистрировать проблему

Question

Когда я пытаюсь зарегистрировать человека с именем Eric например, а затем снова регистрирую Eric, он работает. Это не должно происходить с кодом, который у меня есть. Эрика не следует регистрировать, если в списке уже есть Эрик. Вот мой полный код:

 private Person findName(String name) { 
     for (Person person : personer) { 
      if (person.getName() == name) { 
       return person; 
      } 
     } 
     return null; 
    } 





private boolean containsName(String name) { 

     return findName(name) != null; 

} 

Answer 1

Ваша проблема с этой линией:

if (person.getName() == name) 

Вы хотите

if (person.getName().equals(name)) 

В Java, "==" является ссылкой равенства для не -примитивные типы. Ссылка на name (то есть его «адрес», если хотите) не совпадает с ссылкой на объект, возвращаемый getName().

Смотрите также: ответ http://www.java-samples.com/showtutorial.php?tutorialid=221

Answer 2

Брайана является правильным. Тем не менее, на уровне «лес вместо деревьев» необходимо также исправить дизайн кода:

Если вам нужна коллекция отсортированных имен и вы можете искать существующие имена, вы должны использовать TreeMap. ArrayList (который вы сортируете по каждой вставке, не менее) - это совершенно неправильная структура данных для этого.