rubyで関数中のブール関数の値を返り値に渡す方法を教えてください。

Question

実現したいこと

しりとりのコードを書いています。
配列の要素を先頭から順に2個ずつ取り出し、shiritori()に引数として与えて、もし1単語目の最後尾の文字と2単語目の先頭の文字を比べて一致しなければその値を返してプログラム終了にしたいです。
今回の例だと欲しい値は以下の文章が出てループを抜けたい。
「「xとa」が違うよ」

どのようにすれば解決するのか、あるいはもっと良い方法があるなら教えてください。
よろしくお願いいたします。

発生している問題・分からないこと

「「xとa」が違うよ」の時点でプログラムを強制終了したいのに、次の”abc”と”cc”を比較してしまいます。
出力結果は以下です。
["apple", "et", "tex", "abc", "dc"]
xとaが違うよ
cとdが違うよ

該当のソースコード

p shiritori_words = ["apple", "et", "tex", "abc", "dc"]

shiritori_end = false
def shiritori(words)
    word1, word2 = words
    if word1[-1] != word2[0]
        puts "#{word1[-1]}と#{word2[0]}が違うよ"
        shiritori_end = true
    end
end

shiritori_words.each_cons(2) do |two_words|
    shiritori(two_words)
    break if shiritori_end
end

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

shiritori()内のshiritori_endの値がtrueになる時、each文に戻ってくるとfalseになっています。
だからeach文のbreak ifが使えないみたいで、、

補足

特になし

Answer 1

shiritori_end ではなく shiritori() 関数の戻り値を使ってみてはどうでしょうか。

p shiritori_words = ["apple", "et", "tex", "abc", "dc"]

def shiritori(word1, word2)
    if word1[-1] != word2[0]
        puts "#{word1[-1]}と#{word2[0]}が違うよ"
        return true
    end
    return false
end

shiritori_words.each_cons(2) do |two_words|
    break if shiritori(*two_words)
end

Comments

[off]

回答ありがとうございます！
うまくいきました！

splat演算子の存在をggってきました。こんな方法があったのですね！
コードの記述量が減りすごく見やすくなりました！
ありがとうございました！

Answer 2

まず、基本文法。3行目のshiritori_endは、他の多くの言語ではグローバル変数っぽいのですが、Rubyの場合はローカル変数です。トップレベルのローカル変数というちょっとわかりにくい概念です。
参考: https://docs.ruby-lang.org/ja/latest/doc/spec=2fvariables.html#local
Rubyのメソッドの中から外のローカル変数はスコープ外です。
サンプル：

foo = 123
def try_change_foo
	foo = 789
end
try_change_foo
p foo #=> 123

次に、Rubyのメソッドは返り値を持ちます。これは、

関数中のブール関数の値を返り値に渡す方法を教えてください。

とお書きなので、「返り値」の概念はおわかりですね。
メソッドshiritoriが、「一致しなかったらtrueを、一致したらfalseを返り値として返す」という定義なら、
p shiritori("apple","et")で、falseが表示されて、
p shiritori("tex","abc")で、trueが表示されて、
14行目のbreakは、break if shiritori(two_words)と書けますね？

最低限の書き換えをすると（returnは所により必ずしも必要では無いのですが、「返り値を返す」を明記するために書いています）、

p shiritori_words = ["apple", "et", "tex", "abc", "dc"]

def shiritori(words)
    word1, word2 = words
    if word1[-1] != word2[0]
        puts "#{word1[-1]}と#{word2[0]}が違うよ"
        return true
    end
    return false
end

shiritori_words.each_cons(2) do |two_words|
    break if shiritori(two_words)
end

Ruby的には便利なメソッドを使ってもっとコンパクトに書けますが、プログラミングを始めたばかりの場合は、そういうのに頼らずに、このように自分でロジック判断を考えるのが良いと思います。

チェックするメソッドの中でメッセージまで出していますが、メソッドの機能は何なのか曖昧になるので、返り値を工夫して、メッセージを出すのはメイン処理側で行うように変更してみましょう。

Comments

[off]

回答ありがとうございます。
試したところ問題が解決しました！
ベストアンサーに選ばせていただきました。

グローバル変数だと思っていました汗
ご指摘ありがとうございます！

＞メッセージを出すのはメイン処理側で行うように変更してみましょう。
なるほどと思い、コードを改良しました。問題なく動きましたし、メソッドの内容が見やすくなりました。
こちらもありがとうございました！