質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.03%

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

解決済

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 367

mifne

score 14

開発環境 

ruby version 2.3.3p222

ご用件

rubyで2つの整数を比較して大きい方を表示するプログラムをつくっています。

エラーの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で検索してみたけれども、ぜんぜんりかいすることができませんでした。すいません。

  1. 会社を辞めて在宅ワーカーになったパパの記録]
  2. リンク内容

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

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 4

checkベストアンサー

+3

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を入れた場合などに誤った結果となるでしょう

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/10 08:51

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

    キャンセル

+1

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 メソッドをつかって書き直してみてください。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/10 11:22

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

    キャンセル

  • 2019/02/10 11:34

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

    キャンセル

  • 2019/02/10 11:37

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

    キャンセル

+1

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

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

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/10 10:50

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

    キャンセル

  • 2019/02/10 11:14

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

    キャンセル

  • 2019/02/10 11:51

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

    キャンセル

+1

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/13 15:25

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

    キャンセル

  • 2019/02/13 18:39

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.03%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る