Rubyで文の中から最も長い単語を返すメソッドを表すプログラミング例がよく分からないです。

Question

プログラミング初心者です。
文の中から最も長い単語を返すメソッドを表すプログラミングを書けという課題で、それの解答例の意味がよくわかりません。ご教授願います。

課題の原文は
Write a method that takes in a string. Return the longest word in the string. You may assume that the string contains only letters and spaces.
You may use the String split method to aid you in your quest.
です。

解答例は

def longest_word(sentence) #longest_word(sentence)というメソッドを作る
words = sentence.split(" ") #文を配列のように単語単語に分ける

longest_word = nil #最も長い単語がない(つまり文がない)時にnilを返す

word_idx = 0 #１つ目の単語を０個目とする？?
while word_idx < words.length #ある番目の単語＜単語の文字数？？の間繰り返す
current_word = words[word_idx] #現在の単語にある番目の単語を代入する

if longest_word == nil                                        #もし最も長い単語ならnilを返して、下のコードを実行？？
  longest_word = current_word                           #最も長い単語として現在の単語を代入する
elsif longest_word.length < current_word.length   #もし上のif文の条件ではなく、最も長い単語の長さ＜現在の単語の長さの場合、下のコードを実行
  longest_word = current_word                           #最も長い単語として現在の単語を代入
end                          #if文を終わる

word_idx += 1        #次の単語を見る

end #while文の終わり

return longest_word #longest_wordを返す
end #終わり

です。

自分なりの解説を解答のコードの右に#として書いてみました。正直曖昧ですが、？？が付いているところが分からなかったところです。

？？の部分をご教授願いたいです。宜しくお願いします。

最後にteratailで初めて質問させていただいたので、質問の仕方で改善するべき点があれば是非いってください。

Answer 1

質問文にあるコードは ruby らしさ、単純さの点ではあまり良いコードとはおもえません。
でも、質問文にあるコードも理解ができるようになっておくことは必要です。

?? の部分は次のように考えると良いとおもいます。

word_idx = 0 #１つ目の単語を０個目とする？?
==>
このプログラムでは word_idx を 0, 1, ... words.length - 1 と変化させて wowd をすべてスキャンしています。
ここでは while ループの入るまえに、word_idx の初期値を設定しています。

while word_idx < words.length #ある番目の単語＜単語の文字数？？の間繰り返す
==> words.length は 調べるべき単語の総数です。
word_idx < words.length の間、処理を繰り返します。
while 中では word_idx 番目の単語について処理をします。

if longest_word == nil #もし最も長い単語ならnilを返して、下のコードを実行？？
==> while ループに入る前に longest_word = nil としています。
while ループなかで logest_word.length を参照する必要ができますが、
longest_word == nil の状態だと、logest_word.length という参照はできません。
longest_word != nil のときにだけ logest_word.length をつかった操作をするようにしています。

Comments

[H-JURI]

回答ありがとうございます。
２個目の？？の.lengthメソッドは文字数ではなくて単語数なのですね。混乱が解けました！

そしてこの問題と解答に関してですが、プログラミング初めての人用に作られているものでRubyをベースに最低限プログラマーとして知っておいて欲しい内容も同時に学ばせたいという思惑もあるのでは、認識しています。
正直まだRubyらしさ、というものがわかりませんが、これからしっかり分かるよう勉強していきたいと思います。

Answer 2

word_idx = 0はコメントの通りです。単語の配列(words)は0番目から始まるので、最初に0にしています。
while word_idx < words.lengthは配列のインデックスを一つ一つ増やしていった(word_idx += 1の部分)ときに、単語の配列の長さを超えない限り繰り返すという意味です。
if longest_word == nilの部分は、もし、まだ一つも調べていないか(つまり、一番長い単語が無い状態)を確認しています。つまりこれは、一番最初になるわけで、最初の単語は調べてきたうちの一番長い単語になるから、logest_wordへの代入を行うということの条件分岐です。

で、解答例の解説はいいのですが、解答例が悪すぎます(もっと言うと問題も悪すぎます)。解答のメソッドは1行で終わります。

def longest_word(sentence)
  sentence.split.max_by{|word| word.length}
end

問題点をいくつか述べていきます。

1. 問題文の不備

