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

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

ただいまの
回答率

90.34%

  • PHP

    21325questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

else句の必要性

解決済

回答 6

投稿

  • 評価
  • クリップ 3
  • VIEW 1,674

tomoyuki123

score 204

こう書いてる人がいたので

if($id) {
    return false;
}
retrun true;


else句入れてくださいってコメントしたら
else句つける派とつけない派ができてしまって

if($id) {
    return false;
} else {
    return true;
}

2つの分岐が対になってるかんじがあった方が見やすいかなと思ったのですが、else句をつけない人が多いようで
PSRとかも改めて見たのですが特に言及されてることもなく、結局は好みの問題になるんでしょうか?

でもやっぱり両方が等しく起こり得ることならelse句つけた方がいいと思うのですが普通どうなんでしょうか?

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 6

checkベストアンサー

+13

むしろ、「単にreturnするだけのif文」は1行で書いて、あとにインデントを残さない、という方が主流です(「ガード節」という名前までついている書き方です)。

function some_func($arr) {
  if(!is_array($arr)) return null;
  if(empty($arr['foo'])) return null;
  // チェックを済ませた上で、本題の処理を書き始める(後略)
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/04/11 13:34

    maisumakunさんのご回答に沿った内容が、書籍「リーダブルコード」(オライリージャパン発行)の「7.5 関数から早く返す」「7.7 ネストを浅くする」に少し書かれていますので、機会があれば7章「制御フローを読みやすくする」やそれ以外の項目と併せてご覧になられてはと思います。

    キャンセル

  • 2017/04/11 20:47

    皆様いつもありがとうございます!
    色んな考え方があってどうしようかなと思ってますがガード節使って見たり、リーダブルコード読み直してみようかなと思ってます

    キャンセル

+8

私は、どっちでも良いと思います。

というのも適切に短いメソッドであれば、読み間違える心配がないからです。強制して宗教論争をしてもメリットがあるようには思えません。強制をしないという合意を作った方がいいくらいだと思います。
(逆に、スパゲッティなコードであれば、細かい点まで共通化する意義があります。)

とはいえ、もしこれがガード節であれば、ネストを浅くするためにelseは不要だと思います。

return ! $id;

ちなみに私だっだら質問のコードであれば、このくらいの荒っぽいコードで書きます。elseを書くと5行これだと1行、長いコードよりも短いコードが読みやすいという考えからです。

メソッド名が適切であれば特に問題にはならないとかんがえます。その代り、1行であってもメソッドに分割することは多いです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

+5

可読性に言及すると人それぞれと言われそうですが、
スッキリさせるためにはelse句は要らないのではと思います。

また、コード変更の際に if ($id){}以降にコードを追加したとすると、
else句内のreturn true;を見落としてその後のコードが絶対に通らないなんて不具合も起きえます。

さらにはifの条件を複数にする場合、全ての条件を一つのif文に書かなければならなくなるので
if文条件の可読性が下がる場合があります。

// これはできるが
if($id && $name && $data) {
    return false;
} else {
    return true;
}
// これはできない
if($id) {
    return false;
} else {
    return true;
}
if($name) {
    return false;
} else {
    return true;
}
if($data) {
    return false;
} else {
    return true;
}
// returnが最後なら綺麗
if($id) {
    return false;
}
if($name) {
    return false;
}
if($data) {
    return false;
}
return true;


上記の条件はシンプルなので1つのif文でもいいですが・・・
一番上のソースだと、どの条件に引っかかったのかログを出す場合にも苦労しそうですよね

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

+2

基本的にmaisumakunさんのやり方に賛成ですし
個人的にもほとんどそれでしかコーディングしていませんが
関数を途中で終わらすのが気持ち悪いと思う人もいるでしょうし
プロジェクトによって方針も違うので参考程度に
returnは1度しか発行しないという考え方もあります

function myfunc(){
ret=true;
if(条件式){
ret=false;
}
return ret;
}


ただ、早々に除外されるようなものもその関数の最後まで処理を追いかける分
非効率だというデメリットはあります。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/04/11 17:31

    昔、TRONの開発に携わっていた人とお仕事をする事があったのですが、その人に、入り口は1個、出口は1個がソフトウェア工学の基本だとおしえていただきました。それを突き詰めると、この記述方法がとても有効だと気づいて、使ってました。

    キャンセル

+2

elseを付けるとその次の行に到達しなくなるのが気持ち悪いので、私はまず使わないですね。

    if($id) {
        return false;
    } else {
        return true;
    }
    // ここに到達しない
}

なのでこういう風に書くことは良くあります。

    $ret = true;
    if($id) {
        $ret = false;
    }
    return $ret;
}


しかし一番好みなのはmaisumakunさん回答の方法ですね。
名前がついているのは知りませんでしたが。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

むしろ、自分のつかっているjetbrainのIDEだと、一行で書けるよって忠告が表示されます。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

  • PHP

    21325questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。