Skip to content

Conversation

@Jankabi
Copy link

@Jankabi Jankabi commented Aug 8, 2024

  • 7という数字がつくときはGitHubを表示
    を実装しました。レビューお願いいたします。

@Jankabi
Copy link
Author

Jankabi commented Aug 8, 2024

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

@Jankabi
Copy link
Author

Jankabi commented Aug 8, 2024

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

@Jankabi
Copy link
Author

Jankabi commented Aug 8, 2024

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

@Jankabi
Copy link
Author

Jankabi commented Aug 8, 2024

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

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

@Jankabi
Copy link
Author

Jankabi commented Aug 8, 2024

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

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.

1 participant