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

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

ただいまの
回答率

88.64%

【PHP】クラス内で指定した変数を、同じクラス内の関数から取得するにはどうしたら良いでしょか?

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 3,397

hasshy

score 97

クラス内で定義した変数を、同じクラス内の変数で呼び出せません。
何が原因でしょうか?

 コード

構成は下記です。
ルートディレクトリ直下に全てのファイルがあります。 

  • common.php: class自体が書かれたファイル
  • foo.json: classのコンストラクタで指定するjsonファイル
  • index.php: classを呼び出すファイル。

最初に、constructでjsonを読み込み複数の配列を取得します。  
別のタイミングで、getFoo()で配列から対象の配列のみ返すと言う処理をします。  

 common.php

<?php
class CommonFunctions{
    private $_foo = [];

    function __construct($dir) {
        $this->_foo = json_decode(file_get_contents($dir . '/foo.json'), true);
        // var_dump($this->_foo)で、jsonの中身を見れることは確認済み。
    }

    // jsonファイルから該当する配列を取得する
    public function getFoo($id){
        $foos = $this->_foo;
        $targetFoo = array_filter($foos, function($var) use ($id){
            return $var['id'] == $id;
        });

        $result = [];
        if(count($targetFoo) > 0) {
            $result = current($targetFoo);
        };

        return $result;
    }
}
 ?>

 foo.json

[
  {
    "id": 1,
    "name": "test_1",
  },
  {
    "id": 2,
    "name": "test_2",
  }
]

 index.php

<?php
    require(__DIR__ . '/common.php');
    $commonFunctions = new CommonFunctions(__DIR__);
    $getFoo = $commonFunctions::getFoo(1);
    var_dump($getFoo);
?>

 エラー内容

Fatal error: Uncaught Error: Using $this when not in object context

 selfで呼び出そうとした場合

蛇足かもしれませんが、self::_scenesでも確認しました。

Fatal error: Uncaught Error: Undefined class constant '_scenes'

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • m6u

    2018/11/26 22:53

    第三者が再現できる、シンプルな構造のものは用意できませんか?

    キャンセル

  • m.ts10806

    2018/11/26 23:17

    CommonFunctionsを呼び出している処理も合わせて記載してください

    キャンセル

  • hasshy

    2018/11/26 23:51

    ご指摘ありがとうございます。再現できるように情報を追記しました。

    キャンセル

回答 2

checkベストアンサー

+3

 $getFoo = $commonFunctions::getFoo(1);

でgetFoo()をスタティックメソッドとして使おうとしているので、オブジェクト自身を指す$thisは使えません。

コンストラクタでインスタンス変数を指定する設計なのであれば、

 $getFoo = $commonFunctions->getFoo(1);

とする必要があります。

staticキーワード

static メソッドはオブジェクトのインスタンスを生成せずに コールすることができるので、疑似変数 $this は、 static として宣言されたメソッドの内部から利用することはできません。

同ページのコード例を読むと挙動が理解しやすいかと思います。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

+1

selfはクラス内定数にアクセスするときに使えるけど、
$thisとは目的が違うからね。
クラスの外でも、クラス内定数をpublic宣言しておけば参照することはできたりする。
PHP: オブジェクト定数 - Manual

array_filter()に与えている無名関数からは$thisのことがわからないので
そういうエラーを返しているのかと。
単純変数に一旦入れてからarray_filter()にかければ解決しそうな気がする。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/11/26 22:29

    ご回答ありがとうございます!
    コード側も更新させていただきました。
    単純変数に入れて見たのですが、その部分で同じエラーが出てしまいます。
    無名関数ではなく、そもそも$thisで呼び出せないようです。

    キャンセル

  • 2018/11/26 22:56

    エラーメッセージの示す行は、確実にそこですか?
    他の箇所で同様な呼び出しをしていて、そこと間違っていませんか?(念のため確認)

    キャンセル

  • 2018/11/26 23:53

    エラーが発生した箇所は、「$foos = $this->_foo;」で間違いありませんでした。

    キャンセル

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

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

関連した質問

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

  • トップ
  • PHPに関する質問
  • 【PHP】クラス内で指定した変数を、同じクラス内の関数から取得するにはどうしたら良いでしょか?