コマンドプロンプトで計算ゲームをしたいが、正解率の所でエラーになる件

Question

ご覧いただきありがとうございます。

現在、コマンドプロンプト（ターミナル）で計算ゲームをするプログラムを作っています（おもに練習用です）

まだrubyの勉強初めて一週間もたっていないので知識がばがばで恐縮です（少しだけjavascriptは触れていました）

問題を出題し、結果をreturnするところまでは出来てます。
各問題の配列をlengthで取得したものを変数に代入し、％で出す場合はいったん１００倍にする必要がある為

配列length*100/正解数
で出そうとしていますが、うまく実行できません。

また、正解数は、countで計算して出そうとしています。

意図したとおりに結果を出力するには、いったいどうしたらよろしいのでしょうか?

# coding: utf-8

x=rand(10)
y=rand(5)
z=(15)



puts "ready?"

def ques(a,b)
  return puts "#{a}*#{b}は？"
end
def ans(a,b)
  return a*b
end


def lops(a,b)
  puts ques(a,b)
  answer=gets.chomp.to_i

  if answer==ans(a,b)
    return puts "clear"
  else
    return puts "error!"
  end
end

#
#
#

def ques3(a,b,c)
  return puts "#{a}*#{b}+#{c}は？"
end
def ans3(a,b,c)
  return a*b+c
end


def lops3s(a,b,c)
  puts ques3(a,b,c)
  answer=gets.chomp.to_i

  if answer==ans3(a,b,c)
    puts "clear!"
  else
    puts "error!"
  end
end

$count
setready=gets.chomp

if setready=="yes"
  checks=1
  lopsblock=[lops(x,y),lops(y,z),lops(z,x),lops3s(x,y,z)]
  lopslength=lopsblock.length.to_f
  puts 
  lopsblock.each do|a|
    $count+=1 if a=="clear!"
    end
  end

  if checks==1
    total=lopslength*100/$count.to_f 
#変数と数字の掛算、割り算でエラーになる。
#また、lopslengthのlength取得もできていないため、正確な計算ができない


      puts "あなたの正解率は、#{total}%です。"
  end

Answer 1

# encoding: utf-8

x = rand(10)
y = rand(5)
z = 15

que_mode = [[x, y], [y, z], [z, x], [x, y, z]]

def make_question(mode)
  ans = []
  mode.each do |e|
    if e.size == 2
      ans << ["#{e[0]}*#{e[1]}は？", e[0] * e[1]]
    else
      ans <<  ["#{e[0]}*#{e[1]}+#{e[2]}は？", (e[0] * e[1]) + e[2]]
    end
  end
  return ans
end

count = 0
ques = make_question(que_mode)

puts 'ready?'
sel = gets.chomp
exit if sel != 'yes'
puts ''

ques.each do |que|
  puts que[0]
  input = gets.chomp.to_i
  if input == que[1]
    puts 'clear!'
    count += 1
  else
    puts 'error!'
  end
  puts '-' * 20
  puts ''
end

puts "あなたの正解率は、#{(count / ques.size.to_f) * 100}%です。"

作ってみました。
質問者様のコードでは$countがnilになっていました。
return puts "clear!"では"clear!"は返ってきていないようです。

#補足

# coding: utf-8

x=rand(10)
y=rand(5)
z=(15)



puts "ready?"

def ques(a,b)
  return puts "#{a}*#{b}は？"
end
def ans(a,b)
  return a*b
end


def lops(a,b)
  puts ques(a,b)
  answer=gets.chomp.to_i

  if answer==ans(a,b)
    puts "clear"
    return "clear!"
  else
    return puts "error!"
  end
end

#
#
#

def ques3(a,b,c)
  return puts "#{a}*#{b}+#{c}は？"
end
def ans3(a,b,c)
  return a*b+c
end


def lops3s(a,b,c)
  puts ques3(a,b,c)
  answer=gets.chomp.to_i

  if answer==ans3(a,b,c)
    puts "clear!"
    return "clear!"
  else
    puts "error!"
  end
end

count = 0
setready=gets.chomp

if setready=="yes"
  checks=1
  lopsblock=[lops(x,y),lops(y,z),lops(z,x),lops3s(x,y,z)]
  lopslength=lopsblock.length.to_f
  puts 
  lopsblock.each do|a|
    count+=1 if a=="clear!"
    end
  end

  if checks==1
    total = (count / lopslength.to_f) * 100


      puts "あなたの正解率は、#{total}%です。"
  end

これでいけると思います。
total=lopslength*100/$count.to_f
この計算式だと正解率が100%以外の場合に
「あなたの正解率は、133.33333333333334%です。」
みたいになります。（1問間違えの場合）
あとこの部分の

lopsblock.each do|a|
    count+=1 if a=="clear!"
    end
  end

1つめのendはlopsblock.eachのendで
2つめのendはif setready=="yes"のendです。
count+=1 if a=="clear!"と書いた場合はendはいりません。

Comments

[garchomp]

解答ありがとうございます！

def make...でifを使って実行される配列の内容によって形式を変えることですっきりできるんですね！勉強になりました！

メソッドの使い方がまだまだ甘いと思いやられました^^;

正解率の処理の所でsize.to_f)*100することもできるんですね。。。

また、処理をイメージして、記述の無駄を減らして行けるように頑張ります。
ありがとうございました！

[退会済みユーザー]

いえいえ。私も最近メソッドの勉強を始めたので、そこまでのものではないです。
参考にしてはいけませんよ。

正解率は恐らく
(正解した数/全体) * 100
で求められると思ったのでそのようにしました。

Rubyのスタイルガイドによるとlengthよりもsizeの方が好ましい
ということだったので私は最近sizeを使っています。

[garchomp]

そうなんですね^^;

まだ一週間もたっていない自分はすでに基本抑える所でいっぱいいっぱいです…^^;

自分も負けじと頑張っていきたいですw

[garchomp]

補足ありがとうございます！

なるほど…to_fは末尾に付けるだけでなく、必要な要素それぞれにつけることで対処できると…φ(..)メモメモ

[退会済みユーザー]

その部分はミスです。メモは消しておいてください(; ･`д･´)

[garchomp]

おっと・・・了解しました！
そういえば色々な省略記法もありましたね…慣れていないのでだらだらと書いてしまいました^^;

とりあえず基本をしっかり押さえつつ、スマートに書けるように頑張りますw