Запуск ошибки Ява затмение

Question

Когда я запускаю этот код:

package jmv; 

public class euler3 { 
    static int x=1; 
    static long z = 600851475143L; 
    public static void main(String[] args) { 

     System.out.println(isPrime(123454321)); 
    } 

    public static boolean isPrime(int p){ 
     for(int y=1 ; y<x ;){ 
      if(x%y == 0){ 
       return true; 
      }else{ 
       return false; 
      }   
     } 
     return false; 
    } 
} 

Он всегда возвращает ложь. Зачем? Благодарим вас за помощь. Я очень appereciate каждый ответ, который я даю, но я лучше других.

Answer 1

Факт вас НЕ проверяют p, параметр, заданный функции isPrime().

Это должно быть так:

package jmv; 

public class euler3 { 
    static int x=1; 
    static long z = 600851475143L; 
    public static void main(String[] args) { 
     System.out.println(isPrime(123454321)); 
    } 

    public static boolean isPrime(int p){ 
     if (p%2 == 0) { 
      return false; 
     } else { 
      for(int y=3 ; y<p ; y=y+2){ 
       if(p%y == 0){ 
        return false; 
       }   
      } 
     } 
     return true; 
    } 
} 

Вы должны проверить, если 2 делит р первый, потому что четное число может быть передан в функцию. Тогда вам просто нужно проверить нечетное число> = 3. Но это основная математика, и я уверен, что вы просто забыли ее.

Выполняя это, вы сохраняете множество тестов. Если вы проверите с y ++, вы будете проверять каждый четный номер, и это необязательно, поскольку если число четное и больше 2, оно не является простым.

Надеюсь, это поможет!

Answer 2

for(int y=1 ; y<x ;){<-- loop never gets executed as 1<1 will be always false 

Более того, вы не увеличивающиеся y изменить его на for(int y=1 ; y<x ;y++) и присвоить определенное значение для x сказать x=10 должен быть >y.

И да, поскольку вы передаете параметр p в качестве параметра, вы должны захотеть сделать for(int y=1 ; y<p ;y++) или что-то еще. (УказаноDjon).

Кроме того, если вы пытаетесь найти число, простое или нет, вы должны идти по другой логике, поскольку ваша логика неверна.

Answer 3

Потому что ваша петля никогда не выполняется. y=1 никогда не ниже, чем x=1.

Answer 4

Для цикла не выполняется никогда, потому что 1<1 всегда false, поэтому он будет возвращать false

вы можете определить, простое число так просто, как это:

public static boolean isPrime(int p){ 
     for(int y = 2 ; y < p ; y++){ 
      if(p % y == 0){ 
       return false; 
      } 
     } 
     return true; 
    }