質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.62%

  • 関数

    210questions

    関数(ファンクション・メソッド・サブルーチンとも呼ばれる)は、はプログラムのコードの一部であり、ある特定のタスクを処理するように設計されたものです。

関数の戻り値 TrueとFalseの使い分けについて

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 1,584

pocket8137

score 33

関数の戻り値に真偽値を使用する場合、以下の2つの考え方を耳にします。

  1. TrueとFalseの語源から、
    1.1 正常終了の場合 : True
    1.2 異常終了の場合 : False

  1. 正常終了のステータスコードが0, エラーが1で、
    Trueが1、Falseが0であるので、
    1.1 : 正常終了の場合 : True
    1.2 : 異常終了の場合 : False

どちらの言い分にも一理あり、どちらがいいかは人によるとは思います。
ですが、チームで開発する以上、方針はどちらかに決めるべきだと思うのですが、一般的にはどちらが採用されるのでしょうか。

また、こういったコーディング規約をまとめたガイドラインなどはないのでしょうか。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

+4

真偽を返却する、ということに対して、これ以上決める方針はないですよね。
多分、

1.正常終了のステータスコードが0, エラーが1で、Trueが1、Falseが0であるので、

からの記述については誤記で、

2.正常終了のステータスコードが0, エラーが1で、Trueが1、Falseが0であるので、
2.1 : 正常終了の場合 : False
2.2 : 異常終了の場合 : True

ではないですか?
そうだと仮定して、質問を以下に解釈して回答します。

正常/異常を真偽で返却する時に、どちらをTrue/Falseにしますか?

正常=Trueの方が誤解しないと思います。

エラーステータスの場合の主語はエラーであるのでエラーで無い時に0とするのは一般的で、
かつエラーの内容で細かく1,2,3・・・などと拡張したりしますが、前提として正常/異常の2値だけで扱うのは既に別物です。

それでも同様に主語をエラーにしたエラーステータスとするなら、異常=Trueとすれば筋は通っていますが、真偽という2値にした時点で、真/偽=正常/異常と解釈する人の方が多いと思われます。

関数の名前次第ですが、IsNull()とかIsError()とか関数名が処理を明確に表しているなら誤解はないですが、エラーステータスを引き合いに出される位なので、そういった処理ではないのでしょう。
主語をエラーにしない、例えば「実行結果」として、正常/異常=True/Falseとするのが良いのではないでしょうか。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+2

もっと言語なり環境なりを限定しないとなんとも言えませんが、真偽値を返す関数ならば、真を返すのも偽を返すのも正常終了というのが一般的かと思います。たとえば、「人を引数にとって選挙権があるかどうか返す関数」なら、年齢や住民票などをもとに判定して、選挙権があるなら真値、ないなら偽値を返しますが、どちらも正常終了でしょう。
こういう場合、例外機構があれば異常終了は例外を投げるのでしょう。先の関数で人じゃなくて犬が引数に与えられた、とか。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+1

結論は「どちらの言い分も的外れ」ですね。

関数にしろ変数にしろ、それがどんな機能を持っているかを明確にして命名すべきであり、そこのところが全然分かっていないからtrueを返すべきかfalseを返すべきか、みたいな、頓珍漢な事で悩むんでしょう。

bool judge(int a){} みたいな命名してたりするんじゃないですか?

bool canDivideBy3(int a){} なら、与えられたaに対してどっちが返るべきかは一目瞭然だと思いますけど。

着目すべき場所がおかしいんですよ。

フレームワークとかでModelの大元の呼び出しとかはどうしたらええの? bool Model::exec(){} やねんけど、みたいなのはまぁ、好きにしたら良いんじゃないですかね。

だからまぁ、ステータスじゃなくフラグを返す関数なのに良い命名ができないとしたら、それは関数がでか過ぎて機能が複雑だって事なんですよね。そしたら何をすべきかも見えてくるでしょ。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.62%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • 関数

    210questions

    関数(ファンクション・メソッド・サブルーチンとも呼ばれる)は、はプログラムのコードの一部であり、ある特定のタスクを処理するように設計されたものです。