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

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

ただいまの
回答率

90.51%

  • PHP

    20322questions

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

[PHP]関数の返り値について

受付中

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 221

newyee

score 102

function insert_cart($link, $user_id, $item_id){

    $date = date("Y/m/d H:i:s");


    $sql = 'INSERT INTO cart_table(
                user_id,
                item_id,
                amount,
                created_date
            ) VALUES (
                '. $user_id . ',
                ' . $item_id . ',
                1,
                \'' . $date . '\')';


    return !(mysqli_query($link,$sql) === FALSE); 

}


上記の関数の、returnの部分が良く分からないのですが、mysqli_queryを実行した結果、FALSEでなかったらTRUEを返すという解釈で合っていますでしょうか...?

return !(mysqli_query($link,$sql) === FALSE); 
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 4

+3

mysqli_queryを実行した結果、FALSEでなかったらTRUEを返す

そうなんですが、いかにもパッと見で分かりにくいですよねえ。

return !(query($arg) === FALSE);

PHPに限らず、上のような書き方は、リファクタリングしたくなります。

return (query($arg) === TRUE);

「リターン値はクエリがTUREならTURE」と読み下せて、分かりやすい。

return query($arg);

なおもし、queryの返り値が1や0とかではなく、
直接TUREかFALSEを返してくる場合なら、
そのまま渡した方が簡潔になります。

というかそもそも、queryの返り値を
TUREかFALSEの二値で返すように書けば、
他の部分もみんな上のように簡潔に書けて良いと思います。

逆に、二値ではダメで、結果が1ならそのまま変数に足す
部分とか他にあると支障をきたすかもしれませんが、
そもそも、そういうのもリファクタリング対象です。

細かいことですが、こういう改善を積み重ねると、
読みやすいコードになっていくので地味に大事です。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/11 17:16

    ご丁寧なご回答ありがとうございます。
    ご回答下さったこの部分なのですが、「return query($arg);」これを、僕がご質問させて頂いた、コードに当てはめさせて頂きますと、「return (mysqli_query($link,$sql));」となるかと思いますが、これの返り値と致しましては、クエリを実行した結果「TRUE」だった場合には「TRUE」を返す、という理解で合っていますでしょうか...?

    キャンセル

  • 2017/11/11 18:20

    >「return (mysqli_query($link,$sql));」
    >クエリを実行した結果「TRUE」だった場合には「TRUE」を返す
    関数の返り値そのままリターンで返すので、TUREならTUREです。

    しかし、論理演算子を外すと二値ではなくなるので、
    関数が二値以外を返す場合、【結果が変わって】しまいます!
    たとえば、1とか2を返すとか、オブジェクトを返すとか……。

    だから、一般論として展開するために、わざわざ別の例に書き換えたわけです。
    そして、一番目と二番目は論理的に等価ですが、三番目は違うのです。だから、
    三つ目のサンプルコードだけ出さずに、注意書きも後半に書いておいたわけです。

    そういう結果が変わってしまう場合があるので、
    リファクタリングをするときは、壊さないよう、同時にテストを行いましょう。

    キャンセル

+1

!
は論理否定演算子ですので
mysqli_query($link,$sql)
がFALSEでなければTRUEを返しますね

http://php.net/manual/ja/language.operators.php
! ( 論理否定演算子)

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

+1

このコードはユーザーが不正な値を入力した場合、SQLインジェクションという脆弱性を発生させてしまいます。
そういう場合は

  1. mysqli_prepare 関数を用いてパラメータを疑問符(?)にする
  2. mysqli_stmt_bind_param 関数を用いてユーザー入力をエスケープする
  3. mysqli_stmt_execute 関数でSQLを実際に実行する
  4. mysqli_stmt_bind_result関数を呼び出して実行結果を取得する
  5. mysqli_stmt_close関数を呼び出してプリペアステートメントを終了
<?php

function insert_cart($link, $user_id, $item_id){

    $date = date("Y/m/d H:i:s");

    $result = null;

    $parameters = [
        'user_id' => $user_id,
        'item_id' => $item_id,
        'amount' => 1,
        'created_date' => $date
    ];

    $sql = 'INSERT INTO cart_table SET ';
    foreach($parameters as $column_name => $value) {
        $sql .= sprintf('`%d` = ?, ', $column_name);
    }
    $sql = preg_replace('/,\s$/', '', $sql);

    if ($stmt = mysqli_prepare($link, $sql)) {
        foreach(array_values($parameters) as $param) {
            mysqli_stmt_bind_param($stmt, 's', $param);
        }
        mysqli_stmt_execute($stmt);
        mysqli_stmt_bind_result($stmt, $result);
    }
    mysqli_stmt_close($stmt);
    return $result;
}

$sql = preg_replace('/,\s$/', '', $sql);としているのは、ループするとカンマとスペースが入って文字列が生成されるため、それを詰めるための処理です。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

-1

これどこで見たコードですか?
どこかのサイトか本なら参考にしないほうがいいです。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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

  • PHP

    20322questions

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