php初心者　コンストラクタの使い方はあってるのでしょうか？

前提・実現したいこと

初めてプログラミングで開発をしてみようとしている初心者です。
phpで簡単なおみくじサイトのようなものを作りたいのですが、以下のエラーメッセージが発生しました。
######おみくじサイトのイメージ
「数字を入力してもらう」
↓
「その数字から大吉〜凶のどれかに別れる」
↓
「そのページに飛ぶ」

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

Fatal error: in /Users/Desktop/おみくじ/omikuji.php on line 13

該当のソースコード

php
1<?php
2class Omikuji {
3  public $name;
4  public $state;
5  public $explain;
6  public $health;
7  public $work;
8  public $love;
9  public $money;
10  public $study;
11  public $trip;
12
13  public function __construct ($name, $state, $explain, $health, $work, $love, $money, $study, $trip) {
14    $this->name = $name;
15    $this->state = $state;
16    $this->explain = $explain;
17    $this->health = $health;
18    $this->work = $work;
19    $this->love = $love;
20    $this->money = $money;
21    $this->study = $study;
22    $this->trip = $trip;
23  }
24}
25
26$unsei[1] = new Omikuji('大吉','運勢のピークは今日！', 'やりたいことはやろう！でも明日からは気をつけて…', 'すこぶる元気！疲れを知らない一日だ！', '強気で勝負しよう！', '思い切って告白だ！', '今日は使ったほうが増えるかも？', '今日はできるとこまでやってしまおう。明日は気分が乗らないはず…', '一人でも数人でもどこでも行こう！');
27$unsei[2] = new Omikuji('中吉', '今日は良い日になりそう。', '良い一日だ。気持ち良く過ごせそう。', '体調が良い、運動も◯', '上司に褒められるかも？', '好きな子と良い感じに…？', '思わぬ収入があるかも。', '今までわからなかった問題が解けるように！', '今日行くところは良いところ。');
28$unsei[3] = new Omikuji('吉', '普通の一日だ。', 'いつも通り過ごしましょう。', '普通に過ごしてれば、問題なし。', 'いつも通り仕事しよう。', '好きなこともいつも通り', '買いたいものは買いましょう', '普通に勉強しよう。サボらないでね。', 'プランはそのままで。');
29$unsei[4] = new Omikuji('小吉','今日は微妙かも。', 'なるべく大人しく過ごそう。' , '体調を崩しかねないので、無理しないで！', 'ミスの確かめ重要。', '喧嘩する可能性あり。', '今は貯めておくのが◯', '捗らない。とにかく努力！', 'ゆるーく楽しもう。');
30$unsei[5] = new Omikuji("凶", '史上最悪の一日。', 'ラッキーと思って一日ゴロゴロしてよう。', 'もうすでに体調悪い。', '絶対に連絡を取らない！', '最低限にしよう、食事のみ。', '今日はしなくてOK！', 'キャンセルした方が良いかも…');
31
32?>

試したこと

public function name() {
echo $this->name;
}
public function state() {
echo $this->state;
}
public function explain() {
echo $this->explain;
}
public function health() {
echo $this->health;
・
・
このようなメソッドを追加してみたりもしたのですが、同じエラーが出ました
何がダメなのか全くわかりません、どなたか教えていただけると、本当にありがたいです

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

Macのビルトインウェブサーバーを使って、Chromで見ながらやっています
phpのバージョンはPHP 7.1.32と出ます

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

回答1件

ベストアンサー

エラー詳細を見れば良いかと。

PHP Fatal error: Uncaught ArgumentCountError: Too few arguments to function Omikuji::__construct(), 8 passed

同じエラーは下記のようなコードで確認できます。

php
1<?php
2function test($a,$b){
3    
4    
5}
6
7echo test("a");

引数が足りてないという意味です。

要件次第ですが、きちんと引数を過不足なく設定するか、引数の初期値を設定するか、どちらかが一般的です。
ただ、あまりに引数が多すぎても可読性がよくありませんし、メンテナンス性にも欠けます。
配列にまとめてその中の該当のキーの有無によって処理内容を変えた方がコードとしては見通しがよくなるようにも思います。

投稿2019/12/18 13:17

編集2019/12/18 13:29