インスタンスの使い回しについて

カードゲームのようなものを開発しています。
カードの情報をまとめたCardクラスのインスタンスはどの画面でも使っています。
そこで画面ごとに都度生成するのではなく、予め全カード(103種類）のインスタンスを作っておき、どの画面でも使いまわせるようにしたいと思いました。（なお、カードの内容は使用上、上書きされることはありません。setterもありません。）

イメージとしては以下のような配列をどの画面でも使いたいです。
どのようにすれば、どの画面でも使いまわせるようにできるでしょうか？

php
1$cardInstanceList = [
2   "a" => new Card("a"), // 1
3   "b" => new Card("b"), // 2
4   // 以降 103まで
5];

y_waiwai

2021/11/07 07:02

で、しつもんはなんでしょうか

LOTOyamaguchi

2021/11/07 07:12

すみません、抜けていましたので追加しました。

m.ts10806

2021/11/07 09:31

コードは一部だと色んな憶測や勝手な想像が入ってしまい事実と異なる結果となり得るので、全体の流れが分かる全体像が見える形でコードを提示してください。

tanat

2021/11/07 15:43

> そこで画面ごとに都度生成するのではなく、予め全カード(103種類）のインスタンスを作っておき、どの画面でも使いまわせるようにしたいと思いました。何を目的としてこのような実装をしたいのかを追記されると、より適切な回答が得られるかと思いますよ。

dameo

2021/11/08 10:39

phpのデフォルトだとsessionすらファイルに保存するので、毎回生成する方が低コストな気がします。

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

回答4件

やりたいことは「シングルトン」のように見えますが、ひとまずセッションで保持すれば良いのでは。

投稿2021/11/07 07:03

m.ts10806

総合スコア80850

LOTOyamaguchi

2021/11/08 10:51

ありがとうございました。参考にさせていただきました。

m.ts10806

2021/11/08 10:55

何がどう参考になったのでしょう。 https://teratail.com/help/question-tips#questionTips4-2

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

ベストアンサー

いろいろやり方あると思いますが、自分ならこうするかなという実装を書いておきます。

Cardクラス

php
1<?php
2class Card {
3  public $name = '';
4
5  function __construct(string $name) {
6    $this->name = $name;
7  }
8}

CardRegistryクラス

Cardクラスのインスタンスを管理するためのクラスです。
外部からは$cardListを直接参照せず、getByKeyやgetRandomやlistAllなどアクセス用関数をここに定義すると使い勝手が良いです。

php
1<?php
2require_once './card.php';
3
4class CardRegistry {
5  private static $_shared = null;
6
7  public $cardList = [];
8
9  private function __construct() {
10    $this->cardList = [
11      'a' => new Card('a'),
12      'b' => new Card('b'),
13    ];
14  }
15
16  public static function getSharedInstance() {
17    if (!(self::$_shared instanceof CardRegistry)) {
18      self::$_shared = new CardRegistry();
19    }
20
21    return self::$_shared;
22  }
23}

index.php

CardRegistryのシングルトンインスタンスを使用して、Cardのリストを読み込んで表示しています。

php
1<?php
2require_once './card_registry.php';
3
4$cardRegistry = CardRegistry::getSharedInstance();
5?>
6
7<table>
8  <tbody>
9    <?php foreach ($cardRegistry->cardList as $key => $card): ?>
10    <tr>
11      <th>
12        <?= $key ?>
13      </th>
14
15      <td>
16        <?= $card->name ?>
17      </td>
18    </tr>
19    <?php endforeach; ?>
20  </tbody>
21</table>

投稿2021/11/08 09:19

KAOsaka

総合スコア531

LOTOyamaguchi

2021/11/08 10:51

ありがとうございました。参考にさせていただきました。

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

~~まず、配列内でnewはできないです。~~
※コメントで指摘いただいたのですが配列内でnewできないのはクラスプロパティのみで、クラス外のプロパティであれば可能なようです。
参考：【PHP】クラスプロパティの値には、動的な値を代入することができないようです。~~

イメージとしては以下のような配列をどの画面でも使いたいです。
どのようにすれば、どの画面でも使いまわせるようにできるでしょうか？

画面というより、配列をどのクラスからも使えるようにしたい、ということでよろしいですか？
$cardInstanceList を、どこかのクラス内に定義するかそうじゃないかで話が変わってくるので
クラス内に定義する前提で回答しますね。

オブジェクトを使い回す方法には「static」を使う方法と、「const」を使う方法があります。
staticはオブジェクトをシングルトンとして扱うための修飾子ですが、上書き可能なため危険です。
なのでconst（上書き不可変数とする修飾子）を使って定義する方法が良いかなと思います。

ただしクラスプロパティの配列内ではnewできないため、アプリケーション起動時などでクラスのコンストラクタで、プロパティの配列（$cardInstanceList）にカードの情報をセットしてあげる必要があります。
※クラス外であればnewできるようです。

質問への回答は上記ですが、煩雑になるので、、もし自分が実装するとしたら、
new Card("a") これだけでインスタンス生成できるなら、都度することになってもそれが一番綺麗かなと思いました。
もしくは、cardsパッケージを切って、カードの名前のクラスをそれぞれ作って、プロパティで const でカードの情報を定義
（どんなカードゲームかわからないですが、カードに応じた攻撃力とか防御力とかをconstで定義）
して作るかなと思いました。

どんなカードゲームか分からないので「そうじゃない」となったらすみません。
どんなカードゲームかとか書いておくと回答する人が考え易いと思います。

投稿2021/11/07 08:17

編集2021/11/07 08:38