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

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

ただいまの
回答率

89.65%

PHP 配列要素有無

解決済

回答 2

投稿

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

ciaociao_HK

score 7

PHP if ($array>0){}

外国のPHP職人とjavascript職人の方の書かれたPHPの補修を承っており、解読に苦労しております。
その中で、下記の表現が気になりました。
要素の有無の判定をしているのですが、小生は「あれ?」と思って、count()を書きたくなってしまいます。

下記の表現の、お薦め度を教えて頂きたく、宜しくお願い致します。

Github等での勉強をすれば、よくお目にかかる表現なのかもしれませんが。

$sql="select * ・・・・";
$res=mysql_query($sql);
$row=mysql_fetch_array($res);
if($row>0){

補足情報(FW/ツールのバージョンなど)

オリジナルはPHP5.6で、PHP7.1への移行準備中

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+4

そもそもその関数7系で使えないのですが大丈夫ですか?

このマニュアルによると行がない場合、falseを返すようですが、countにfalseを渡すと1になります。

テストしてみた

なのでcountは使えない状況では?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/06/12 07:37

    ちなみにおすすめ度として、5.6の時点でmysql_*を使ってるので、おすすめできないですね。

    キャンセル

  • 2019/06/12 07:48

    ご回答ありがとうございます。
    PDOへの書き換えを進めていて、
    if($row>0){
    に対面し、「あれ、count()かな?」と思ったところでした。
    うろ覚えでは、いけませんでしたね。
    精進してまいります。

    キャンセル

  • 2019/06/12 07:59

    count()の出力を確認致しました。ありがとうございました。

    キャンセル

+2

既に出ているようにmysql*系の関数はPHP7では削除されておらず、機能しません。
mysqli*系の関数も悪いとは言えませんが、PDOを利用するのが確実です。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/06/12 18:52

    この質問に照らしたとき、fetchAllとmysql_fetch_arrayの大きな違いは、sqlの結果が0件のときに空配列を返すかfalseを返すかだと思います。
    count([])とcount(false)が違うからです。
    これから作る、ではなく既にあるコードの、既にある状態についてのレビューを求める質問に、現在のベストプラクティスの回答は合わないと思うので、デクリメントしました。

    キャンセル

  • 2019/06/12 18:58

    そうですね。私の場合は質問に提示されたコードがどうかというより別の機能でやりましょうということを書いていますし。
    何かない限りはfetchAllのほうが扱いやすいと思います。

    キャンセル

  • 2019/06/12 19:39

    皆様、ありがとうございます。

    お聞きしたかったのは、
    「if($row>0){」でして、
    「判定は、されているようだけど、配列と数値の比較ってってどうよ?」という事でしたが、
    $rowがSELECTの結果でしたので、上記の質問文になりました。

    うろ覚えのcount()再確認、文脈確認、フレームワーク活用、等々沢山の材料を頂きました。

    キャンセル

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

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

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