Date:  Tue, 5 Sep 2000 09:07:39 +0900
From:  omura@....jp
Subject:  [XP-jp:00814] Re: Member, MemberList, MemberAuthorization
To:  extremeprogramming-jp@....jp (extremeprogramming-jp ML)
Message-Id:  <20000905090646omura@....jp>
In-Reply-To:  Your message of "Mon, 4 Sep 2000 19:25:58 +0900"             <50158B900247D411B8120090CC01C0EC1E059D@....jp>
References:  <50158B900247D411B8120090CC01C0EC1E059D@....jp>
Posted:  Tue, 05 Sep 2000 09:06:46 +0900
X-Mail-Count: 00814

大村です。

コードをみていて二点気になりました。

もとのメールは、対話形式の中のコードをまとめられただけみたいだし、
対話形式の部分では、意図してこういうコードを書かれているのかもしれないし、
こういうコメントを書くのは、ちょっとはずしてるかもしれないなあと思うんですけど、
そのときはすいませんです。


1. MemberListTestクラスの中の次の部分

>    public void testEmptyGet() {
>        boolean exceptionThrown = false;
>        MemberList memberList = new MemberList();
>        try {
>            Member member = memberList.get(0);
>        } catch (IndexOutOfBoundsException e) {
>            exceptionThrown = true;
>        }
>        assert(exceptionThrown);
>    }

ですが、例外があがるかどうかのテストについては

public void testEmptyGet() {
     MemberList memberList = new MemberList();
     try {
         Member member = memberList.get(0);
         fail( "IndexOutOfBoundsException が必要なところ" );
     } catch ( e) {
         assert( true );
     }
 }

のように書くとよいと、FAQかなにかに書いてありました。


2. 同じく MemberListTestクラスの中で、

>        Member firstMember = new Member("aaa@....ccc");
>        Member secondMember = new Member("ddd@....fff");

はfixtureにしてしまうのがよいですよね。
実際の作業だと、同じテストデータを二度目に書くことになった時点で、それをfixtureにしようと
考えるようにするといいのかなと思います。

fixtureについて思うところがあるので、それはまた別メールで書きます。