モデルにおけるデータ記録処理の連携について

Question

現在構築しているwebサービスのシステムにおいて、複数テーブルをまたいだデータの保存方法に悩んでいます。

例題

あるユーザーAさんが商品Iを購入したというデータを新規で記録する場合

• ユーザーデータが存在する

「userテーブル.ユーザーA」

• そのユーザーが商品を買う

「receiptテーブル.商品IのID」

• 購入した商品に好きなタグがつけられる

「tagテーブル.タグ名X」、「tagテーブル.タグ名Y」・・・

といったイメージです。

ここで実際に記録する際、Laravel(Eloquent）上では次のような記述になるかと思われますが、

/* コントローラー上の動作 */

// アイテム情報取得
$item_model = Item::find('アイテムのID');

// ユーザー情報取得
$user_model = User::find('ユーザーID');

// 購入情報記録
$reciept_model = Receipt::create([
    'user_id' => $user_model->id,
    'item_id' => $item_model->id,
]);

// アイテムに対するタグ付け
$tag_model = Tag::create([
    'user_id' => $user_model->id,
    'item_id' => $item_model->id,
    'tag_value' => 'タグ名'
]);

この場合登録ごとにcreateをするのでどこか冗長的な気がします。
そこで「ユーザーは商品を買わないと好きなタグが付けられない」という事から、次のような方法を検討しています。

/* Receiptモデル上 */
protected $tag_value;

public function saveWithTag() {
    self::create([
        'user_id' => $this->user_id,
        'item_id' => $this->item_id,
    ]);
    Tag::create([
        'user_id' => $this->user_id,
        'item_id' => $this->item_id,
        'tag_value' => $this->tag_value,
    ]);
}


/* コントローラー上 */
// アイテム情報取得
$item_model = Item::find('アイテムのID');

// ユーザー情報取得
$user_model = User::find('ユーザーID');

// 購入記録とタグ付けを同時に行う（※変更点※）
$receipt_model = new Receipt();
$receipt_model->user_id = $user_model->id;
$receipt_model->item_id = $item_model->id;
$receipt_model->tag_value = 'タグ名';

$reciept_model->saveWithTag();

ここでお伺いしたいのが、上記の通り

• 「ユーザーは商品を買わないと好きなタグが付けられない」

という点から

• 「購入履歴モデルにはタグ付けをする責務も負わせることが出来る」

という解釈で、今回のようにその他テーブルへの（読み）書きを行うという考えは普通なのでしょうか？

もしくは

• 「1つのモデルは1つのテーブルしか取り扱ってはいけない」

という考えの元、関係性があったとしても連携はあくまでコントローラー上など「別の適切な場所」で行うのが普通なのでしょうか？

宜しくお願いいたします。

Comments

[mikkame]

複数回に渡る同一商品の購入は想定されていますか？

[mcddx330]

はい。同一商品の購入を行った場合も新規で登録が行える（＝追加登録可能）事も想定しています。
「買う度に増える一方では？」という点についてはタグ付を行わないケースや別途編集・削除機能が存在するという見立てで問題ありません。
あくまで複数テーブルを使用する際の記述作法についての内容です。

Answer 1

この場合登録ごとにcreateをするのでどこか冗長的な気がします。

リレーションに対するsave/createを行う事でもう少しスマートにかけるかと思います。
関連したモデルの挿入／更新

そこで「ユーザーは商品を買わないと好きなタグが付けられない」という事から、次のような方法を検討しています

注文ごとにタグをつける事ができるなら、Receiptに直接タグ名を突っ込めば良いかと思います

Comments

[mcddx330]

> リレーションに対するsave/createを行う事でもう少しスマートにかけるかと思います。
なるほどリレーションの手がありましたね。元データから連携して読みに行く印象が強かったので盲点でした。

> 注文ごとにタグをつける事ができるなら、Receiptに直接タグ名を突っ込めば良いかと思います
こちらも考えたのですが、独立して管理するほうが都合がいいケースもあり（例自体が良いものではなかったかもしれません・・・）、あえて分けて考えるようにしていました。
ただ上記点でも十分コードの質的に間違いが起きにくそうなので、こちらをベストアンサーにします。ありがとうございます！