Help us understand the problem. What is going on with this article?

参照渡しでのエラー「Only variables should be passed by reference」について

最近コード書いてたら出くわしたエラーに関して忘備録として残しておきます。
間違えてる箇所がありましたら優しくご指摘お願いします。

「Only variables should be passed by reference」
ざっくりというと「参照渡しは変数しか使えませんよ〜」っていうエラーです。

下のコードのように、storeContactInfoの引数の中に直接「$id = null」を入れていたのですが、
このやり方ではダメっぽいです。

foreach ($infoList as $info) 
{
  // 登録処理
  $this->storeContactInfo($name, $id = null);
}

private function saveContactInfo(string $name, ?int &$id)
{
 (省略)
  // DBに登録
  $contact->save();
}

そのため、一旦変数に入れてあげて、その変数を引数で使うようにしてあげたら無事に解決しました。

foreach ($infoList as $info) {
  $id = null;
  // 登録処理
  $this->storeContactInfo($name, $id);
}

一見おかしくはなさそうなんですけど、saveContactInfo()側は参照渡しを要求しているのに、
呼び出し側で値渡ししているのが原因でおきてるそうです。

解決できたので、めでたしめでたし。

phper_sugiyama
LaravelをメインにCakePHP,Symfonyを書いているPHPerエンジニアです。 最近は趣味でGolangやVue.jsも勉強しています。間違ってる箇所があったら指摘してくれると助かります。。。
Why not register and get more from Qiita?
  1. We will deliver articles that match you
    By following users and tags, you can catch up information on technical fields that you are interested in as a whole
  2. you can read useful information later efficiently
    By "stocking" the articles you like, you can search right away
Comments
No comments
Sign up for free and join this conversation.
If you already have a Qiita account
Why do not you register as a user and use Qiita more conveniently?
You need to log in to use this function. Qiita can be used more conveniently after logging in.
You seem to be reading articles frequently this month. Qiita can be used more conveniently after logging in.
  1. We will deliver articles that match you
    By following users and tags, you can catch up information on technical fields that you are interested in as a whole
  2. you can read useful information later efficiently
    By "stocking" the articles you like, you can search right away
ユーザーは見つかりませんでした