Abstract Factory: Lin

[Johnny850807]

命名

setup中文是建置、準備，不應該有回傳值， 而create才是產生的意思，應該要有回傳值， 兩個使用情境相反了。

維護性

你的Role的變動性是生命跟魔力，而每個Role依賴抽象的工廠來取得裝備，此為Bridge模式，很棒。 但是假如我今天要開發一個新的Role，繼承你的Role之後完全不知道接下來該怎麼做，此為維護性疏失。 開發子類別的人根本不知道有hp跟mp可以使用。

解決方法：使用兩個Template Method讓子類別回傳自己的生命及魔力 (就像名稱一樣)

重複利用

有重複的邏輯，請把可重複利用的函數產生出來並重構，這樣的程式碼一看便不重用你。 請想辦法解決這問題，非常重要，這樣的程式碼請勿留下。

多型

重複邏輯*2，這樣的程式碼請你改到正確為止，你似乎忘記多型怎麼用了，自己走回去二轉第一題。

可讀性

請化簡並重構你的程式碼，不自律的工程師，是沒辦法有成就的。