データを管理するクラスの作り方

Question

商品データベースの商品画像を管理するクラスを作りました。

ある商品に関連づけられる画像には区分で分類されています。
（画像区分2種 × 重要度3種 = 6パターン）

【画像区分】
共通・個別

【重要度】
高・中・低

//画像管理クラス
class ImageManager {

    //画像データ
    private $_image_data = [];

    //出力順序
    private $_output_order;

    public function __construct(){

        $this->_image_data = [
            'common' => [
                'imp_high'   => ['images' => [] ],
                'imp_middle' => ['images' => [] ],
                'imp_low'    => ['images' => [] ]
            ],
            'individual' => [
                'imp_high'   => ['images' => [] ],
                'imp_middle' => ['images' => [] ],
                'imp_low'    => ['images' => [] ]
            ],
        ];

        //出力の順序を決める
        $this->_output_order[] = &$this->_image_data['common']['imp_high'];
        $this->_output_order[] = &$this->_image_data['common']['imp_middle'];
        $this->_output_order[] = &$this->_image_data['common']['imp_low'];
        $this->_output_order[] = &$this->_image_data['individual']['imp_high'];
        $this->_output_order[] = &$this->_image_data['individual']['imp_middle'];
        $this->_output_order[] = &$this->_image_data['individual']['imp_low'];

    }

    public function get_images()
    {
        foreach($this->_output_order as $images)
        {
            foreach($images['images'] as $img)
            {
                yield $img;
            }
        }
    }
    //画像を追加
    public function add($img_kbn, $importance,$image)
    {
        $this->_image_data[$img_kbn][$importance]['images'][] = $image;
    }

上記プログラムはなかり簡略化してしまったので間違っているかもしれませんが、
目的は区分/重要度別に画像を保持することと取り出しの順序の管理です。

データの入れ物は配列で作ってしまいました。
画像区分と重要度をキーにして必要なデータにアクセスできるので便利です。

出力順はコンストラクタで$this->_output_orderの配列に参照を入れています。
仕様変更で出力の順を変えたくなったらここを変更すればいいと思ったからです。

今のところ問題なく動いていますが、配列や参照を使ったこのプログラムは古いように思えます。

現在プログラムの見直しをしていまして、配列ではなくオブジェクトを使った分かりやすい実装に変えることができないか検討しています。

ここからが質問です（前置き長くてスイマセン）

画像を入れておく配列を下記クラスと置き換えてみました。

class ImageBox {
    private $img_kbn;
    private $importance;
    private $images = [];

    public function __construct($img_kbn,$importance){
        $this->img_kbn = $img_kbn;
        $this->importance = $importance;
    }
    public function add($image){
        $images[] = $image;
    }
    public function get(){
        return $images;
    }
}

最初のプログラムは下記となります。

//画像管理クラス
class ImageManager {

    //画像データ
    private $_image_data = [];

    //出力順序
    private $_output_order;

    public function __construct(){

        $this->_image_data = [
           new ImageBox('common','imp_high'),
           new ImageBox('common','imp_middle'),
           new ImageBox('common','imp_low'),
           new ImageBox('individual','imp_high'),
           new ImageBox('individual','imp_middle'),
           new ImageBox('individual','imp_low'),
        ];

        //出力の順序を決める
        $this->_output_order[] = $this->_image_data[0];
        $this->_output_order[] = $this->_image_data[1];
        $this->_output_order[] = $this->_image_data[2];
        $this->_output_order[] = $this->_image_data[3];
        $this->_output_order[] = $this->_image_data[4];
        $this->_output_order[] = $this->_image_data[5];
    }

    public function get_images()
    {
        foreach($this->_output_order as $images)
        {
            foreach($images->get() as $img)
            {
                yield $img;
            }
        }
    }
    //画像を追加
    public function add($img_kbn, $importance,$image)
    {
        //ここがわかりません
    }

ImageManagerクラスのaddメソッドについて質問ですが

引数で与えられた$img_bknと$importanceから該当するImageBoxオブジェクトを選択して画像を追加したいのですが、どのようにImageBoxオブジェクトを探すのがいいでしょうか？

foreachで回しながら1つ1つ確認する方法しか思いつきません。

これならキーでアクセスできる配列の方がわかりやすい感じがしますが、データのチェックなど配列ではできない処理を実装する予定なので配列はやめようと思います。

よろしくお願いします。

Comments

[mather]

「配列ではなくオブジェクト」と言っている「オブジェクト」とは何のことですか？連想配列なのか、別のクラスなのか。

[m.ts10806]

用語の定義がごっちゃになっているように思うのでなるべくPHPマニュアルにあるような形で用語の意味を統一してください。　例：http://php.net/manual/ja/language.oop5.basic.php

[msx2]

質問がわかりにくくて申し訳ありません。質問したいポイントを絞ってみました。

[matsuand]

回答ではないですが、$this->_output_order[] = $this->_image_data[0]; と書いている時点で、ImageManager は、配列のゼロ番目に 'common' の 'imp_high' が入っていることが分かっているわけですから、これを許容するのであれば、add メソッドでも、'common' の 'imp_high' であれば、ゼロ番目に追加するという作りになると思います。