ActionとActionFormの記述

質問

strutsを使用してデータベース接続を行っています。

ActionFormにはgetter、setterのみを記述、
Actionにはデータベース接続の確立、SQL文の取得、などを記述、

ActionFormにはgetter、setter、データベース接続の確立などを記述、
ActionにはfindForwardのみを記述、

現在、上記２通りの記述方法を使用しています。
そこで疑問なのですが、このまま２通りの記述方法を
使用するか、一つにするかを考えています。
特に、使用感に問題も無いのですが少々気になりました。
好みの問題等もあるのでしょうか。
よろしくお願い致します

回答No.2

sire · ベストアンサー

ご指摘のように好みの問題かもしれませんが、
私は前者です。

ActionFormはあくまでFormとしてgetter、setterのみを定義して、
ロジックはロジックとして、Actionに持たせたほうがよいと思います。

もちろんアクションクラスにベタには書きませんが、
ロジッククラスを作っておいて、
アクションはロジッククラスを呼び出し、
ロジッククラスが、DB接続、SQL文発行しフォームに取得した値を設定し、
最後にアクションがフォワードする

といった形でしょうか。

このあたりは、No１さんもおっしゃるとおり
Strutsの各クラスの役割を理解して
明確にコーディング規約を作っておいたほうがよさそうですね。

お礼

回答ありがとうございます。
アクションとフォームの役割をはっきりさせます。
その中でルールを定めます。

回答No.3

noname#45950

こんにちは。

私も前者の方です。
もっともgetter/setter以外に、formに関するflag等の設定処理が若干入っていたりもしますが。

でも、現状一番問題なのは「２つが混在していること」じゃないでしょうか？
特に「データベース接続の確立」の処理が記述されている場所がばらばらっていうのは、リファクタリング最優先事項だと思います。

お礼

回答ありがとうございます。
やはり、統一させるべきなのですね。

皆様の言うとおり、ルール作りをはっきりとさせます。
この場を借りて、再び皆様にお礼を言わさせてもらいます。
どうも、ありがとうございました。

回答No.1

noname#19197

で、結局何が聞きたいんでしょうか？
どっちがいいかアドバイスが欲しいということでしょうか？

それなら、どちらか一方に決めた方がいいと思います。
ある程度のルールがないと煩雑になりメンテナンス性が悪くなります。

もうひとつ言うと、ActionFormはパターンでいうところのValuObjectです。
なので、ActionFormにビジネスロジックは含めるべきではないですね。

何の為にStrutsを使っているのか、MVCモデルとは何かを調べてみてください。

お礼

こちらの言葉足らずで申し訳ありません。
StrutsやMVCモデルについて調べなおし、
ルールを定めようと思います。
まだまだ勉強不足だと痛感しました。
回答ありがとうございました。