CまたはC++戻り状況

Question

ステータスコードを表すintを返すCまたはC++関数を記述するためのベストプラクティスは何ですか？

具体的には、クライアントの使用方法について知りたいが、他のヒントを歓迎します。例えば

は、私はこのような何か書くことができます。

int foo() { 
    return 0; // because everything was cool 
} 

をし、このようにそれを使うのか？

if (foo()) { 
    // what to do if false, e.g. non-zero, e.g. not OK 
} else { 
    // what to do if true, e.g. zero, e.g. OK 
} 

のベストプラクティスは、一般的に0のステータスコードはすべてがOKだったとも0はブール文でfalseを意味を意味することを指示するので、これは動作するはずです。

しかし、これは正しい、良いではないでしょう。

if (!foo()) { 
    // what to do if true 
} else { 
    // what to do if false 
} 

Answer 1

は私が働く場所gccはそれを嫌う）。

C++の場合、例外がある場合はそれを使用します（そうでない場合は上記を参照）。

編集： 成功した場合は0を返し、それ以外の場合はエラーを返すことをおすすめします。これは、UNIXのコマンドラインユーティリティが行うものです。

Answer 2

リターンステータスは、あなたのインターフェイスで定義され、呼び出し側に知られている必要があります。エラーが発生した場合は0を返します（!で確認するのは簡単です）。成功した場合は0が返されます（enumのエラーコードがあり、OKが最初の項目です）。

法律や標準はありません。各インターフェイスには独自の規則が定義されています。 C++では - 例外を使用します。

int err = foo(); 
if (err) { 
    // armageddon 
} 

割り当てと組み合わせることができれば、より多くの複雑な機能とそれがさらに混乱取得し、一部の人々は、条件付きで割り当てによって混乱している呼び出します（と：私たちはCでこれを使用

Answer 3

int foo() { 
    try{ 
    ... 
    return 1 
    } 
    catch 
    { 
    return 0; // because everything was cool 
    } 
} 

try/catchブロックですべてをラップすることから始めます。また、intを使用する代わりに、より多くのシーンを作成してブール値を返すこともできます。これは、ifステートメントでテストするときには少し直感的です。

Answer 4

ベストプラクティスは、自分自身と他の人がエラーチェックの際に返されるコードをすばやく検索できるように、コードを文書化することです。

Answer 5

あなたが本当にステータスコードの意思を記述#define文のenumまたはブロックでそれらを使用する方法、ステータスコードを使用したい場合。例えば

：

enum 
{ 
    kSuccess = 0, 
    kFailure = -1, 
} 

function foo() 
{ 
    return kSuccess; 
} 

if (kSuccess == foo()) 
{ 
    // Handle successful call to foo 
} 
else 
{ 
    // Handle failed call to foo 
} 

この方法で、意図が明確であり、誰かが将来的にあなたのコードを使用したり、維持したいエラーを起こしやすい当て推量はありません。

Answer 6

ちょうどあなたの状況で適切な場合があり、別のオプションを使用してボード上のジャンプ：

enum fooret { GOOD, BAD, UGLY, WORSE }; 

fooret foo(); // defined elsewhere 

switch(foo()) 
{ 
case BAD: 
case UGLY: 
    // maybe a recoverable failure(s)... 
    // take appropriate actions 
    break; 
case WORSE: 
    // maybe non-recoverable 
    break; 
case GOOD: 
    // successful, take appropriate actions 
    break; 
} 

Answer 7

if (foo()) { 
    // what to do if false 
} else { 
    // what to do if true 
} 

このアプローチの問題は過剰ネスティングです。 、過剰ネスティング問題を解決するため、戻り値を反転させるには

if(foo1()) { 
    if(foo2()) { 
     if(foo3()) { 
      // the rest of your code 
     } else { 
      // handle error 
     } 
    } else { 
     // handle error 
    } 
} else { 
    // handle error 
} 

：

if(!foo1()) { 
    // handle error 
    return; 
} 

if(!foo2()) { 
    // handle error 
    return; 
} 

if(!foo3()) { 
    // handle error 
    return; 
} 

このソリューションは、別の問題に苦しんでいるあなたは、あなたが呼び出したい3つの機能を持っていると仮定します。プログラムロジックとエラー処理コードが混在しています。これはすべてを複雑にします。理想的には、プログラムロジックとエラー処理を分離したいとします。この問題はGotoで修正できます

if(!foo1()) 
    goto error1; 

if(!foo2()) 
    goto error2; 

if(!foo3()) 
    goto error3; 

return; 

error1: 
    // handle error 
    return; 
error2: 
    // handle error 
    return; 
error3: 
    // handle error 
    return; 

多くのクリーナー。

また、gotoはリソースの割り当て解除の問題を解決できます。詳細については、Eli BenderskyのUsing goto for error handling in Cを参照してください。