関数が思い通りに動かない

前提・実現したいこと

DBには「faceicon1_1」～「faceicon2_3」といったカラムがあり、そこには0～4の値が入っています。DBへ接続し、各faceiconが1～4であればそれぞれの画像のURLを代入するという関数を作成したいのですが、どうもswitch文のところが思い通りの挙動になりません。

発生している問題・エラーメッセージ

エラーメッセージはないが、変数には欲しい値が入っていない。
var_dump $faceicon1
→
string(0) ""

該当のソースコード

php
1function getFaceicon($icon,$i){
2    $dsn = "mysql:host=ip;dbname=dbname;";
3    $user = "user_name";
4    $password = "password ";
5    $dbh = new PDO($dsn,$user,$password);
6
7    $sql = "select * from table_name where id = 1";
8    $stmt = $dbh->query($sql);
9    $res = $stmt->fetchAll();
10
11    $faceicon = "faceicon".$i;
12    global $faceicon1,$faceicon2,$faceicon3,$faceicon4,$faceicon5,$faceicon6;
13    $$faceicon = "";
14
15    switch($icon){
16        case 0:
17            $$faceicon = "";
18        break;
19        case 1:
20            $$faceicon = "~~/ico_base.png";
21        break;
22        case 2:
23            $$faceicon = "~~/ico_fun.png";
24        break;
25        case 3:
26            $$faceicon = "~~/ico_ang.png";
27        break;
28        case 4:
29            $$faceicon = "~~/ico_sad.png";
30        break;
31    }
32}
33getFaceicon('$res[0]["faceicon1_1"]',1);
34getFaceicon('$res[0]["faceicon1_2"]',2);
35getFaceicon('$res[0]["faceicon1_3"]',3);
36getFaceicon('$res[0]["faceicon2_1"]',4);
37getFaceicon('$res[0]["faceicon2_2"]',5);
38getFaceicon('$res[0]["faceicon2_3"]',6);

試したこと

switch($icon) の箇所をはじめから switch($res[0]["faceicon1_1"]) にしておくと
うまく取得できる。しかしコード量が多くなるため当方が求めている関数ではなくなる。

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

ここにより詳細な情報を記載してください。

行動規範の内容に同意します

回答2件

シングルクォートで囲っても変数は展開されませんよ。そういう文字列として解釈されるだけになります。

php
1$test = "hello";
2echo '$test world'; //$test world
3echo $test.' world'; //hello world
4echo "$test world"; //hello world
5echo "{$test} world"; //hello world
6echo "${test} world"; //hello world
7echo $test." world"; //hello world

[php]phpにおける、"（ダブルクォート）と、'（シングルクォート）の使い分けAdd Star

また変数だけであればクォートで囲う意味はないです。

確認すべきは関数で受けとる引数が想定通りの情報かどうかです。
あと変数で$$となっていますがこれは大丈夫ですか？
→失礼しました。よくみたら可変変数ですね。自身は明示的にするため{}で囲うので違和感がありました

php
1$t = "test";
2${$t} = "hello";
3echo ${$t}." world"; //hello world
4echo $test." world"; //hello world

途中でglobalが使われていたり全体の流れがよくわからないコードなので
現状から言えるのはこれくらいです。

投稿2018/08/26 06:05

編集2018/08/26 10:02

m.ts10806

総合スコア80850

re_yama

2018/08/26 11:17 編集

ご回答頂き誠に有難うございます。＞変数だけであればクォートで囲う意味はないです。 →説明不足で申し訳ございません。こちらも試したことですが、 getFaceicon("$res[0]['faceicon1_1']",1); にするとresが未定義の変数とエラーになってしまいます。それとswitch($res[0]["$icon"])としてgetFaceicon("faceicon1_1",1);でも値が空白で返ってしまいます。＞途中でglobalが使われていたり →関数内で定義した変数ですが、関数の外で使用しなければいけないためglobal化しています。この考え方は間違っていますでしょうか。グローバルにしないと関数外では、同じくエラーになっていました。こちらも説明不足で申し訳ごじませんが、最終的に使用したい変数は関数の外での$faceicon1~6です。

m.ts10806

2018/08/26 12:35

globalは関数の外で宣言された変数を関数で使うものなので違います。変数未定義はそれはそうですよ。今のコードではどこにも定義されてません。全体の流れがよくわからないコード、と回答に書きましたが、まさかこのコードで全部じゃないですよね？全てだとしたら相当構造がおかしく、変数や関数の使い方の基礎から見直した方が良いです。関数で宣言した変数を関数の外で使いたければ、関数でreturnするか、そもそも引数を参照渡しするかだと思います。

行動規範の内容に同意します