プログラミング超初心者の質問です。

Question

問題の解答例を実行するとエラーが出ることと、解答例で意味がよくわからないところがあるのでご教授願いたいです。

まず問題です。

# Write a method that takes an array of numbers in. Your method should
# return the third greatest number in the array. You may assume that
# the array has at least three numbers in it.

そして、それの解答例です。

def third_greatest(nums)
 first = nil
 second = nil
 third = nil
 
 idx = 0
 while idx < nums.length
   value = nums[idx]
   if first == nil || value > first
     third = second
     second = first
     first = value
   elsif second == nil || value > second
     third = second
     second = value
   elsif third == nil || value > third
     third = value
   end
   
   idx += 1
 end
 
 return third
end

これを使って以下のことを実行するとコードが間違いがなければ、３つ"true"と返ってきます。

# These are tests to check that your code is working. After writing
# your solution, they should all print true.

puts(
  'third_greatest([5, 3, 7]) == 3: ' +
  (third_greatest([5, 3, 7]) == 3).to_s
)
puts(
  'third_greatest([5, 3, 7, 4]) == 4: ' +
  (third_greatest([5, 3, 7, 4]) == 4).to_s
)
puts(
  'third_greatest([2, 3, 7, 4]) == 3: ' +
  (third_greatest([2, 3, 7, 4]) == 3).to_s
)

しかしこれを実際実行すると、

10-third-greatest.rb:19: syntax error, unexpected keyword_elsif, expecting keyword_end
   elsif second == nil || value > second
          ^
10-third-greatest.rb:22: syntax error, unexpected keyword_elsif, expecting keyword_end
   elsif third == nil || value > third
          ^
10-third-greatest.rb:30: syntax error, unexpected keyword_end, expecting end-of-input
```‘
のようにエラーが出てきます。質問が３つあります。

1.エラーが出るのは解答例に不備があると思うのですが、見つけられませんでした。どうか不備を教えてください。

2.最初のコードの
```lang-言語名
first = nil
second = nil
third = nil

の意味がわかりません。ご教授願います。

3.if文

if first == nil || value > first
      third = second
      second = first
      first = value
    elsif second == nil || value > second
      third = second
      second = value
    elsif third == nil || value > third
      third = value
    end

がわかりません。一体何をしているのでしょうか。。。の

Comments

[genovese]

回答とは関係ありませんが、Rubyのプログラムについての質問なので、タグはRubyにしたほうが、Rubyに詳しい回答者の方々に回答していただきやすくなります。

Answer 1

タイプミスですね。ifが正しく認識されていません。前後の空白が全角だったりしませんか？
それ以外の原因かもしれませんが、とにかくifの周辺を見直してください。

しかし、そのエラーメッセージで、ifのところが原因だとは気づきにくいですね。

firstが今までに出てきた中で一番大きな数値、secondが今までに出てきた中で二番目に大きな数値、thirdが今までに出てきた中で三番目に大きな数値です。文字通りですね。
繰り返しを始めた最初では、まだ「今までに出てきた」がゼロ件なので、「今までに出てきた中で一番大きな数値」が存在しませんので、firstには代わりに、nil という値（別に数値で無ければ何でも良いのですが)をセットしておきます。
if文で、firstがその値のままであれば、繰り返しの初回だと分かるのでそのときはfirstにセットします。コンテストのトップバッターは暫定一位みたいな意味です。
second、thirdも同様。

firstが今までに出てきた中で一番大きな数値、secondが二番目、thirdが三番目。
になるように、しています。

なお、その問題に対しては、普通はこう書きます。

def third_greatest(nums)
   nums.sort[2]
end

Comments

[H-JURI]

