件名:

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/エンブレムが設定されていません。
デザインパターン・メーリングリスト (3963)
 
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 2
メンバーリストは非公開です。
メンバーの日記/フォト/動画
メンバーの投稿は非公開です。
 読み込み中...