質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.80%

Ruby fizzbuzz問題

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 4,734

当方初心者で、要件通り(実行結果通り)にコードが書けなくて困っています。
具体的には、作成したコードだと、4回ループしてしまうので、1回で済む様にしたいです。
ご教授下さい。

FizzBuzzのメソッドの仕様
特定の整数値を引数として受け取り
数値によって戻り値が変わる(条件は下記)
数値が3の倍数であれば、戻り値は”Fizz”
数値が5の倍数であれば、戻り値は”Buzz”
数値が3の倍数であり5の倍数でもある場合は、戻り値は”FizzBuzz”
上記のどれも満たさない場合は、その数値自体を戻り値に.

FizzBuzzのメソッドの使い方
FizzBuzzのメソッドを実行する前に下記の1行を記述(代入する数値は100でなくても構わない)
num_max = 100
1から num_max までの整数値についてループを作り、FizzBuzzのメソッドを実行
FizzBuzzのメソッドの戻り値をターミナルに表示
num_max の値を100としたとき、正しく書かれたプログラムを実行した結果を下記に示す。


以下作成したコード

def fizzbuzz(num)
    (1..100).each do |n|
        if n % 15 == 0
            puts "FizzBuzz"
        elsif n % 3 == 0
            puts "Fizz"
        elsif n % 5 == 0
            puts "Buzz"
        else
            puts n
        end
    end
end

puts fizzbuzz(1)
puts fizzbuzz(3)
puts fizzbuzz(5)
puts fizzbuzz(15)


以下要件通りの実行結果

1
2
Fizz
4
Buzz
Fizz
7
8
Fizz
Buzz
11
Fizz
13
14
FizzBuzz
16
17
Fizz
19
Buzz
Fizz
22
23
Fizz
Buzz
26
Fizz
28
29
FizzBuzz
31
32
Fizz
34
Buzz
Fizz
37
38
Fizz
Buzz
41
Fizz
43
44
FizzBuzz
46
47
Fizz
49
Buzz
Fizz
52
53
Fizz
Buzz
56
Fizz
58
59
FizzBuzz
61
62
Fizz
64
Buzz
Fizz
67
68
Fizz
Buzz
71
Fizz
73
74
FizzBuzz
76
77
Fizz
79
Buzz
Fizz
82
83
Fizz
Buzz
86
Fizz
88
89
FizzBuzz
91
92
Fizz
94
Buzz
Fizz
97
98
Fizz
Buzz
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • winterboum

    2019/07/15 08:28

    判らないことが3つあります
    1)「実行する前に下記の1行を記述  num_max = 100」とありますが、codeにそれがない。どう扱うのか?(まぁ大体わかりますが、codeに示してください)
    2)引数の num が使われていない。これはどうつかう?
    3)「作成したコードだと、4回ループしてしまう」
     100回ループしているように見えます。
    それとも if--endが4つにわかれていることを言ってますか?

    キャンセル

  • y_waiwai

    2019/07/15 08:28

    現状のコードでは何が問題なんでしょうか

    キャンセル

  • jiyu-jin2393114

    2019/07/15 09:50

    1) = num_max=100はどの行に配置すべきなのでしょうか?
    2) = num修正させていただきました。
    3) = 伝え方が下手ですみません、以下コードだと実行結果が1~100までを4回繰り返してしまいます。
    環境はcloud9です。
    def fizzbuzz(num)
    (1..100).each do |num|
    if num % 15 == 0
    puts "FizzBuzz"
    elsif num % 3 == 0
    puts "Fizz"
    elsif num % 5 == 0
    puts "Buzz"
    else
    puts num
    end
    end
    end

    puts fizzbuzz(1)
    puts fizzbuzz(3)
    puts fizzbuzz(5)
    puts fizzbuzz(15)

    キャンセル

  • otn

    2019/07/15 10:59

    仕様を完全無視してコードを書いているようですが、これが意図的じゃなくて、仕様に沿ってコードを書いたつもりなら、日本語読解能力がかなり低いと言うことなので、まずは日本語読解の練習をしないと、また同じ事を繰り返すだけだと思います。

    キャンセル

回答 1

checkベストアンサー

+1

こういうことでしょうか?

def fizzbuzz(n)
      if n % 15 == 0
          puts "FizzBuzz"
      elsif n % 3 == 0
          puts "Fizz"
      elsif n % 5 == 0
          puts "Buzz"
      else
          puts n
      end
end
num_max = 100
(1..num_max).each do |n|
  puts fizzbuzz(n)
end

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/07/15 10:51

    有り難う御座います!理解できました。定義とメソッドを分けずに書いて、勝手に混乱していていました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.80%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る