IsPrime? - missing return statement

Question

Я получаю сообщение об ошибке отсутствия (32) при компиляции, и я не уверен, почему. Какие-нибудь советы?

Я думал, что статус возврата был моим заявлением о возврате.

import java.util.Scanner; 

public class PrimeChecker 
{ 
    public static void main(String[] args) 
    { 
    int number; 
    Scanner keyboard = new Scanner(System.in); 

    System.out.print("Enter a number: "); 
    number = keyboard.nextInt(); 

    if (isPrime(number)) 
     System.out.println("Your number is not prime."); 
    else 
     System.out.println("Your number is prime."); 
    } 

    public static boolean isPrime(int number) 
    {  

    for (int i = 2; i < number; i++) 
    { 
     boolean status; 

     if (number % i == 0) 
      status = true; 
     else 
      status = false; 
     return status; 
    } 
    } 

} 

Answer 1

Добавить ложное возвращение в конце. Потому что, если ваш number меньше 2 для блока не будет работать, так что вы должны иметь оператор возврата из ряда для блока

public static boolean isPrime(int number) 
{  

for (int i = 2; i < number; i++) 
{ 
    boolean status; 

    if (number % i == 0) 
     status = true; 
    else 
     status = false; 
    return status; 
} 
if(number==2){ 
    return true; 
} 
return false 
} 

Answer 2

Вы должны добавить оператор возврата после цикла:

public static boolean isPrime(int number) 
    {  

    for (int i = 2; i < number; i++) 
    { 
     boolean status; 

     if (number % i == 0) 
      status = true; 
     else 
      status = false; 
     return status; 
    } 
    return false; 
    } 

Поскольку метод должен возвращать значение, если ваш цикл никогда не выполняется.

Но ваш цикл никогда не заканчивался после первого значения, потому что вы возвращаетесь в каждом случае.

Answer 3

Вы должны поставить инструкцию return в конце функции. Вы добавили возврат в for loop.

public static boolean isPrime(int number) { 
    for (int i = 2; i < number; i++){ 
     boolean status; 

     if (number % i == 0) 
      status = true; 
     else 
      status = false; 
     return status; 
    } 
    return false; 
} 

Answer 4

Во-первых, эта функция не является простым контролером числа: алгоритм будет закончен после первой итерации. Эта функция будет простой проверки числа, если и только если i==number&&number%i==0. Извлечь status переменную от цикла и поместить ее в конце функции. Удерживайте в памяти: полное сканирование числа неэффективно.

Answer 5

Как уже указывалось, причина ошибки компиляции заключается в том, что выполнение кода может завершиться до конца функции, где нет оператора возврата.

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

public static boolean isPrime(int number) 
{ 
    if (number < 2) return false; 

    for (int i = 2; i < number; i++) 
    { 
     if (number % i == 0) return false; 
    } 
    return true; 
}