配列の中の2数が足して0になるものを判断するメゾットを作成したいです。

Question

前提・実現したいこと

n個の整数が配列arrとして与えられたとき，その中で，足して0となる2数があるかどうか判定し，答えるメソッドを作成したいです。

出力としては、足して0になる場合はtrue, そうでない場合はfalseとする。

発生している問題・エラーメッセージ

irb(main):021:0> has_duplicates? [9, 5, 2, -6, 2, -8, 1, 10, -5, -2]
true
true
true
true
true
true
true
true
true
true
=> [9, 5, 2, -6, 2, -8, 1, 10, -5, -2]
大量にtrueが出力してしまい、何をしたらいいのかよくわかりません。全体的に間違っているかもしれないので、ご協力のほどよろしくお願いします・

実際に出力させたいのは次のよう。
irb> has_duplicates? [9, 5, 2, -6, 2, -8, 1, 10, -5, -2]
=> true

該当のソースコード

def has_duplicates?(arr)
  arr.each_with_index do |i, j|
    if i = -j
      puts 'true'
    else
      puts 'false'
    end
  end
end

試したこと

なんどか試した結果ここまでたどり着きました。

補足情報（FW/ツールのバージョンなど）

Comments

[TaniguchiTakaki]

falseになる例もあったほうが意図がはっきりするかと思います。

[退会済みユーザー]

アドバイスありがとうございます。

Answer 1

まず、if i = -jは、iへの-jの代入ですので、常に真です。
=と==の区別がついていないとすると、プログラムを書くのは厳しいですね。

方針としては、
・１つでもそういうペアがあれば真
・全部探してそういうペアがなければ偽
という風に考えないといけません。

each_with_indexというメソッドも意味を誤解して使っています。
Arrayのメソッドの中から、2つずつの組み合わせを全部抜き出すようなメソッドを探して使ってください。

また、最後に?が付いた名前のメソッドは普通はtrueかfalseを返す意味を持っています。メソッド内部で表示はしません。

Answer 2

こんにちは

注意点は、他の回答者様が良いアドバイスをされているので、この回答では、まずコード例を挙げます。

def has_inverted_pair?(arr)
  arr.each_with_index do |e, i|
    return true if arr[i+1...arr.length].include?(-e)
  end
  false
end

p has_inverted_pair? [9, 5, 2, -6, 2, -8, 1, 10, -5, -2] # => true
p has_inverted_pair? [9, 5, 2, -6, 2, -8, 1, 10, -50, -20] # => false
p has_inverted_pair? [5, 0, 1, 2, 0, -3, -4]   # => true (0を２個含んでいる)

• 動作確認用Repl.it: https://repl.it/@jun68ykt/Q224458

次に、ご質問に挙げられているコードで気になったことを挙げさせて頂きますと、has_duplicates? というメソッド名です。これだと、 引数の配列が「同じ値の要素を重複して含んでいるか？」の判定のように思えてしまいますので、上記の回答では、「プラス・マイナスを反転(invert)させた要素のペアがあるかどうか？」がより分かるように has_inverted_pair? としました。これがベストな名前かは分かりませんが、メソッドや変数、クラスなどの名前にこだわることの重要性については、以下が参考になります。

• 参考: プログラマが知るべき97のこと - 名前重要 (Matzさん)

以上、参考になれば幸いです。

Comments

[退会済みユーザー]

初心者であり、学校でこのように指定するように言われたのでメゾット名に関してはよくわかりません。

[jun68ykt]

なるほどですね。

> 初心者であり、学校でこのように指定するように言われたので

ということであれば、今のところは

> メゾット名に関してはよくわかりません。

ということでよいかと思います。
ただ、もうひとつ細かいことを指摘させて頂くと、

> メゾット名に関してはよくわかりません。

とのことですが、

> メゾット

ではなく

メソッド

です。英語のつづりは、 "method" です。以下で発音を聴けます。

https://ejje.weblio.jp/content/method

Answer 3

プログラムの勉強、rubyの勉強が必要なのはお二人のご指摘の通りです。
が、
プログラムの勉強ってなにか具体的な対象がないとやりにくいもの。
それがこのパズルなのかな？と言うことで

module Enumerable　を調べるのが otn さんの回答への追加助言です

方針としては
「総当りでall? 若しくは any? を調べる」というのが一番わかりやすですが、falseの結果には n!回の繰り返しが必要になるので、実際のデータ数と相談です。例では10個なのでたった3628800回ですが20個になると、え〜〜相当な回数になります。垓には届きませんが京は越えます。
正負の二組に分けて大きさの順に並べて比べて行くのが回数はn回で済みますが、プログラムは厄介です

まずは総当りにトライして、数が増えたらどういうことになるのか見てみるのが面白いかも

Comments

[otn]

> module Enumerable　を調べるのが otn さんの回答への追加助言です
ああ、初心者の学習という面ではそちらの方が良かったですかね。

Arrayの組み合わせメソッドだと簡単なのですが、メソッド覚えるだけになってしまいますね。

[winterboum]

私の場合、rubyの展望が変わったのが３回ありまして、その最初のきっかけが inject だったのです。Arrayに無いじゃん！　ってEnumerableを教わっていろいろあるのに感激した覚えがあったので。。。。

[退会済みユーザー]

ありがとうございます。

Answer 4

each_with_index の使い方がおかしい、= の使い方がおかしい、真偽値を返すメソッドだと思われるのに puts を使っている点がおかしい、たとえそれらが正しかったとしてもおそらくロジックもおかしい（each_with_index の意味をどう勘違いしたとしても、まずこんなコードにはならないと思うので）。

全体的に間違ってます。
もっと簡単な問題からチェレンジすることをおすすめします。