栗原＠昼休み中です。
FileLockクラスとTODO.txtをコミットしました。
FileLockクラスは現時点では、attemptの引数に0を渡したときは、ロ
ックに失敗すると即時に戻る実装です。
また、InterruptExeptionのキャッチ部で、notifyを呼び出していました
が、synchronizeブロック内でもないのに使用できるわけもないので、
削除しました。
以上確認ください。
firo [mailto:firo@....jp] wrote:
> 多分三郎さんだと思うのだけれど、私の机の上に上のようなメモが
> あったわ。私もこのメモの意見には賛成。ただ
> if (contains(member))
> continue;
>
> add(member);
>
> のところは
>
> if (! contains(member)) add(member);
>
> じゃ、だめかしら？
しつこいと思われたらごめんなさい。
三郎さんは、メンバーの追加時に既にメンバーが登録されているときは、
特殊なケースであると考えました。
よって、リファクタリング（250）の、ガード節による入れ子条件記述の
置き換えを適用して、メンバーの情報が重複しているというのは、正常
ルートではないと明示したかったのだと思われます。
---
Tetsuya Kurihara
tetsuya@....jp