のassertEquals（）HashSetの

Question

と私はプレイヤーのクラスのコンストラクタを持って

public Player(String name, String playerDescription, 
     HashSet<String> abilities) { 

      this.name = name; 
      this.playerDescription; 
      this.abilities = abilities; 
} 

私はに追加する、私はaddAbilities()メソッドを作成しました

Player p = new Player ("Jon Snow", "Brave Swordsman", new HashSet<String>()); 

このクラスのインスタンスを作成しますHashSet。しかし

public void addAbilities(String newAbility) { 
    abilities.add(newAbility); 
} 

、私は

public String getAbility() { 
    String abilityString = abilities.toString(); 

    return abilityString; 
} 

私のテスト（getメソッドを使用して）メソッドをテストするために行く：

@Test 
public void testAddAbility() { 
Player p = new Player("Jon Snow", "Brave Swordsman", new HashSet<String>()); 

    s.addAbility("Leadership"); 
    assertEquals("Leadership", s.getAbility()); 
} 

私は、テスト出力の違いを取得します。

比較の失敗：予想：<[Leadership]>しかし<[[Leadership]]>

私の質問は、なぜ出力の差がありますされ、HashSetこの方法を作成することをお勧めしましたか？

Answer 1

HashSet.toString()が[]内の要素を表示する文字列を返す（およびそれらの間の,文字で）行うことをお勧めしたいです。
あなたのアサーションでは、Leadershipと[Leadership]を比較します。

これは、アサーションのために[OK]を次のようになります。

assertEquals("[Leadership]", s.getAbility()); 

しかし、私はあなたではなく能力を取得する方法を変更する必要があることだと思います。
検索と書式設定の方法ですか？ この意味でメソッドの名前を変更してください。

そうでない場合は、Set抽象化を維持し、それがあるとしてだけSetを返す：

public Set<String> getAbilities(){ 
    return abilities; 
} 

Answer 2

Set toString出力のフォーマットは "[item1、item2、...]"です。アイテムは1つしかないので、「[item1]」です。

私はあなたがこの方法

まだ

public Set<String> getAbilities() 

またはそれ以上

public Set<Ability> getAbilities() 

Answer 3

あなたのコードは、最初のものは

能力が収集されているポイントのカップルで、潜在的な欠陥を持っています、 「ゲッター」は、文字列というのJava JDKのdeveloをバック与えているように、それは文字列

public String getAbility() { 
    String abilityString = abilities.toString(); 

    return abilityString; 
} 

を返すためので、このゲッターは正しくありません。人は誰にも知らずに変わることがあります...そのことを考えると、

@testが失敗しています....

あなたは文字列値が存在するかどうかを確認ゲッターにし、そこから（変更不可能な）コレクションを返す必要があります...

Set<String> mySet = new HashSet<>(); 
mySet.add("Leader"); 
mySet.contains("Leader"); 

は今、あなたはのtoStringと、将来的に何が起こっているかのindeendentです

public boolean hasAbility(String ability) { 
    return abilities.contains(ability); 
} 

を追加をテストするためとして：私はgetAbility()メソッドを削除し、追加することをお勧めHashSetの

Answer 4

の実装この方法、あなたがこれを行うことができます：

@Test 
public void testAddAbility() { 
    Player p = new Player("Jon Snow", "Brave Swordsman", new HashSet<String>()); 
    p.addAbility("Leadership"); 

    assertTrue(p.hasAbility("Leadership")); 
}