ソースコード添削お願いします。

Question

###該当のソースコード

day_sum = gets.chomp.to_i
weathers = gets.chomp.split(",")
weathers.each do|weather|
s_sum = 0
if weather == "S"
s_sum += 1
end
c_sum = day_sum - s_sum
print s_sum
print c_sum
end

Answer 1

いちばん根本として、paizaの問題を、公開の場所で質問することが間違っています。

Comments

[Anzunba]

問題文など全て削除しました。以後、気をつけます。

[maisumakun]

そういう問題じゃない気もしますが…

[izkn]

悪辣、もしくは馬鹿だね。

[Anzunba]

この業界のことは初めてで、ただプログラミングというのをやってみたくて、制限時間いっぱい考えて恐らく惜しいとこまで行ってるけど結局わからなかったので質問したというのが経緯です。恐らくpaizaの求人に関わっているということが問題なのですね(これも見当違いだったらすみません。)まず質問の仕方から他の人の投稿をみて参考にしてみます。

Answer 2

day_sum = 5
weathers = 'SSRSR'.split("")
s_sum = 0
weathers.each do |weather|
  if weather == "S"
    s_sum += 1
  end
end
c_sum = day_sum - s_sum
print "#{s_sum}\s#{c_sum}"

間違えているところ
1.区切り文字がカンマではない。
2.繰り返しの中にs_sum = 0を入れているため、毎回数字がリセットされている。
3.出力形式が異なっている。

その他の方法

a, b = $stdin.map(&:chomp)
puts "#{b = b.count('S')}\s#{a.to_i - b}"

Comments

[Anzunba]

ありがとうございます。

[退会済みユーザー]

いえいえ。どういたしまして。