> このデータ作るためにforeachが多すぎる気がしちゃいます。。

全てのエンティティクラスで同様に記述するなら、確かに多いような気はしますね。

ただ、ざっと見た印象では、ある程度は基底クラス化できそうなので、相対的な記述数は削減できるのかも。

あと、SC_Entity_OrderDetail#getCategoryIds の foreach あたりは、冗長に書いてしまったポカミスかなとも。


> update分も逆に冗長になってるような

確かにサンプルだけ見ると、そんな印象はありますね。

現実的に重要なのは、もっと実際の実装のように、込み入った時にどうなるかでしょうか。


> updateしたい値以外不要な情報ですよね

最近は、DB の処理が最適化される方向なので、この辺りの心配は軽視できそうです。

そうでなくても、エンティティクラスサイドで、差異のあるカラムのみ出力するとかは簡単にできそうな気もします。

----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。