FizzBuzz問題について

Question

FizzBuzz問題を解いています。
３の倍数の時にFizz
５の倍数の時にBuzz
15の倍数の時にFizzBuzzと出力するものです。
今回は、数字をカンマ区切りで表示したいのですが、
最後に出力した数字の後ろにはカンマを表示したくありません。
どのように記述したら可読性を損ねずにカンマ区切りで数字を出力できますか？

(1..100).each do |i|
  print "FizzBuzz" if i % 15 == 0
  print "Fizz" if i % 3 == 0
  print "Buzz" i % 5 == 0
  print i
end

Answer 1

まず、ご質問の本題から外れますが、
質問文のサンプルコードは、
"Buzz"の後ろにifが抜けています。

それから、文字が表示されるときは数字が表示されないのが
一般的なFizzBuzz問題の仕様ですが、分かりやすくするため
なるべくサンプルに近づけて修正すると以下のようになります。

(1..100).each do |i|
  print "FizzBuzz," if i % 15 == 0
  print "Fizz," if i % 3 == 0
  print "Buzz," if i % 5 == 0
  print i
  print "," if (i % 100 != 0)
end

「print i」の後に一行足すのと少々で、
カンマ区切りについては実現できます。

Answer 2

結果を逐一出力せずarrayに入れて
ループを抜けた後、Join(",")したものを出力するのはどうでしょう？

Comments

[pinokio]

ご回答ありがとうございます。
以下のようにコードを書いてみたのですが、正しく出力されました！
```lang-ruby
nums = []
(1..100).each do |i|
i = "FizzBuzz" if i % 15 == 0
i = "Fizz" if i % 3 == 0
i = "Buzz" if i % 5 == 0
nums << i
end

print nums.join(",")
```

このコードよりも可読性が上がったり、コードを短くする方法をご存知でしょうか？
教えていただけましたら幸いです。

[otn]

FizzBuzzはいろいろありますが、この流れの延長線では、こうでしょうか。
倍数の時には、i を出さない本来のFizzBuzzという前提で。
puts((1..100).map do |i|
case 0
when i % 15 then "FizzBuzz"
when i % 3 then "Fizz"
when i % 5 then "Buzz"
else i
end
end.join(","))

Answer 3

',' の出力を if 文で判定させることを追加すれば良いと思います。

num = 100
(1..num).each do |i|
  print "FizzBuzz" if i % 15 == 0
  print "Fizz"     if i % 3  == 0
  print "Buzz"     if i % 5  == 0
  print i
  print ','        if i != num
end

Answer 4

邪道ですが、最後にバックスペースを出力して末尾のカンマを消すとか・・・

(1..100).each do |i|
    if i % 15 == 0
        print "FizzBuzz"
    elsif i % 3 == 0
        print "Fizz"
    elsif i % 5 == 0
        print "Buzz"
    else
        print i
    end
    print ","
end
print "\b"