bool関数でエラー "制御が非空白関数の終わりに達しました"

Question

他の関連する質問は見ましたが、それは全く意味がありません xcodeを使用しているC++クラスのコンソールプロジェクトを作成しています。

プログラムの目標は、数値が素数かどうかを評価することです。 （私の機能の論理がオフになっていることは分かりますが、今はちょうどいいアイデアです） 誰かが間違っていることを教えてくれますか？私はクラスノートを正確に追いかけていますが、私自身の工数である数学部分を除いています。

// Assignment 6 - Prime Number 
// 9-28-17 

#include <iostream> 
#include <cmath> 
using namespace std; 

bool primeCheck(int); //function prototype 

int main() 
{ 
int num;  // declares number for input 

cout << "Enter an integer to see if it is prime. "; 
cin >> num; 

while (num < 1) // validation loop 
{ 
cout << "Invalid. Enter an integer. "; 
cin >> num; 
} 

if (primeCheck(num)) 
    cout << num << " is not prime" << endl; 
else 
    cout << num << " is prime" << endl; 

    return 0; 
} 


bool primeCheck(int val) // function to check for primality 
{ 
    bool status; 

    if ((val %2 != 0) || (val %3 != 0) || (val %5 != 0) 
     || (val %7 != 0) || ((sqrt(val)) != 0)) 
    status = false; 
    else 
    status = false; 
} 

Answer 1

あなたはprimeCheckから何も返されていません。単に結果をステータスに割り当てるだけでは不十分です。関数の最後にreturn statusと書く必要があります。そうすれば、何がprimeCheckを呼び出すかを正確に知ることができます。

bool primeCheck(int val) 
{ 
    bool status; 

    if (val %2 != 0 || val %3 != 0 || val %5 !=0 || (val %7 != 0 && (sqrt(val)) != 0)) 
     status = false; 
    else 
     status = false; 

    return status; 
} 

また、あなたは間違いを犯した可能性があります。とにかく、改善するためにできることがいくつかあります。

1. あなたのすべての枝（ifとelse）falsestatusを割り当てます。おそらく最初のブランチ（ifがtrueの場合）は、ステータスにtrueを割り当てる必要がありますか？

2. 通常、中括弧はifとelseのステートメントで常に使用することをお勧めします。これが問題のブランチに属するどのようなコード、それは明らかに：

   if (foo) 
       doSomething(); 
       doSomethingElse(); 
   

   は、実際として、コンパイラによって解釈される：

   if (foo){ 
       doSomething(); 
   } 
   doSomethingElse(); 
   

3. あなたが実際にbool statusを宣言するか、trueを割り当てる必要はありませんかそれにfalse。単に結果を直接返すことができます。これはちょうどいくつかの人々のためのスタイルの問題です。念頭に置いて

、ここにあなたの改良版は、関数である：

bool primeCheck(int val) 
{ 
    if (val %2 != 0 || val %3 != 0 || val %5 !=0 || (val %7 != 0 && (sqrt(val)) != 0)){ 
     return true; 
    } else { 
     return false; 
    } 
}