「一番長い単語」とありますが、それが複数あった場合を想定していません。
2. splitの引数は不要
splitのデフォルトでスペース区切りです。" "は不要です。
Ruby:String#slpit
3. max_byを使わない理由が無い
ArrayがMix-inしているEnumerableにはmax_byという最大を求めるのに便利なメソッドがあります。
Ruby:Enumerable#max_by
上のコードでは配列にある単語の長さ(word.length)同士で比較し、最大になる要素を返すという意味です(最大が複数あった場合、どれが返るかは不定です)。

他にも、0からwords.lengthまで繰り返すなら、words.length.timesでいいだろうとか、そもそも、配列をeachすればいいだろうとか、まったくもってRubyらしさも無い、どっか他の言語でもやりたいんですか？という解答例です。もし、Rubyをしたくて勉強しているのであれば、その問題と解答例が載っているところは参考にしないことをお勧めします。

Comments

[H-JURI]

回答ありがとうございます。
3個目の？？の意味がわかった気がします！助かりました。
問題の不備もなるほど、と思いました。

そしてこの問題と解答に関してですが、プログラミング初めての人用に作られているものでRubyをベースに最低限プログラマーとして知っておいて欲しい内容も同時に学ばせたいという思惑もあるのでは、認識しています。
正直Rubyもまだまだ途中なのでこれから精進させていただきます。

Answer 3

私なりの理解でコメントを入れてみました

def longest_word(sentence)
  words = sentence.split(" ") # 文字列をスペースを区切り文字として配列に変換
  # (スペースが対象の場合は、単にsplit としても可)

  longest_word = nil # とりあえず初期化(ループ処理開始のためのnil)

  word_idx = 0 # 配列走査インデックスの初期化
  while word_idx < words.length # words 配列の終端まで(コードブロック内の処理を)繰り返す
    current_word = words[word_idx] # インデックス値を利用して値を取り出し、current_word を更新

    if longest_word == nil
      longest_word = current_word # longest_word がnil なので現在の値を代入
    elsif longest_word.length < current_word.length
      longest_word = current_word # 現在の値のほうがより長いので値を更新
    end

    # 条件に応じた値を返す形式の書き方でも可(longest_word に値が代入される)
    # longest_word =
    #   if longest_word.nil?
    #     current_word
    #   elsif longest_word.length < current_word.length
    #     current_word
    #   else
    #     longest_word
    #   end

    word_idx += 1 # インデックス値をインクリメント
  end

  return longest_word # 最長単語を返す
  # (単にlongest_word と記述するだけでも可)
end

---

似たような動作をする関数として、longest_word2 を定義してみました:

def longest_word2(sentence)
  desc_sorted = sentence.split.sort_by { |word| word.length }.reverse
  # 単語の長さで昇順に並べ替えて、そのあとで逆順にする(=降順になる)

  longest_word_length = desc_sorted.first.length
  # 並べ替えた配列のアタマの単語の長さを取り出す

  desc_sorted.find_all { |word| word.length.eql?(longest_word_length) }
  # 最長単語と、それと同じ長さの単語を配列で返す
end

---

実行結果：

target = <<EOF
Write a method that takes in a string. Return the longest word in the string. You may assume that the string contains only letters and spaces. 
You may use the String split method to aid you in your quest.
EOF

p longest_word(target)   # => "contains"
p longest_word2(target) # => ["contains"]

何か参考になれば幸いです

Links

• Amazon.co.jp： プログラミングRuby 第2版 言語編: Dave Thomas, Chad Fowler, Andy Hunt, まつもと ゆきひろ, 田和 勝: 本
• Amazon.co.jp： プログラミングRuby 第2版 ライブラリ編: Dave Thomas, Chad Fowler, Andy Hunt, まつもと ゆきひろ, 田和 勝: 本
• Amazon.co.jp: たのしいRuby第4版 電子書籍: 高橋 征義, 後藤 裕蔵, まつもと ゆきひろ: Kindleストア
• ドキュメント
• オブジェクト指向スクリプト言語 Ruby リファレンスマニュアル (Ruby 2.2.0)

Comments

[H-JURI]

ご回答ありがとうございます。
==nilで初期化、word[word_idx]がインデックス値で値を取り出す、等理解しやすかったです！
さらにlongest_word2として新しいものの定義までしていただき大変ありがたいです。参考にさせていただきます！
Rubyの学習もまだまだ最初なのでこれから頑張らさせてもらいます。