java - registerの問題

Question

たとえば、私がEricという名前の人物を登録しようとすると、もう一度Ericを登録すると動作します。これは私が持っているコードでは起こらないはずです。 ericが既に登録されている場合は登録しないでください。 "=="

if (person.getName() == name) 

あなたはJavaでは

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

をしたい、非ため参照の等価です：

 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

あなたの問題は、この行である：ここでは私の完全なコードです - 主な型。 nameへの参照（つまり、その "アドレス"）は、getName()によって返されたオブジェクトへの参照と同じではありません。

も参照してください：http://www.java-samples.com/showtutorial.php?tutorialid=221

Answer 2

ブライアンの答えは正しいです。ただし、「ツリーの代わりにフォレスト」レベルでは、コードのデザインも修正する必要があります。

ソートされた名前のコレクションが必要で、既存の名前を検索できるようにするには、TreeMapを使用する必要があります。 ArrayList（各挿入時にソートしてもそれ以下である）は完全に間違ったデータ構造です。