Date:  Sat, 30 Sep 2000 16:26:53 +0900
From:  firo@....jp
Subject:  [XP-jp:00990] Re: MemberList 更新
To:  extremeprogramming-jp@....jp (extremeprogramming-jp ML)
Message-Id:  <39D59A0C.4CF43F14@....jp>
References:  <97BA340C0480D411BDA800062939A1890607BA@....jp>
Posted:  Sat, 30 Sep 2000 16:45:19 +0900
X-Mail-Count: 00990

矢崎です。

花子モードで。

tetsuya@....jpさん wrote:

>
> >     if (contains(member))
> >         continue;
> >
> >     add(member);
> >
> > のところは
> >
> >     if (! contains(member)) add(member);
> >
> > じゃ、だめかしら？
>
> 三郎さんは、メンバーの追加時に既にメンバーが登録されているときは、
> 特殊なケースであると考えました。
>
> よって、リファクタリング（250）の、ガード節による入れ子条件記述の
> 置き換えを適用して、メンバーの情報が重複しているというのは、正常
> ルートではないと明示したかったのだと思われます。

私の早とちりで、正しい以下のコードを

     if (contains(member))
         continue;

     add(member);

下のように取り違えていたの。

     if (contains(member))
         continue;

     else add(member);

それと、
リファクタリング（250）の、ガード節による入れ子条件記述
の件は了解しました。
だから、上記部分は元のとおりにして、コミットしておきます。


--
矢崎 博英 <firo@....jp>