Ruby 配列要素の逆順ソート（コード問題)について

Question

Rubyのeachメソッドを使って、配列を逆順にソートしたいです。
しかし下記のコードでは出力が上手くいきません。
その原因は何でしょうか？

def reverse_string(s)
    
    result = Array.new
    n = 1
    
    s.each do |i|
      result[s.length - 1] = i
      n += 1 
    end
    
    result
end

出力結果等は以下になります。

Your input ["h","e","l","l","o"]
Output ["h","e","l","l","o"]
Expected ["o","l","l","e","h"]

どうぞよろしくお願い致します。

Answer 1

Your input ["h","e","l","l","o"]

Expected ["o","l","l","e","h"]

はソートとは言いません。逆順に並び替えですね。

プログラム1行ずつの「意味」を、声に出して言ってみましょう。
気づくことがあるかと思います。

Comments

[ryouri1998]

アドバイスありがとうございます。
result[s.length - 1] = iを
result[s.length - n] = iに修正しました。

プログラムの意味をコメントに書いたのですが、
知識不足で原因が分かりません。
申し訳ありません。

```Ruby
def reverse_string(s)

result = Array.new
#result配列を作る
n = 1
#nに1を代入

s.each do |i|
result[s.length - n] = i
　　　　#配列resultに配列sを代入、この時に逆順並び替え
n += 1
　　　　#nをインクリメントする
end

result
　　　#resultを出力
end

```

[otn]

> #nに1を代入
全く無意味なコメントですね。

> #nをインクリメントする
これも全く無意味。

> #配列resultに配列sを代入、この時に逆順並び替え
これは行の説明では無い。

コメントが駄目なのは別として、nの意味を思い出したことで（それを期待して読み上げるように言いました）、プログラムは期待通り動くようになっていると思いますが、何か問題が発生していますか？

[ryouri1998]

コメントのコードの通りに修正したところ
それでも期待通りに動きません。
出力結果は以下です。

Your input　　["h","e","l","l","o"]
Output　　　　["h","e","l","l","o"]
Expected　　　["o","l","l","e","h"]

[otn]

表示するコードが書かれてませんが、そこがおかしいのでは？
p reverse_string(["h","e","l","l","o"])
してますか？

[ryouri1998]

表示するコードを書いたところ、

Wrong Answer　Runtime: 40 ms
Your input["h","e","l","l","o"]
stdout　　["o", "l", "l", "e", "h"]
Output　　["h","e","l","l","o"]
Expected　["o","l","l","e","h"]
となり、上手くいきませんでした。

Answer 2

Leetcodeの問題ならば
よく問題文(特に太字部分)やコード入力欄に入力されているコメントを読むとよいです。

Comments

[ryouri1998]

ありがとうございます。
問題文の太字を読み、解決いたしました。