最高点、最低点、平均点など

Question

データファイルのファイル名を，コマンドライン引数から受け取る。
合計点を記録するための変数（たとえば sum） をゼロに設定する．
最高点，最低点を記録するための変数（たとえば max, min ） を適切な値に設定する．
データ行の数を数えるカウンタをゼロに設定する。
ファイルをオープンする。
ファイルを1行ずつ読み込むためのループを作る。
1行読んでは，カンマで区切られたデータを分離し， 英数国3教科の点数の和（このための変数には，上述のsum は当然使えない．適当に短い変数名を使えばよい）を求める。
その点数と最高点を比較して，必要な処理を行う。 最低点についても同様。
3教科の点数の和を合計点に加算する．
カウンタを増やす。
ループが終了したら，平均を計算する。
実行例に従って，平均は小数第2位(ここでは printf を使う必要があることに注意！)まで，最高点と最低点は整数値で出力する。 行末には改行を入れ

この流れでプログラムを書いたつもりだったのですが、
うまく走りません。
どこが違うのでしょうか。

datafile = ARGV[0]
sum = 0
max = 100
min = 1000
count = 0
File.open(datafile) do |fp|
#while line = fp.gets
fp.each_line do |line|
line = line.chomp!
c = line.split(",")
goukei = c[2].to_f + c[3].to_f + c[4].to_f
if max <= goukei then
max = goukei
end
if min >= goukei then
min = goukei
end
sum += goukei
count += 1
end
#end
end
#p sum
#p count
average = sum /count
printf("平均点 %3.2f\n",average)
printf("最低点 %3d\n",max)
printf("最低点 %3d\n",min)

Comments

[sokha]

自分で自分の質問を見た時にわかると思われますが、ソースをきちんと表示するマークダウンがありますので、それを使って綺麗にソースを置きましょう。
また、どこで、どういうエラーが起きたのか詳しく書きましょう。

[otn]

コード部分を```Ruby と```という行で囲んでください。あと、インデント（字下げ）を行ってください。

Answer 1

動作するコードを書いてみました。
質問文のコードとの差を研究してみていただければ幸いです。

datafile = ARGV[0]
sum = 0
max = -1      # 可能性がある最高点より小さい値を設定する
min = 100 * 4 # 可能性がある最低点より大きい値を設定する
count = 0

File.open(datafile) do |f|
  f.each_line do |line|
    next if line.chop == '' # 空行は無視する

    items = line.split(',').map(&:to_i)
    fail "データ不正：#{line}" if items.count != 3  # 3つの点数が記載させていなければエラー

    goukei = items.inject(0) { |a, e| a + e }
    max = goukei if max < goukei
    min = goukei if min > goukei
    sum += goukei
    count += 1
  end
end

puts "  sum = #{count}"
puts "count = #{count}"

sum = min = max = 0 if count == 0
puts "平均点 #{format('%3.2f', sum / count)}" if count > 0
puts "最低点 #{format('%3d', max)}"
puts "最低点 #{format('%3d', min)}"

実行例

$ cat data.txt
100, 90, 80
30,  40, 50

$ ruby aa.rb
  sum = 2
count = 2
平均点 195.00
最低点 270
最低点 120

$ cat  empty.txt

$ ruby aa.rb
  sum = 0
count = 0
最低点   0
最低点   0

items = line.chop.split(',').map(&:to_i)
goukei = items.inject(0) { |a, e| a + e }

が、よくわからなければ、つぎのように書いても同じです。

line.chop
items = line.split(',')
goukei = items[0].to_i + items[1].to_i + items[2].to_i

Comments

[peke]

ありがとうございました！！

Answer 2

最初の所で、
max = 0ですね。なぜ100を代入したのでしょうか？
ロジックを振り返って考えると、初期値として、maxにはあり得る最大値以下の値、minにはあり得る最小値以上の値を入れる必要があります。

今回は全員が全教科0点だと三教科合計最大値が0ということがあり得るので、0以下の値を入れます。
minには、全員が全教科100点だと三教科合計最小値が300という事があり得るので、300以上の値を入れます。
一般的には、「この場合はこれで十分」といちいち判断しないなら、max = -Float::INFINITY、min = Float::INFINITYとします。

あと、
line = line.chomp!は、
line.chomp!もしくはline = line.chompと書くべきです。
普通は、それらをどちらも書かずに、
c = line.chomp.split(",")
でしょうか。