Все простые числа между двумя разобранными номерами пользователей

Question

У меня есть вопрос о проблеме. Я новичок в этом, поэтому я ценю вашу помощь! Код должен запросить два пользовательских номера ввода и распечатать на консоли все простые числа между этими двумя. Вот код, который я не получил до сих пор: (. Это не проверяет число между этими двумя, только распечатав один номер, если это простое)

package questionsAndAnswers; 

import java.io.BufferedReader; 
import java.io.IOException; 
import java.io.InputStreamReader; 

public class Number { 
    static boolean primes; 
    public static boolean number() { 
     try { 
      for (int i = 0; i < 2; i++) { 
       BufferedReader br = new BufferedReader(new InputStreamReader System.in)); 
       String num1 = br.readLine(); 
       String num2 = br.readLine(); 

       int number1 = Integer.parseInt(num1); 
       int number2 = Integer.parseInt(num2); 

       for (int j = number1; j < number2; j++) { 
        if ((j % 2) == 0) return true; 
        for (int k = 3; (k*k) <= j; k+=2) { 
         if(j % k == 0) { 
          return false; 
         } 
         System.out.println("All the primes b/n number " + number1 + " and number" + number2 + " are :" + j); 
         return true; 
        } 
        System.out.println(primes); 
        // br.close(); 
       } 
      } 
     } catch (IOException e) { 
      e.printStackTrace(); 
     } 
     return primes; 
    } 
} 

... и в основном классе:

public class TwoWholeNumbers { 

    public static void main(String[] args) { 

     System.out.println("Enter two integer numbers to see what is the multitude b/w them: "); 
     Number.number(); 

    } 
} 

Answer 1

У вас есть скопированный код с какого-либо места. Проблема в том, что вы ломаетесь после первого значения.

import java.io.BufferedReader; 
import java.io.IOException; 
import java.io.InputStreamReader; 

public class Number { 
    static boolean primes; 

    public static boolean number() { 
     try {  
      StringBuffer buffer = new StringBuffer(); 
      for (int i = 0; i < 2; i++) { 

       BufferedReader br = new BufferedReader(new InputStreamReader(
          System.in)); 
       String num1 = br.readLine(); 
       String num2 = br.readLine(); 

       int number1 = Integer.parseInt(num1); 
       int number2 = Integer.parseInt(num2); 

       for (int j = number1; j < number2; j++) { 
        if ((j % 2) == 0) { 
         continue; 
        } 
        int x=0; 
        for (int k = 3; (k*k) <= j; k+=2) { 
         if(j % k == 0) { 
          x=1; 
          break; 
         } 
        } 
        if(x==1) continue; 
        buffer.append(j + " "); 
       } 
       System.out.println("All the primes b/n number " 
         + number1 + " and number" + number2 + " are :" 
         + buffer.toString());   
      } 
     } catch (IOException e) { 
       e.printStackTrace(); 
       } 
     return primes; 
    } 

    public static void main(String[] args) { 

     System.out.println("Enter two integer numbers to see what " 
          + "is the multitude b/w them: "); 
     Number.number(); 
     } 
    } 

Answer 2

Мой самый быстрый способ проверить, если число простое:

boolean isPrime(int p) { 
    int m = (int) Math.sqrt(p); 
    if(m<2) { 
    return true; 
    }else { 
    for(int i = 2;i++;i<=m) { 
     if((p % i)==0) { 
     return false; 
     } 
    } 
    } 
    return true; 
} 

Answer 3

Вы дали return в number() методе вместо break и continue. number() метод не вызывается в цикле в main(), который нужно возвращать для каждой итерации. Также возврат boolean просто не имеет смысла.

Я покажу вам несколько изменений, которые вам нужно сделать, что заставит программу распечатать несколько номеров. Но это не значит, что он правильный выход. Логика для простых чисел сильно ошибочна. Вы должны это исправить. Я просто помогаю вам решить основную проблему, почему вы не получаете какой-либо результат.

for (int j = number1; j < number2; j++) { 
    if ((j % 2) == 0) 
     continue; // return true replaced by continue 
    for (int k = 3; (k * k) <= j; k += 2) { 
     if (j % k == 0) { 
      break; // return false replaced by break 
     } 
     System.out.println("All the primes b/n number " 
       + number1 + " and number" + number2 + " are :" 
       + j); 
    } 
} 

Я дал break и continue избежать return заявлений, но на самом деле, логика действительно нуждается в пересмотре.