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

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

ただいまの
回答率

89.55%

引数に複数の値を渡す

解決済

回答 4

投稿

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

uverworld101nm

score 98

再投稿になります。
isset関数は引数に複数の値を渡すことができるので使いました。
※前回の修正後のコードです。

<?php
if (isset($_GET['keyword']|| $_GET['budget'])) { 
$key='私のキー';
$count='50';
$large_area='◯◯';
$start =1+($page-1)*10;
$keyword=$_GET ["keyword"];
$budget=$_GET ["budget"];
$format='xml';
$url='http://webservice.recruit.co.jp/hotpepper/gourmet/v1/?key='.$key.'&count='.$count.'&large_area='.$large_area.'&start='.$start.'&keyword='.$keyword.'&budget='.$budget.'&format='.$format;
$xml=simplexml_load_file($url);
・・・
・・・
・・・
}


しかし、isset($_GET['food_category']をパラメータに追加したらエラーになりました。
isset関数の三つ目以降は、isset関数を少し改良しなくてはいけないのですか?従来のisset($_GET['food_category']だとエラーになりやすいですか?
他に考えられる原因がないので。

if (isset($_GET['keyword']) || isset($_GET['budget']  || isset($_GET['food_category'] )) { 
$key='私のkey';
$count='50';
$large_area='◯◯';
$start =1+($page-1)*$count;
$keyword=$_GET ["keyword"];
$budget=$_GET ["budget"];
$food_category=$_GET["food_category"];
$format='xml';
$url='http://webservice.recruit.co.jp/hotpepper/gourmet/v1/?key='.$key.'&count='.$count.'&large_area='.$large_area.'&start='.$start.'&keyword='.$keyword.'&budget='.$budget. '&food_category='.$food_category.'&format='.$format;
$xml=simplexml_load_file($url);
・・・
・・・
・・・
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • 退会済みユーザー

    2016/08/29 09:05

    こちらの質問が他のユーザから「やってほしいことだけを記載した丸投げの質問」という指摘を受けました
    「質問を編集する」ボタンから編集を行い、調査したこと・試したことを記入していただくと、回答が得られやすくなります。

回答 4

+2

ライブラリの説明をちゃんと読んでください。
http://php.net/manual/ja/function.isset.php

複数の変数をセットする場合、[||] じゃなく、[,]です。

bool isset ( mixed $var [, mixed $... ] )

「||」の意味は以下のページを参考してください。
http://qiita.com/Keech/items/54a2760ede488d127f04

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/08/29 11:05

    ご回答ありがとうございます。
    そうですね。確認してみます。

    キャンセル

+2

if (isset($_GET['keyword']) || isset($_GET['budget']  || isset($_GET['food_category'] ))


if (isset($_GET['keyword']) && isset($_GET['budget']  && isset($_GET['food_category'] ))


if (isset($_GET['keyword'], $_GET['budget'], $_GET['food_category'] ))

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/08/29 11:04

    ご回答ありがとうございます。
    わかりやすい説明ありがとうございます。

    キャンセル

checkベストアンサー

+1

以下2点方針がきまらないと回答がぶれますね

  • keyword,budget,food_categoryすべてが揃っていないといけないのか
    一部でもあればよいのか?
  • 一部で良い場合設定されていない他の値のデフォルト値はどうするのか

ちなみに、GETで受けた値を直接urlにつなげるのはNGです
必ずエンコードしましょう

 追記

仮にHTML側のフォームが

<form method="geT">
keyword:<input type="text" name="keyword" value=""><br>
budget:<input type="text" name="budget" value=""><br>
food_category:<input type="text" name="food_category" value=""><br>
<input type="submit" value="search">
</form>


だった場合、keyword,budget,food_categoryは空文字でデータが渡るので
issetのチェックでは不十分です。
以下のように、中身までチェックした上で分岐を考えてください

$keyword=filter_input(INPUT_GET,"keyword",FILTER_DEFAULT,array("option"=>array("default"=>"")));
$budget=filter_input(INPUT_GET,"budget",FILTER_DEFAULT,array("option"=>array("default"=>"")));
$food_category=filter_input(INPUT_GET,"food_category",FILTER_DEFAULT,array("option"=>array("default"=>"")));

if(!in_array("",[$keyword,$budget,$food_category])){
print "ok";
}else{
print "ng";
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/08/30 16:34

    というと、何かのエラーというわけではないんですね?

    キャンセル

  • 2016/08/30 16:45

    分岐をテストしているだけなのでエラーではないです

    キャンセル

  • 2016/08/30 16:46

    ありがとうございます。
    助かりました。

    キャンセル

0

カッコの数がたりてないのでは?
syntax errorがでていると思うのですが?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/08/29 11:05

    ご回答ありがとうございます。
    確認してみたところ、そうでした。
    大変すみませんでした。僕の落ち度です。

    キャンセル

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

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

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