Rubyでの素数判定プログラム

Question

前提・実現したいこと

プログラミングの勉強を始めて二日の者です。
rubyでprimeを使わず素数判定をするプログラムを書こうとしています。
正しく動かないので、私の理解できるように改善方法を提案していただきたいです。

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

ターミナルで実行したところどんな値を与えても「false」と返ってくるようになりました。

該当のソースコード

puts "値を入力"
i = gets.chomp.to_i
result = 0

if i = 1 || i = 0 || 
  judge = 1

else
  while result == 0
    (2..i-1).each do |num|
      result += i % num
    end
  end
end

if judge == 0
  puts "true"
else
  puts "false"
end

試したこと

知識が浅くネットで見つけたコードを見ても理解ができません。

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

win10でRuby 2.6.6p146を動かしています

###追記2

puts "値を入力"
i = gets.chomp.to_i

if i == 0 || i == 1
  puts "false"

elsif i == 2
  puts "true"

else
  (2..i - 1).each do |num|
    if 0 == i % num
      puts "false"
      break

    elsif num == i - 1
      puts "true"
      break

    else
    end
  end
end

追記

puts "値を入力"
i = gets.chomp.to_i

if i == 0 || i == 1 || i == 2
  puts "false"

else
  (2..i - 1).each do |num|
    if 0 == i % num
      puts "false"
      break

    else
      puts "true"
      break
    end
  end
end

Answer 1

if i = 1 || i = 0 || i = 2

iと1を比較するならi == 1です。i = 1なら代入です。

while result == 0

(2..i-1).each do |num|
  result += i % num
end

end

何をしたいのは不明です。judgeは設定しなくて良いのでしょうk？

#回答例
修正が完了したようなので、回答例を載せておきます。
その1：元の形を残した物。

def is_prime?(i)
    if i < 2
        return false
    else
        (2...i).each do |num|
            if i % num == 0
                return false
            end
        end
        return true
    end
end

print  "値を入力? "
i = gets.chomp.to_i
puts is_prime?(i)

その2：便利なメソッドを使った物。

def is_prime?(i)
    (2...i).all?{ |n| i % n != 0 }
end

print  "値を入力? "
i = gets.chomp.to_i
puts is_prime?(i)

除数はi未満でなくiの平方根未満で良いとか、2以外の偶数で割らなくても良いとか、速度向上策はいろいろありますが、それはRubyの話じゃなくて数学の話なので、割愛します。

Comments

[退会済みユーザー]

== に修正しました、ありがとうございます。

resultが1でなくなった時にjudgeに1を加えるにはどう書き換えればいいですかね？

[otn]

> resultが1でなくなった時にjudgeに1を加えるにはどう書き換えればいいですかね？

それ自体意味不明ですが、上記日本語をRubyで書くと、
if result != 1
judge = 1
end

まずは、素数をどうやって判定するか、日本語で書いてみましょう。
それが正しく書けないとプログラムも書けません。

[退会済みユーザー]

すみませんでした。

入力された値が0,1,2では無い場合に、確実に割れる1、またその値を除いた｢2..i-1｣でループ処理
｢2..i-1｣のそれぞれの数で入力された値を割り、余りが出たらresultに余りを足し｢2.i-1｣の処理そのものを終わらせる(つまり2から約数未満の数でiを割り、1つでも割れるものがあれば割るのを止める)
最後までループ処理が終わらなかったiは素数であり、judgeは0のままで最後の分岐処理でtrueが出力される

という考えでした。

[退会済みユーザー]

2から約数未満は、2からi未満の誤字です

[otn]

> 2から約数未満の数でiを割り、1つでも割れるものがあれば割るのを止める
> 最後までループ処理が終わらなかったiは素数であり、
だけでいいです。

「余りが出たらresultに余りを足し」が無意味です。

[退会済みユーザー]

説明ありがとうございます。
実行し正しく動作したコードを追記しました。

もしよろしければ、～～するclassを作るなどの、基礎的な文法だけでこのプログラムを効率的に書く方法を教えていただけないでしょうか。

[otn]

> 実行し正しく動作したコードを追記しました。
惜しいです。10までの数値を全部確認しましたか？

[退会済みユーザー]

確認したら1以外の奇数のべき数はtrueになるようでした。
まったく原因がわからないので教えて頂きたいです。

[otn]

今のプログラムは、
「2で割り切れたら素数でない。2で割り切れなかったら素数」
です。

[退会済みユーザー]

なるほど、2で割り切れなかった時点でbreakしてしまっていたから、breakするのを計算が終わってからにする必要があったんですね。
追記しましたがこれで正しいでしょうか？

[otn]

はい。2は素数であるのに素数で無いと判断しているという点を除いては正しい動作だと思います。

あとで、改善例を載せておきます。

[退会済みユーザー]

なぜか素数でないと思い違いをしていました。
親切に回答して頂きありがとうございました。

[otn]

これだけのヒントで正常プログラムに到達できたと言うことは、最初から真剣に考えていれば自力で到達できていた気がします。

[退会済みユーザー]

勿論真剣に考えたり、資料を探したりしていたのですが、コードを見てどこが間違いなのかを考察する力が足りないのだと思います。
また、質問サイトでの苦い経験から一つの回答で限界まで自力で進むよう気を付けているのもあると思います。