Open Johnny850807 opened 7 years ago
可讀性
你十年後再來看這一行，你真的能秒懂嗎？全世界就你懂。
第一個， == 2 這是什麼鬼 ? 誰知道 2 代表什麼，改掉。 第二個，你第一眼看到 setVictory() 覺得他在幹麻? 你會知道他在累積分數而不是設定勝利 ? 我還真以為這邊就剩定他贏了咧。
== 2
setVictory()
這種非可讀性的寫法要人教幾次才能舉一反三? 請用全世界的角度看你的程式碼。
可讀性2
你的CardStack根本沒有用阿，他根本就不是一個牌堆，你根本就沒用他來抽牌。 所以他是一個工廠好嗎，改名為 CardStackFactory 那你的層次就往上兩倍，可讀性也翻倍。
可讀性3
一樣，抽牌的動作要有回傳值才符合現實生活， 不然你就必須自己控管狀態，這種思維不可取，同樣的議題這邊提過。
下面就是你這樣做的後果，可讀性極差！
這種程式你自己必須思考怎麼改進吧，放著給我電幹嘛，handCards跟draw到底會做什麼事情最好一眼看得出來，你必須這樣自己記著這些狀態阿，不用十年，你三天後回來就矇了。
handCards
draw
你十年後再來看這一行，你真的能秒懂嗎？全世界就你懂。
第一個，
== 2這是什麼鬼 ? 誰知道 2 代表什麼，改掉。 第二個，你第一眼看到setVictory()覺得他在幹麻? 你會知道他在累積分數而不是設定勝利 ? 我還真以為這邊就剩定他贏了咧。這種非可讀性的寫法要人教幾次才能舉一反三? 請用全世界的角度看你的程式碼。
你的CardStack根本沒有用阿，他根本就不是一個牌堆，你根本就沒用他來抽牌。 所以他是一個工廠好嗎，改名為 CardStackFactory 那你的層次就往上兩倍，可讀性也翻倍。
一樣，抽牌的動作要有回傳值才符合現實生活， 不然你就必須自己控管狀態，這種思維不可取，同樣的議題這邊提過。
下面就是你這樣做的後果，可讀性極差！
這種程式你自己必須思考怎麼改進吧，放著給我電幹嘛，
handCards跟draw到底會做什麼事情最好一眼看得出來，你必須這樣自己記著這些狀態阿，不用十年，你三天後回來就矇了。