rubyでの2つの整数を比較の仕方について

Question

開発環境　

ruby　version 2.3.3p222

ご用件

######rubyで２つの整数を比較して大きい方を表示するプログラムをつくっています。
####エラーのNilClassがなぜ出るのか、どうすれば改善するかわかりません。
どうかご教授ください。

発生している問題・エラーメッセージ

Main.rb:5:in `<main>': undefined method `chomp' for nil:NilClass (NoMethodError)と返ってきました。

該当のソースコード

i = 0
while i < 10 do
s = gets.chomp.split(" ")
	if s[0] > s[1]
		puts "#{s[0]}"
	
	elsif s[1] > s[0]
		puts "#{s[1]}"
	
	elsif s[1] == s[0]
	puts "eq"
	end
	i += 1
end

試したこと

googleで検索してみたけれども、ぜんぜんりかいすることができませんでした。すいません。
0.
会社を辞めて在宅ワーカーになったパパの記録]
0. リンク内容

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

undefined method `chomp'

とのことなので

問題はgetsがnilを返していることです。

Kernel.#getsを見ると

ファイルの終り(EOF)に到達した時、 nil を返します。

と書いてあります。

なので対策としては、

• getsをしたあとにnilチェックをしてからchomp.split(" ")をする
• 入力データの仕様を見直す

コード上では10件分のデータが書いてあるのを想定しているが、データが足りていない

---

puts "#{s[0]}"

単にputs s[0]でいいです。

if s[0] > s[1]

これが整数比較になるかは微妙です
文字列をsplitしただけなのでs[0] s[1]ともに文字列のままです。
あくまでs[0] s[1]が数値文字列であった場合
長さ順・辞書順による比較が行われ似たような結果が返りますが
先頭に0を入れた場合などに誤った結果となるでしょう

Comments

[mifne]

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

Answer 2

UFOではどうでしょう？

puts "2つの数字を空白で区切って入力してください"
10.times {
s0,s1=gets.split.map(&:to_i)
case (s0 <=> s1) <=> 0
when 0
    puts :eq
when 1
    puts s0
else
    puts s1
end
}

参考
https://qiita.com/jkr_2255/items/51ecffeaf62e15d9a782

Comments

[mifne]

わざわざご回答いただきありがとうございます。
このような書き方、短くて便利です。
わかりやすく書く方法が分かって良かったです。
ありがとうございます。

[DrqYuto]

競プロでよく使ってたので使えますよ

Answer 3

繰り返しの回数が10回と決まっているのなら、iを使わず10.times do 〜 endの方が簡潔です。

ただし、10回と決めうちするのが原因でgetsがnilを返す可能性があるのだとすると、

while line = gets
  s0, s1 = line.split.map.(&:to_i)
  if s0 > s1
    puts s0
  elsif s1 > s0
    puts s1
  elsif s1 == s0
    puts "eq"
  end
end

というように、getsにちゃんと入力がある間だけ、whileのボディを実行するように書き換えて見るのもいいかもしれません。

なお、line.splitすると、末尾の改行が取れるので（改行もwhitespaceの一種です）、chompする必要はありません。

また、配列内の文字列を一気に整数に変換するのなら、map(&:to_i)が便利です。

そして、その返り値である配列を変数sに代入して使うより、s0, s1 = 〜というように、多重代入して整数ごとに変数に代入した方が分かりやすいように思いました。

---

【追記】
超トリッキー版です。

while line = gets
  s0, s1 = line.split.map.(&:to_i)
  puts ["eq", s0, s1][s0<=>s1]
end

Comments

[mifne]

こんなにも丁寧にご回答頂きありがとうございます。
とてもあったかい気持ちになります。
知らないことが多いため、回答を参考にして作り直したいと思います。
どうもありがとうございます。

[kts_h]

どういたしまして。
ついでに、比較して表示する部分ですが、トリッキーな方法を思いついてしまいました。演算子「<=>」は、左が大きいと1、等しいと0、右が大きいと-1を返すので、「puts ["eq", s0, s1][s0<=>s1]」で表現できます。

[mifne]

面白い発想ですね。こんな記述法があるとは思いませんでした。
puts ["eq", s0, s1][s0<=>s1] 自分のコードにも実装させていただきます。
アドバイスありがとうございます。

Answer 4

2つの数字をいれていないときのチェックをしていないことが原因です。
つぎのようにチェック処理を追加してみてください。

i = 0
while i < 10 do
  s = gets.chomp.split(" ")
  if s.size < 2
    puts "2つの数字を空白で区切って入力してください"
    next
  end

  if s[0] > s[1]
    puts "#{s[0]}"
  elsif s[1] > s[0]
    puts "#{s[1]}"
  elsif s[1] == s[0]
    puts "eq"
  end
  i += 1
end

実行例

追記
コメントでのやりとりのことやエラーチェックを強化したコードを示します。
(数値であることのチェックの追加、数字として大小比較をする)

i = 0
while i < 10 do
  s = gets.chomp.split(" ")

  if s.size != 2
    puts '2つの数字を空白で区切って入力してください'
    next
  end
  unless s[0] =~ /^[0-9]+$/ && s[1] =~ /^[0-9]+$/
    puts '整数を入力してください'
    next
  end

  s[0] = s[0].to_i
  s[1] = s[1].to_i
  if s[1] == s[0]
    puts 'eq'
  else
    puts s.max
  end
  i += 1
end

s[0], s[1] のチェック、整数への変換はコードが重複しています。
map メソッドをつかうと短く書くことが可能です。興味があれば map メソッドをつかって書き直してみてください。

Comments

[mifne]

ご丁寧に回答いただきありがとうございます。
ですが実行してみても、改善されませんでした。
Main.rb:3:in `<main>': undefined method `chomp' for nil:NilClass (NoMethodError)
申し訳ないのですが、もう一度ご回答いただけませんか。

[katoy]

なにを入力したらエラーになったかを教えてください。

[mifne]

すいません。上のエラーは直りました。お騒がせしてすいません。
ですが、
90 100 と入力すると90 90と返ってきました。100とは返ってきません。
なぜだか教えてください。

[katoy]

別回答にあるように 文字列として比較をしているからです。
s0 = s[0].to_i と s1 = s[1].to_i と整数に変換してから s0, s1 を比較するようにしてみてください。

[mifne]

例を示して、教えてくれてありがとうございます。
すぐに、質問にもお答えいただいて、感無量です。
とてもわかり易かったです。

[katoy]

整数に変換する処理を追加したコードを回答に追記しました。

[mifne]

コードの改定ありがとうございます！