yasushito111 / python Goto Github PK

Bob.pyを拝見しました。再帰関数を見事に実装されてますね。

昨日お伝えしましたが、

1. bottle数の文字列を関数に切り出し
2. 変数tmpを削除する

ともっと良くなるかと思います。

1. bottle数の文字列を関数に切り出し

bottle数は以下の3パターンで文字列が分かれるかなと思います。

そして、bottles_of_beer() では bob , bob - 1 の2パターンのbottle数を表示する必要があるかと思います。

これを bottles_of_beer() 内で実装すると冗長になるため、別関数に切り出すとスッキリします。

2. 変数tmpを削除する

昨日、tmpを削除したらうまく行かなかったと仰っていましたが、おそらく bob -= 1 を削除していないのかなという気がします（tmpを削除した場合の書き方はいくつかあると思います）。

最後に

Issueに対応するPullRequestを作成すると、変更理由と変更内容が紐づけられて便利です。