データをファイルで管理しています。

Question

商品マスタと受注データを以下のファイルで管理しています。

【item.csv】
1,たまご
2,牛乳
3,ピーマン

【order.csv】
2,400
3,100
1,8
2,20

index.phpは以下の実装です。

//受注データ読み込み
$file = explode("\n", file_get_contents("order.csv"));
foreach ($file as $line) {
	//商品コード、数量
	list($itemCd, $suryo) = explode(",", $line);
	//インスタンスの配列
	$orderList[] = new Order($itemCd, $suryo);
}


foreach ($orderList as $order) {
…以下略…

//Orderクラス
class Order {
	private $itemCd;//商品コード
	private $itemName;//商品名
	private $suryo;//数量

	function __construct($itemCd, $suryo) {

		$this->itemCd = $itemCd;
		$this->suryo = $suryo;

		//商品マスタから商品名を調べる
		$file = explode("\n", file_get_contents("item.csv"));
		foreach ($file as $line) {
			list($cd, $name) = explode(",", $line);//商品コード、商品名
			if ($cd === $itemCd) {
				$this->itemName = $name;
				break;
			}
		}
	}
}

これでも動くのですが、Orderクラスをnewするたびにitem.csvを読み込みに行ってしまいます。
もっと効率のよい書き方はないでしょうか。
よろしくお願いします。

Answer 1

class Items {

    private static $instance;
    private $items = array();

    public static function getInstance()
    {
        if (self::$instance === null){
            self::$instance = new self;
        }

        return self::$instance;
    }

    private function __construct()
    {
        $file = explode("\n", file_get_contents("item.csv")); 
        foreach ($file as $line) { 
            list($cd, $name) = explode(",", $line);//商品コード、商品名 
        $this->items[$cd] = $name;
    }

    public getItems()
    {
        return $this->items;
    }
}


$items = Items::getInstance()->getItems();
 

あとは全く同じ。

こんな感じでどうでしょう？

Comments

[退会済みユーザー]

そうすると$itemsの引き渡しは必要なくなるから

$file = explode("\n", file_get_contents("order.csv"));
foreach ($file as $line) {
//商品コード、数量
list($itemCd, $suryo) = explode(",", $line);
//インスタンスの配列
$orderList[] = new Order($itemCd, $suryo);
}

//Orderクラス
class Order {
private $itemCd;//商品コード
private $itemName;//商品名
private $suryo;//数量

function __construct($itemCd, $suryo) {

$this->itemCd = $itemCd;
$this->suryo = $suryo;
$items = Items::getInstance()->getItems();
$this->itemName = isset($items[$itemCd])? $items[$itemCd] : "";

}
}

こっちか。

[jk233]

やりたかったことと完全に一致しました。
ありがとうございました。

Answer 2

幾らでもやり方はありますが、とりあえずI/Oを最初に１回だけして、あなたのコードを活かすとこんな感じ。

$items = array();
$file = explode("\n", file_get_contents("item.csv"));
foreach ($file as $line) {
list($cd, $name) = explode(",", $line);//商品コード、商品名
$items[$cd] = $name;
}

$file = explode("\n", file_get_contents("order.csv"));
foreach ($file as $line) {
//商品コード、数量
list($itemCd, $suryo) = explode(",", $line);
//インスタンスの配列
$orderList[] = new Order($itemCd, $suryo, $items);
}

//Orderクラス
class Order {
private $itemCd;//商品コード
private $itemName;//商品名
private $suryo;//数量

function __construct($itemCd, $suryo, $items) {

    $this->itemCd = $itemCd;
    $this->suryo = $suryo;
    $this->itemName = isset($items[$itemCd])? $items[$itemCd] : "";

}

}

PHPの場合、変数引き渡しは全て参照で行われ、内容を変更する場合に複製が作られてメモリを消費するはずなので、この場合はずっと参照で行われ、メモリは最初の１回分だけのはずです。

まぁ、ファイルで運用なんかしないでDB使いたいですね。

Comments

[jk233]

ご回答ありがとうございます。
index.php側には商品マスタを見に行く処理を隠蔽したいのですが、そういったことは可能でしょうか？

>ファイルで運用なんかしないでDB使いたいですね。
仰るとおりで...

[退会済みユーザー]

そうすると、シングルトンモデルになりますね。

[退会済みユーザー]

コード書くので別スレッドにしました。