Re: [問題] 一個關於 strategy pattern 的疑惑

[adrianshum (Alien)]

開宗明義第一句：Stragety 做成 Singleton 不是正解，
不要浪費時間。（後面再來詳解）

首先，Strategy 解決的問題是，把不同情況需要不同的
邏輯，包裝成所謂 Strategy, 然後只要替換 strategy，
就能不影響caller 的 code 之下做到不同的工作。（
很大概的說法）。

回到你的問題，在替換 strategy 究竟需不需要每次生成，
這根本不是 Strategy 涵蓋的範圍。

解決方法可以很多。每次生成當然可以（後面再說怎麼
解決你的擔心），更直接的是，一個 Character 可以存
著自己可用的技能, 所謂替換，就是在手頭可用的技能
拿出對應的來使用而已：

class Character {
    AttackStrategy activeAttackStrategy;
    List<AttackStrategy> attackStrategies;

    void selectStrategy(int index) {
        activeAttackStrategy = attackStrategies.get(index);
    }
}

或者用類似的想法。簡單而言，要替換，並不代表每次要生成
新的。

然後有些回應提到用 把 strategies 做成singleton。請千萬不要。
在這種情況下99% 不是正解。Singleton 的用意是這 class 在意義上
在整個系統只存在一份，但這些 Strategy 並沒有這樣的需要。更何況，
很多時候一個 strategy 會存有 state, 例如:

class FireballStrategy implements AttackStrategy {
  int level = 1;
  public void setLevel(int level) {...}
  public void attack() {
    reduceHpBy(level * 100);
  }
}

這樣怎麼可能適合用 singleton 呢？就算你的設計上不存在 state，
概念上你的 strategy 也不是 singleton。

要是你擔心生成太多，解決方法也應該是 Flyweight pattern 而非
Singleton。

至於有一篇回文大概是存個 string 然後每次找 factory 去拿
strategy 再 invoke，個人覺得只是換湯不換藥，再加上上面提
到 strategy 很多時候是有internal state 的，這種解決方法
並不理想。把 factory 弄成 singleton 更是不必要。這可以算是
濫用 singleton 的好例子 :p

（整個討論本身好像和 Java 沒太大關係...）

Alien

--
※ 發信站: 批踢踢實業坊(ptt.cc)
◆ From: 223.19.42.175
※ 編輯: adrianshum      來自: 223.19.42.175        (06/23 23:11)
※ 編輯: adrianshum      來自: 223.19.42.175        (06/23 23:15)
※ 編輯: adrianshum      來自: 223.19.42.175        (06/23 23:19)
推 ooooooo:感謝，這篇揭開了一開始我聽到 singleton 莫名不安的原因!
→ TonyQ:同意
推 PsMonkey:嗚嗚... 單純作成公式不好嗎？ [淚目]
→ PsMonkey:還想確認一下，這裡套用 flyweight pattern 不會出問題嗎
→ PsMonkey:我狗了一下幾個範例，flyweight 的 class 都沒有 setter
→ swpoker:釐清物件的責任~樣式混用~我的例子是工廠環境單一~
→ swpoker:雖是單一~程式其實沒有單一~而是用注入來做
→ swpoker:哈~我後來有補充~就是解釋你的問題
→ adrianshum:PsMonkey: strategy 就是代表公式，只是想把公式替換
→ adrianshum:另 Flyweight 重點是存著一份重覆使用，當然沒 setter
→ adrianshum:的 immutable obj 比較合，所以我也提及 strategy 會有
→ adrianshum:internal state, 套singleton (flyweight) 或會出問題
→ adrianshum:@swpoker:另有factory去生成strategy沒錯（有沒有必要
→ adrianshum:另一問題），可是錯在每次去找factory生成。這樣的
→ adrianshum:strategy 用法並不恰當。另當中提到 facade也是多餘的
→ adrianshum:建議你先把pattern 作最少的使用並用得清楚，再來套更
→ adrianshum:多 pattern. 我在你的回應中嗅到濫用pattern的味道
推 PsMonkey:喔喔... 我把整篇文章混在一起看了，所以誤會了 XD
→ swpoker:其實這是我剛開始想的~首先HELLO WORLD然後再重構
→ swpoker:利用TDD來作為開始,並且用來確認整理釐清物件及行為
→ swpoker:好吧我只點出問題並不在"策略樣式"而已~