Rubyで配列の要素の余りを合計するメソッドを作りたいです。

Question

Rubyで、配列の要素の余りを合計するメソッドを作りたいです。
具体的には、配列arrayと割る数nを引数にとり、arrayの各要素をnで割った合計値を返すメソッドmod_sumを作りたいです。自分の書いたコードは、

def mod_sum(array, n)
sum = 0
array.each do |i|
sum += i%n
end
return sum
end

なのですが、実行しようとすると、実行出来ません、というエラーが表示されます。どこが間違っているのでしょうか？
初心者なので見つけられません。よろしくお願い致します。

Comments

[maisumakun]

> 実行出来ません、というエラーが表示されます。

正確なエラーメッセージを書いていただけますか？

[ji46ksi]

メソッド add([1, 2, 3, 4, 5],1) の実行に失敗しました。実装を見直してください。

/root/codecheck/main.rb:2:in `mod_sum'

test.rb:6:in `runTest'

test.rb:24:in `block in <main>'

test.rb:24:in `each'

test.rb:24:in `<main>'

メソッド add([1, 2, 3, 4, 5, 6],2) の実行に失敗しました。実装を見直してください。

/root/codecheck/main.rb:2:in `mod_sum'

test.rb:6:in `runTest'

test.rb:24:in `block in <main>'

test.rb:24:in `each'

test.rb:24:in `<main>'

メソッド add([100, 120, 10055, 102, 64, 23],10) の実行に失敗しました。実装を見直してください。

/root/codecheck/main.rb:2:in `mod_sum'

test.rb:6:in `runTest'

test.rb:24:in `block in <main>'

test.rb:24:in `each'

test.rb:24:in `<main>'

メソッド add([],111) の実行に失敗しました。実装を見直してください。

/root/codecheck/main.rb:2:in `mod_sum'

test.rb:6:in `runTest'

test.rb:24:in `block in <main>'

test.rb:24:in `each'

test.rb:24:in `<main>'

というエラーが表示されました。CODEPREPという学習サイト上で書いたものなので変なエラーになっているかもしれません。

[maisumakun]

メソッドの名前はmod_sumなのですか、addなのですか？

[ji46ksi]

一応問題文のうえではmod_sumで書けということになっていました。addでも試したのですが、やはり同じエラーで失敗しました。

[maisumakun]

問題出題側で回答すべき形と書いたコードが合っていない、ような状況にも思えます。

[otn]

エラーメッセージと、コードがマッチしてませんので、自分が何をやっているのか理解できていない状態のようです。
実行したコードのそのままと、それを実行して表示されたそのままを、手を加えずにそのまま質問文を編集してコピペしましょう。

[ji46ksi]

CODEPREPというサイト上で出題さている問題の一つでして、自分はメソッドだけ書いて、その他の作業は全てサイト側がしてくれています。なので、ここに書いてあるものは全てそのままの状態です（文がわかりにくくてすみません）。

mapを使って書き直したら無事にクリアすることが出来ました。ご回答、ありがとうございました。

[gouf]

解決した場合、他者の回答ないし自己回答として内容をまとめて それをベストアンサーに設定することで、この質問を解決済みにすることができます

[ji46ksi]

すみませんありがとうございます。

Answer 1

sum=0

def mod_sum(array, n)

array.map(|i| sum+=i%n)

return sum

end

としました。

Answer 2

同じような動作として、次のように書くことができるかもしれません

def mod_sum(array, n)
  array.map { |x| x.modulo(n) }
       .sum
end

puts mod_sum([3, 3, 2], 2) # => 2

なにか参考になれば幸いです。

Answer 3

自分が試した限りは、特に問題なく動作します（paiza.io）。

ということで、問題はメソッドを呼ぶ部分や書く位置、処理系側など、質問に書かれていない部分にあると思われます。

Comments

[ji46ksi]

ご回答ありがとうごさいます。CODEPREPという学習サイト上で書いたものなのでもしかしたらそちら側に問題があったのかもしれないです。