テストプログラムの作成について

Question

以下のMoney.javaの★★部分のテストプログラムをMoneyTestクラスのtestIsEqualsメソッドに追加したいのですが、うまくできません。。

---

if (obj == null) {
return false;
}

---

objがnullならfalseを返す部分なので、引数がnullのisEqualsメソッドを実行させ、falseが返ってくることを確認できればいいのですが、どのように記載すれば良いでしょうか。

MoneyTest.java

package practice;

import static org.junit.Assert.*;

import org.junit.Test;

public class MoneyTest {

	@Test(expected = IllegalArgumentException.class)
	public void testMoney() {

		   int amount1 = 1;
		   int amount2 = 0;
		   int amount3 = -1;

		   Money result1 = new Money(amount1);
		   assertEquals(amount1, result1.amount);

		   Money result2 = new Money(amount2);
		   assertEquals(amount2, result2.amount);
		   new Money(amount3);
	}

	@Test(expected = NullPointerException.class)
	public void testAdd() {

		Money self1 = new Money(1);
		Money m1 = new Money(2);

		Money self2 = new Money(1);
		Money m2 = null;

		Money result1 = self1.add(m1);
		assertEquals(3, result1.amount);

	    self2.add(m2);
	}

	@Test
	public void testIsEquals() {

	    Money self1 = new Money(1);
	    Money obj1 = new Money(1);

	    Money self2 = new Money(1);
	    Money obj2 = new Money(2);

	    Money self4 = new Money(1);
	    String obj4 = "unit08";

	    boolean result1 = self1.isEquals(obj1);
	    assertTrue(result1);

	    boolean result2 = self2.isEquals(obj2);
	    assertFalse(result2);

	    boolean result4 = self4.isEquals(obj4);
	    assertFalse(result4);
	}

}

Money.java

package practice;

/**
 * 金額を表すクラス
 */
public class Money {

    /** 金額 */
    int amount;

    /**
     * コンストラクター
     *
     * @param amount 金額
     * @throw IllegalArgumentException
     *             amountが負の場合にスローします。
     */
    public Money(int amount) {

        if (amount < 0) {
            throw new IllegalArgumentException();
        }

        this.amount = amount;
    }

    /**
     * 金額を足し算します。
     *
     * @param m 金額
     * @return 加算金額
     */
    public Money add(Money m) {
        return new Money(amount + m.amount);
    }

    /**
     * 同値を検査します。
     * amountの値が等しい時に限り、同値と判断します。
     *
     * @param obj 比較対象オブジェクト
     * @return 等しければ真
     */
    public boolean isEquals(Object obj) {

//★★ 
        if (obj == null) {
            return false;
        }
//★★


        if (!(obj instanceof Money)) {
            return false;
        }

        Money m = (Money) obj;
        return amount == m.amount;
    }
}

Comments

[maisumakun]

テストとは別件の問題なのでこちらに書きますが、equalsではなくisEqualsという名前であることにどういった意味がありますでしょうか（Object標準のequalsをオーバーライドするほうが適切ではないかと考えます）。

[icecleam]

ご質問ありがとうございます。
メソッド名がequalsではないことには、特に意味はなかったです。
でも確かにequalsの方がわかりやすいですね。

次回以降プログラムを書く際は命名する際には気をつけてみます。

Answer 1

ん？

     boolean result2 = self2.isEquals(obj2);
     assertFalse(result2);

が、書けるならこれと同じですけど。

     boolean result2 = self2.isEquals(null);
     assertFalse(result2);

みたいな感じでいいと思います。