[PHP] foreach内の処理を関数化したら動かなくなった

Question

わからないこと

• PHPの処理を関数化したら元々の処理が動かなくなった。
• そもそも関数化できていないのかも知れないです。

関数化したコード

  function getData($id) {
    $apiUrl = "https://example.com";
    $context = stream_context_create(
          array(
            'http' => array(
              'ignore_errors' => true
            )
          )
    );
    $response = file_get_contents($apiUrl . $id, false, $context);
    $json = json_decode($response);
    $name = $json->Name;
    $prefName = $json->PrefName;
    $areaName = $json->AreaName;
    $latitude = $json->Latitude;
    $longitude = $json->Longitude;
}

foreach文

foreach ($ids as $id) {
    $testId = $id->meta_value;
    getData($testId);
    // 以下省略
}

Comments

[退会済みユーザー]

動かなくなった、とは、なにかphp由来のエラーメッセージが表示されるのか、それとも。

[spellbound]

関数に返り値がない状態となっていました。また、関数を呼び出しているのにも関わらず、繰り返し文内で変数を定義していなかったのでデータがからの状態でした。ありがとうございます。

Answer 1

名前が違いますよ

定義名：getData()
呼び出し側（foreach内）：getAcmData()

関数の未定義エラー出てません？

追記：
ただ関数化したもののデータ取得するだけで何も返却してないようですが、サンプルコードだからですか？

Comments

[spellbound]

ご指摘ありがとうございます。
テストコードで消し忘れていました。

[spellbound]

なお、エラー文は出ておらず、Foreach内でINSERT INTOしていますが、関数化したところだけ保存されなくなりました。

[m.ts10806]

getData関数で何も返却していないからです。
関数内で定義した変数のスコープは当該関数内のみです。
return json_decode($response);　
として呼び出した側で受け取ると良いでしょう。
つまり、 $name = $json->Name;から下は関数化しなくていいってことですね。

[m.ts10806]

呼び出し側は、
$json = getData($testId);
として
$name = $json->Name;　と続ける感じです。

[spellbound]

ありがとうございました！！！！！！????