Skip to content

Conversation

@Daikipro
Copy link

・7という数字がつくときはGitHubを表示

を実装しました。レビューお願いします。

@dfpro073
Copy link

インデントがおかしいようです。

@dfpro073
Copy link

今回の実装に関するテストコードがないので追加してください。

lib/fizzbuzz.rb Outdated
'fizz'
elsif number % 5 == 0
'buzz'
elsif number.to_s.include? '7'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

インデントがおかしいようです。

@Daikipro
Copy link
Author

@dfpro073 さんレビューありがとうございます!

仕様の質問があります。
例えば、75という数字は本件の実装前は"fizzbuzz"と表示されていたかと思います。
本件の実装後だと"7という数字"に該当するので"GitHub"と表示しています。
"fizzbuzzGitHub"などのように、複合して表示したいということはありませんか?

@dfpro073
Copy link

なるほど。そう言ったケースについて触れていませんでした。
仕様を以下の通りとしてください。

・3の倍数や5の倍数でも、7という数字がつく時はGitHubを表示

@Daikipro
Copy link
Author

テストコードを追加して、実装が終了しました。
レビュー後、問題がなければマージをお願いいたします。

@dfpro073
Copy link

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants