PHP メソッドの呼び出し

Question

      //クラス定義
        class zodiac_signs {

            //プロパティ
            private $animals1 = array('子', '丑', '寅', '卯');
            private $animals2 = array('辰', '巳', '午', '未');
            private $animals3 = array('申', '酉', '戌', '亥');
            private $animals = array($animals1, $animals2, $animals3);
            
            //メソッド定義
            public function foreach($animals as $val) {
                echo $animals[0][0].$animals[0][1].$animals[0][2].$animals[0][3].'<br />';
            }
            //メソッド定義
            public function foreach($animals as $val) {
                echo $val[0].$val[1].$val[2].$val[3];
            }

        }   

すみませんメソッドで呼び出しのですが、上手く呼び出されません。
アドバイス頂けないでしょうか？

Comments

[退会済みユーザー]

そもそもメソッド定義の文法がおかしいので、呼び出す方法なんてないと思いますが。

Answer 1

https://www.php.net/manual/ja/functions.user-defined.php
ここに書いてある様に関数の定義の仕方を見てもらえるとわかると思うのですが。

public function 関数名 {
//処理
}

で最初にメソッドを正しく定義。

クラスのプロパティーの呼び方も修正して。この場合こんな感じになると思います。

$this->animals;

最後に呼び出しをできます。

$obj = new zodiac_signs();
$obj->関数名();

参考までにこちらもどうぞ
https://qiita.com/mpyw/items/41230bec5c02142ae691

Answer 2

どんな仕様を求めているのか分からないのでなんとなく。
ヒントになりますでしょうか。

【修正】色々見直さずにコピペだけで書こうとしたのが間違いでした。酷すぎるコードでごめんなさい；；　ちゃんと書き直しました。

/*
$zodiac_sings = new zodiac_signs();
$zodiac_sings->my_hogehoge();とすると、
子丑寅卯<br />

$zodiac_sings->my_foreach();とすると、
子丑寅卯辰巳午未申酉戌亥
と出力されます
*/

// Your code here!
class zodiac_signs {

    //プロパティ
    private $animals1;
    private $animals2;
    private $animals3;
    private $animals;

    //コンストラクターで設定
    public function __construct()
    {
        $this->animals1 = array('子', '丑', '寅', '卯');
        $this->animals2 = array('辰', '巳', '午', '未');
        $this->animals3 = array('申', '酉', '戌', '亥');
        $this->animals = array($this->animals1, $this->animals2, $this->animals3);
    }

    //メソッド定義
    public function my_hogehoge() 
    {
        echo $this->animals[0][0] . $this->animals[0][1] . $this->animals[0][2] . $this->animals[0][3] . '<br>';
    }

    //メソッド定義
    public function my_foreach()
    {
        foreach( $this->animals as $val ) {
            echo $val[0].$val[1].$val[2].$val[3];
        }
               
    }

}   

Comments

[t-i]

すみません
private $animals = array($animals1, $animals2, $animals3);
の所で
Fatal error: Constant expression contains invalid operations in C:\xampp\htdocs\php.basic\index.php on line 20
このようなエラーが出てしまいます。

[yukikp]

すみません。あまり中身を水酷いコードを書いてました。ホント恥ずかしい。。。

書き直して、実行して確認しました。

[t-i]

わざわざ修正して頂き、ありがとうございます！
すみませんエラーは出なくなったのですが、画面に何も表示されなくなりました。

[yukikp]

それはそうです。クラスを読み込んだだけでは何もしません。
関数だってfunction hogehoge(){ }みたいに書いただけでは何も動かないでしょ？
使うところで、
$result = hogehoge();
って書いてあげる必要がありますよね。

クラスも同じです！
上の方でコメントアウトしたところに書きましたが、
同じファイル上で構いませんが、クラスを定義してるclass zodiac_signs {　}の外側で、
$zodiac_sings = new zodiac_signs();
と書いて、クラスをインスタンス化し、それぞれのメソッドを呼び出してください。
例：$zodiac_sings->my_hogehoge();

[yukikp]

とかなんとか偉そうん書いてて実は間違ってたりして。

https://paiza.io/projects/AkW8CuE9AfzYEjWhW0qhdg

て試しに実行してみましたが、ちゃんと動くようです。

[t-i]

丁寧なご返信ありがとうございました。
すみません上手く出来ました。
全然理解してなくて申し訳ありません。
もう少し勉強して質問の方もしたいと思います。
今回は本当にありがとうございました。
助かりました。

[yukikp]

クラス・インスタンス化・オブジェクトあたりってプログラムを学習するうえでの一つの山場ですよね～
頑張ってください！