回答ありがとうございます！
"if"のまわりを直してもエラーが以前なくならないので、また改めて質問させていただきます。
２、３の説明が一番わかりやすかったのでベストアンサーにさせていただきます。
なお、書いていただいた別解なのですが
```Ruby-言語名
def third_greatest(nums)
nums.sort(2)
end

puts(
'third_greatest([5, 3, 7]) == 3: ' +
(third_greatest([5, 3, 7]) == 3).to_s
)
puts(
'third_greatest([5, 3, 7, 4]) == 4: ' +
(third_greatest([5, 3, 7, 4]) == 4).to_s
)
puts(
'third_greatest([2, 3, 7, 4]) == 3: ' +
(third_greatest([2, 3, 7, 4]) == 3).to_s
)
```
のようにし、実行すると
```bash-言語名
$ ruby 10-2-third-greatest.rb
10-2-third-greatest.rb:3:in `sort': wrong number of arguments (1 for 0) (ArgumentError)
from 10-2-third-greatest.rb:3:in `third_greatest'
from 10-2-third-greatest.rb:8:in `<main>'
```
のようにエラーが返ってきます。これはどういうことなのでしょうか。。。

[otn]

> "if"のまわりを直してもエラーが以前なくならないので、
エラーになるコードをそのままコピペしてください。

> エラーが返ってきます。
タイプミスですね。よく見比べてください。
また、エラーメッセージの見方、Rubyリファレンスの見方も学んでください。
Rubyリファレンス： http://docs.ruby-lang.org/ja/2.2.0/doc/

Answer 2

調べてみましたが，Rubyでしょうか？

1. if使う場合，thenが必要です→複数の条件分岐

thenがないため，単一のifと解釈され，endが来るであろう場所にelsifがあるためエラーになっていると思われます．
2.「値が何も入っていない状態」です．
3.
i.渡された配列の1番目から順に値を取り出す
ii.取り出した値をfirstから順に比較し(その場所が空ならそこに入れる)，より小さな値を見つけたら，それ以下のランクの数値を1ランクずつ下げて入れる
例えばfirstに5，secondに3がある状態で7を取り出した場合，firstの5より大きいので，今ある5と3を1ランクずつ下げ，空いたfirstに7を入れます．

Comments

[H-JURI]

回答ありがとうございます。言語はRubyで間違いないです。次からは明記します。
"then"を入れてもエラーがなくならないので、再度改めて質問します！

Answer 3

使われているプログラミング言語はRubyですよね、ファイルの拡張子が .rb ですから。（使っているプログラミング言語の種類は明記してください。言語ごとに文法が異なりますので）

だとすれば、thenが抜けているように見えます。
下記コードの"THEN"のところを、"then"に変更すれば、動くと思います。

def third_greatest(nums)
 first = nil
 second = nil
 third = nil
 
 idx = 0
 while idx < nums.length
   value = nums[idx]
   if first == nil || value > first THEN
     third = second
     second = first
     first = value
   elsif second == nil || value > second THEN
     third = second
     second = value
   elsif third == nil || value > third THEN
     third = value
   end
   
   idx += 1
 end
 
 return third
end

回答例が不完全なのは、単なるミスか、回答者が文法エラーに気付くかどうかを試しているか、だと思います。

＝＝
この質問は、本来ご自分で解決すべきものです。

判らなければ、Rubyに関する本を読んでください。

Comments

[H-JURI]

回答ありがとうございます！
確かにこの質問は基本的なことで自分で解決すべきものだと認識していましたが、完全な理解ができなかったので質問させていただきました。
ご指摘いただいたように"then"をくわえさせてもらったのですがエラーがなくならないので、再度改めて質問させていただきます！

Answer 4

あれ、Rubyのif文のthenって省略可能ですよね？
実際、私の環境では正しくtrueが出力されてますし

Comments

[coco_bauer]

質問者が使っている環境では、syntax errorになっているので、最も考えられるのは then の省略に対応していないコンパイラが使われているのではないかと推測しました。私の回答は、この推測に基づくものです。

[H-JURI]

IDEはcloud9というものを使っております。コンパイラはbash？というもののようです。
次からはこれも明記しようと思います！

Answer 5

1.についてはどの言語を使用しているのかわからないので、わかりません。

2.については処理の中で使用する変数の初期化をしていると思います。
NULL値（空の値）を各変数にセットしています。

3.についてはこの関数に渡した引数を大きい順に並び替えています。
例えば「[5, 3, 7]」をこの関数に渡すと各変数に
first ：7
secoud：5
third ：3
がセットされ、
return third
としているので戻り値としてthirdの値（3）が返ってきます。
この関数は渡した引数の中で3番目に大きい値を返しています。

Comments

[H-JURI]

回答ありがとうございます！
ご指摘いただいたように次からは言語を明記しようと思います。