既読ユーザー
貯金箱プログラムを作成してみた
ディスカッション
daidai3110 2020/11/22 10:13
全体的に前回のよりすっきり書けてる印象ですね！
気になった点を列挙すると
how_much?→ ?がついたメソッドはtrue/falseを返すという慣習があり、それに合致していないtotalの中でputsしてるのが違和感。合計の金額（要は数値だけ）を取得したい場合困っちゃう。continue_saving_answerメソッドって必要ないかも。全体的にインデントが4つになってる。
また、これはJavaの例だけど「オブジェクト指向エクササイズ」いいですよ。
https://qiita.com/opengl-8080/items/6f0a458df9c34eccf76c
Taishikun0721 2020/11/22 19:56
レビューありがとうございます！
how_much? → ?がついたメソッドはtrue/falseを返すという慣習があり、それに合致していない
確かにだいそんさんのコード全部そうなってますね！曖昧で使ってました笑
totalの中でputsしてるのが違和感。合計の金額（要は数値だけ）を取得したい場合困っちゃう。
あー確かに数値だけ取り出すパターンは考えてなかったです”！了解です！
continue_saving_answerメソッドって必要ないかも。
おけです笑
インデント了解す笑。言い訳ですが会社のvscodeで書いたからかな笑
Taishikun0721 2020/11/24 22:02
bankクラスがすっきりする様に分割してました
お手すきでみてください！daidai3110 2020/11/27 10:15
めちゃ遅くなってすみません...！！
だいぶよくなってると思います！
puts "何円貯金箱に入れますか？ press...#{CoinType::ALL_KINDS_OF_COIN.join(' or ')}"
こういう感じでハードコードを避けてるのも高評価！