Rubyでハイアンド・ローのアルゴリズムについて

Question

Rubyでハイアンド・ローのアルゴリズムについて

プログラミングをはじめたばかりなのですが、
思い通りの結果が出力されなくて困っています。

入力される値

入力は以下のフォーマットで与えられます。

a b
n
A_1 B_1
...
A_n B_n

aとA_nを比べてaの方が大きいなら"High"
aとA_nを比べてaの方が小さいなら"Low"
aとA_nが同じだった場合、bとB_nを比べてbの方が大きいなら"High"
aとA_nが同じだった場合、bとB_nを比べてbの方が小さいなら"Low"

と表示したいです。

(例)
与えられた標準入力
1 2
3
2 3
1 4

欲しい結果
"Low"
"High"

該当のソースコード

a, b = gets.split( )

n = gets.to_i

for i in 1..n do
      A_i,B_i = gets.split ( )
    if a > A_i then
        puts "High"
    elsif a = A_i then
      elsif b < B_i
        puts "High"
    else puts"Low"
    end
             
                    
end

試したこと

A_i,B_i自体をputsで出力させると値は取れてくるのですが...

for の使い方が間違っているのでしょうか、
初歩的なところで申し訳ないですが、どうかよろしくお願いします。

以下のエラーが出ます。

Main.rb:10: warning: already initialized constant A_i
Main.rb:10: warning: previous definition of A_i was here
Main.rb:10: warning: already initialized constant B_i
Main.rb:10: warning: previous definition of B_i was here

Answer 1

ruby では 大文字の変数は 定数と扱います。
定数は一度しか値を設定できません。
参考

• ruby 定数

https://docs.ruby-lang.org/ja/latest/doc/spec=2fvariables.html#const

...
アルファベット大文字 ([A-Z]) で始まる識別子は定数です。 定数の定義 (と初期化) は代入によって行われますが、メソッドの 中では定義できません。一度定義された定数に再び代入を行おうと すると警告メッセージが出ます。
...

質問文に示されている"望んでいる出力" とは一致しませんが、
エラーが出ない様に、書き換えてみました。

lowhigh.rb

a, b = gets.split

n = gets.to_i

n.times do
  a_i, b_i = gets.split
  if a_i < a
    puts 'High'
  elsif a_i > a
    puts 'Low'
  else
    if b_i < b
      puts 'High'
    elsif b_i > b
      puts 'Low'
    end
  end
end

実行例：

$ ruby lowhigh.rb 
1 2
3 
2 3
Low
1 4
Low
1 2

Comments

[ffj]

コードまで書いていただいてありがとうございます。
参考にして自分でもう一度書いて見たいと思います。

Answer 2

Rubyでは大文字で始まる変数名は定数となり、再代入時に警告が出ます。
forによってA_i,B_iに繰り返し代入されるので当然警告が出ます。

    elsif a = A_i then
      elsif b < B_i

• a = A_i

比較は==を使います。=ではaにA_iが代入されます。

• Rubyではインデントは基本無視されます。

よって、これはa = A_iが偽の場合にb < B_iが比較されます。

Comments

[ffj]

ありがとうございます。
比較を==にするのを忘れていました...
基本的なところで申し訳ないです。

Answer 3

Rubyでは、英大文字で始まる変数名は、定数という意味になっていて、2回目以降の代入では警告が出ます。
エラーじゃ無いので、そのまま実行できますが。

あと、文字列を大小比較すると、辞書順的な比較になるので、数値として比較するならto_iしてください。

Comments

[ffj]

ありがとうございます！
変数名に大文字がよくなかったのですね。
なるほど、.to_iで数値にするのを忘れていました。ありがとうございます。

Answer 4

皆様のおかげで解決いたしました。
本当に感謝しております。

a, b = gets.split( )
n = gets.to_i
for i in 1..n do
      a_i,b_i = gets.split ( )
      
    if a.to_i > a_i.to_i then 
        puts "High"
    elsif a.to_i == a_i.to_i && b.to_i > b_i.to_i then
            puts "High"
        
    else puts "Low"
    
    end         
                    
end