Warning: control reaches end of non-void function [-Wreturn-type]

Question

Warning: control reaches end of non-void function [-Wreturn-type]

Александр Кротов Ученик (187), на голосовании 2 месяца назад

Здравствуйте! Подскажите, пожалуйста, как убрать ошибку
warning: control reaches end of non-void function [-Wreturn-type]
18 | }
| ^
в этом коде:

 #include  
#include  
using namespace std; 
 
bool is_prime(unsigned long long n) 
{ 
    int k = 0; 
    for (int i = 2; i <= sqrt(n); ++i) 
    { 
        if (n % i == 0) 
        { 
            return false; 
            k += 1; 
            break; 
        } 
    } 
    if (!k) return true; 
} 
 
int main() 
{ 
    unsigned long long n; 
    int k = 0; 
    cin >> n; 
     
    if (is_prime(n)) cout << n; 
    else 
    { 
        for (int i = 2; i <= sqrt(n); ++i) 
    { 
        if (n % i == 0) 
        { 
            if (is_prime(i)) cout << i; 
            break; 
        } 
    } 
    } 
 
    return 0; 
}

Answer 1

if (n % i == 0)
{
return false; //здесь будет выход из функции
k += 1; //?
break; //?
}

Answer 2

У тебя после return false ничего работать не будет. Так как возвращается управление

Answer 3

Не думаю что проблема в return false. Ошибка указывает, что из функции is_prime нет однозначного выхода, из-за наличия условия "if (!k) return true" в самом конце функции.
Исправь это на "return !k" - и должно взлететь

P.s. А учитывая, что k - это int, а возвращаешь ты bool, то лучше "return k == 0" или что-то подходящее под логику функции, т.к. не особо вчитывался в алгоритм, иначе можно словить ошибку преобразования типов, если k будет больше одного (т.к. int в bool неявно может преобразоваться только в случае 0 - Ложь, 1 - Истина)