## 環境 ```xml org.springframework.boot spring-boot-starter-parent 2.0.1.RELEASE ``` ## 以下のページを見ました https://sites.google.com/site/soracane/home/springnitsuite/spring-mvc/04-ji-ben-gai-nian-controllerno-chu-lifuro ## サンプルとして載っているコード ```java @ModelAttribute("user") //②モデルの前処理 public User newRequest(@RequestParam(required=false, value="id") String id) { if(id== null) return null; return this.accountService.get(id); } ``` ## 以下が無駄だと思う理由です > （１）．@ModelAttributeの呼び出し　　：　事前準備として、DBからPOJOを取得して、Model に登録します。 > （ModelAttributeメソッドでDBからPOJOを取得し、RequestMappingメソッドを呼び出すときにSpringが自動的に画面の値で必要なPOJOフィールドを更新する） - 調べてみたら確かに`@RequestMapping`の前に必ず`@ModelAttribute`が呼ばれて初期化する事を確認しました。 - また、説明通り`@RequestMapping`で受け取る`form`は画面の値で上書きされていました。画面の値で上書きするのであれば毎回初期化するのは無駄に感じます。 ※特にDBにアクセスする場合は毎回SQL発行する事になる為また、`if(id== null) return null;` に関して、初期化が必要ない場合は`id`が入ってこないから`null`を返しているのかと考えましたが、'@ModelAttribute'で`return null;`してしまうと`@RequestMapping`の引数で受け取る`form`が`null`になってしまうためvalidate処理や画面表示が正しく行われない為目的は違うんだろうと判断しました。みなさん気にならずにそのまま実装しているのでしょうか、または別の方法で実装されているのでしょうか。 ## 以下ならまだわからないでもないです ```java @ModelAttribute("user") //②モデルの前処理 public User newRequest( @RequestParam(required=false, value="id") String id, User form) { if(id== null) return form; User user = this.accountService.get(id); BeanUtils.copyProperties(user, form); return form; } ```

	@@ -1 +1 @@
1	- SpringMVCのModelAttributeで初期化~~処理に~~無駄~~な箇所があるよう~~に感じます。
1	+ SpringMVCの@ModelAttributeで毎回初期化されるのが無駄に感じます。

質問編集履歴