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

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

ただいまの
回答率

90.45%

  • PHP

    24613questions

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

  • SQL

    3094questions

    SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。

PHP 配列要素有無

解決済

回答 2

投稿

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

ciaociao_HK

score 1

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 09:36

    ご回答ありがとうございます。
    PDOに書き換え途中で「有無判定にcount()かな?」と思ったところでした。実際、count()の出力を確認して、冷汗をかいた次第です。

    キャンセル

  • 2019/06/12 09:40

    SELECT結果の件数をもって何をしたいかにもよります。
    提示した記事にあるように別途select count(*) の実行結果で判断する方法とfetchAll()で得た結果をPHPでcount()関数にて判断する方法が2つあります。

    キャンセル

  • 2019/06/12 09:53

    ありがとうございます。単純作業で、PDOに変更しようとしましたが、前後の文脈も、しっかり確認する必要がありましたね。

    キャンセル

  • 2019/06/12 10:13

    そうですね。
    内容次第ではコードだけをリファクタリングするのではなく
    新しく作り直したほうが良いこともあります。
    mysql*系関数を利用しているということはPHP5.6とはいえ
    相当レガシーなシステムであると推察できますので、単純置き換えよりはPHP7をきっちり使った実装にしたほうが良いでしょう。
    (※時間が許されない事情もあると思いますので、そこは時間と相談してください)

    キャンセル

  • 2019/06/12 10:22 編集

    ありがとうございます。CSRFとXSS対策も、対応準備中で、既に大がかりになっております。

    キャンセル

  • 2019/06/12 10:42

    んーそこまでいくとフレームワークに乗せ換えたほうが良さそうですね。
    元々のインジェクション対策ができていないのであれば自力でなんとかするのは酷だと思います。
    https://qiita.com/tadsan/items/fb496e450fc27c8c4494#フレームワークを利用しろ

    キャンセル

  • 2019/06/12 13:06

    ありがとうございます。小規模プロジェクトに、自前のセキュリティ対策を施した事が、ありまして、その延長線上で、考えておりましたが、フレームワークで作り直した方が良さそうですね。

    キャンセル

  • 2019/06/12 13:43

    学習コストとの兼ね合いで決めてください。
    選ぶフレームワークによっては前提となる知識や技術が高いものがあります(その分、新進気鋭で便利な機能を提供してはくれますが)

    キャンセル

  • 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で質問しよう!

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

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

  • PHP

    24613questions

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

  • SQL

    3094questions

    SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。