件名:

Memento 、Stateパターンの章の誤記?

MLNo. [DP/ML:1465]
差出人: torutkさん
TAKAHASHI Toru<torutk…>
送信日時: 2002/03/31 18:14
本文:
文字を小さく文字を中くらいに文字を大きく
高橋(徹)です。

先日の読書会でMemento,State,Flyweight,Proxyを読みました。
月1回のペースで順調に開催しております。次回(6回目)で
読了となる予定です。
読書会のページ
http://www.t3-jpn.com/bof/

今回気付いた書籍の誤りと思われる点をお知らせします。

・Fig.18-1のGamerクラス
bet, createMemento, restoreMementoは公開メソッドなので+記号
付いた方がよいのではないかと思います。可視性の記号は省略可能
ですが、同じクラスの箱の中に可視性の記号がついた属性やメソッド
があったので、付けていいものについても何か意図があるのかも
しれないと考えてしまいました。

・Fig.18-4のMementoクラス
<<wide interface>>
getPublicInfo
<<narrow interface>>
#getProtectedInfo
は、逆ではないでしょうか?
Caretaker役が利用するのがナローインタフェースなので、ここでは
getProtectedInfoではなくgetPublicInfoになると思われます。

・List A19-3
doClockメソッドのif文において、1つ目と2つ目の条件判断の順番が
逆ではないでしょうか?
下記のコードでは最初の条件が2つ目の条件を含んでしまい、2つ目の
条件が成立する時は必ず1つ目の条件が成立します。
if (9 <= hour && hour < 17) {
context.changeState(DayState.getInstance());
} else if (12 <= hour && hour < 13) {
context.changeState(NoonState.getInstance());
}


---
Toru TAKAHASHI
引用文表示/非表示 辞書リンク表示/非表示
添付:
このMLメールに返信(レス)する!
このMLメールには下記のMLメールが返信されています:
[DP/ML:1477] Memento, Stateのエラー 結城浩さん)
 読み込み 中...
<<1464:自動リスト分岐 1466:Re: Memento 、 Stateパターンの章の誤記?>>
MLメール表示切替 トピック表示 メール表示
MLメール一覧

MLトピック検索:
MLメールの詳細(メール表示)
MLメール表示切替 トピック表示 メール表示
MLメール一覧

メーリングリストでつながる!ひろがる!みんなのHappyコミュニティ!

詳細検索
NOEMBLEM/エンブレムが設定されていません。
デザインパターン・メーリングリスト (3906)
 
28 29 30 1 2 3 4
5 6 7 8 9 10 11
12 13 14 15 16 17 18
19 20 21 22 23 24 25
26 27 28 29 30 31 1
メンバーリストは非公開です。
メンバーの日記/フォト/動画
メンバーの投稿は非公開です。
 読み込み中...