Webアプリ、バッチのコードマスタの読み込み方法について

Question

現在社内でJavaの勉強中で、サンプルECアプリを作っています。

アクセスするテーブルで、コードと、名称を格納したコードマスタテーブルがあります。
（例：注文ステータス、注文種別、発送区分など）
100レコード程度格納されています。

注文テーブルには上記項目がコードで格納されていて、
サーブレットから画面(JSPです)に表示したり、
別途作成したバッチプログラムからCSVに出力しようとしています。

そこで、何度もコードマスタを読み込みたくないので、
私はコードマスタの内容をメモリ上にキャッシュしておく共通部品を作ろうと思いました。

目的
0. 番号リスト画面上のセレクボックスなどを構築する。
0. 一覧表示時に、JSPで共通部品を使用してコードを名称に変換する。
0. CSV出力時に共通部品でコードを名称に変換する。

トレーナーに上記の件を相談したところ、「1.」については賛成してもらったものの、
「2. 3.」 については、そんな共通部品など使わず、注文を取ってくるSQLで一緒にコードマスタを
取って来いと指導されました。余計なことするなと怒られてしまいました。
ですので、それ以上聞くことができませんでした。

SQLが簡潔になって良いと思ったのですが、「2. 3.」の案が駄目な理由を教えてください。

Comments

[hentaiman]

金貰いながら現場の経験者に教えて貰える立場なんだから、まずは言われたもの作りましょう。それから自分の考えを含めたものを作りましょう。二つ作ればいいだけです。
そうすれば怒る事なく逆に親切に色々教えてくれるでしょう。
なんなら２，３だけでなく１も不要です。教える側からしたら教育の為の指示を実行もせずに１に答える事もくだらねーめんどくせーって思われていてもおかしくありません。
本気で実力をつけたいのならまずは言われたことをやってから自分の考えを追加したものを作りましょう。

ただし、オレ様アイデアではなく指示内容自体で理解が出来ずに進められない個所は指導係にどんどん質問しましょう。

Answer 1

コードマスタの内容をメモリ上にキャッシュしておく共通部品

この設計次第ではあるのですが、考えられる問題点としては次のようなものでしょうか。

• 現状コードマスタは100件程度とのことですが、これが10倍、100倍と増えた場合、メモリを大量に消費してしまう可能性がある。
• コードマスタが更新されたにも関わらず、キャッシュ側の情報が正しく更新されず、利用者に不正確な情報を提供する可能性がある
• SQLの発行回数が増えて、SQLの発行そのものがボトルネックになってしまう (cf.N+1問題)

SQLが簡潔になって良い

これは実際のアプリケーション開発において、とても大事な観点です。ただ、注文テーブルとコードマスタをJOINにより結合する程度であれば、それほど複雑なSQLではないとは思います。

Answer 2

余計なことするなと怒られてしまいました。

単に2、3が要件に反するのではないでしょうか。
この後の教育を進める上で都合が悪い仕様が入ってしまうとか。
（とはいえ、それ程度なら怒らずに理由を言って質問者さんに取り下げてもらう方が角が立たないよな…）

私はコードマスタの内容をメモリ上にキャッシュしておく共通部品を作ろうと思いました。

これは私も場合によっては却下しますね。
商品メンテナンスでDB上のマスタに更新が入る場合、
キャッシュがある事で不具合が起こるパターンを考慮する必要があります。

2、3を実施した時に、マスタを更新したのに
CSVに出したデータが古いぞ！

など