MVC：私のコントローラーは半分の時間、役に立たないようです。これは問題ですか？

多くの場合、MVCを使用してプログラムを設計すると、コントローラーが半分の時間無駄になります。

つまり、ビューで何かが発生します（ボタンのクリックなど）。次に、ビューはコントローラーに通知します。次に、コントローラーはモデルに直接委任し、何もする必要がないため、他に何もしません。

例えば：

ユーザーはボタン「カラーブルー」を押します>ビューはコントローラーに通知しますcontroller.colorBlue()>コントローラーはモデルに通知しますmodel.colorBlue()>モデルの色が何か青いです。

この例では、コントローラーは役に立たないようです。それは何も追加しません。ビューはモデルに直接話しかけたかもしれません。

ただし、残りの半分の時間は、コントローラーがビューとモデルの間で何らかのメディエーションを行います。

私の質問はこれです：これはMVC構造でどのくらい一般的ですか？コントローラの半分の時間が不要であるように見えるのは理にかなっていますか？それともこれは問題ですか？これは一般的ですか？私はこれにどのように取り組むべきですか？

私の質問が十分に明確でない場合は、そのように言ってください。

ユーザーインターフェイスとモデルの間に抽象化の層を設けることの重要性を過小評価しています。コントローラーは、この機能を100％実行します。

あなたの例model.colorBlue()は少し怪しげです。実際のモデルでは、これはおそらくCRUDメソッドになります。したがって、ボタンは[Create Customer]ボタン、コントローラーメソッドはCreateCustomer()、モデルはになりますCreateCustomer()。確かに、あなたはただ通話を通過しているだけです。

しかし、モデルの動作方法を変更する必要がある場合はどうでしょうか。ビューがモデルを直接呼び出している場合、モデルを変更するとアプリケーションが壊れます。コントローラメソッドは、ビューに「アクセスポイント」を提供します。おそらくModel呼び出しをCreateCustomerWithVerification()に変更することで、コントローラーメソッドに簡単な変更を加えることができ、すべてが引き続き機能します。

同じ理由がサービス層を持っているために適用されます。モデルに単にCRUDメソッドを含めるのではなく、ビジネスアクションを用意する必要があります。このようにして、ビジネスロジックをコントローラーの外に置き、モデルをWPFアプリケーションなどの別の場所で使用できるようにします。

コントローラを「スイッチヤード」と考えてください。これは、UIとモデルの間の仲介的なリクエストである必要がありますが、コントローラーメソッドのロジックはできるだけ少なくする必要があります。

コーディング中は、次のように考えることができます。

• ビューを別のもの（コンソール、サービス）に変更したり、同じコントローラーを別のビューに変更したりできますか？
• モデルを変更して別のデータベースを処理したり、外部サービスに書き込んだりできますか？

あなたの場合、ドメインロジックをモデルに入れているようで、それを変更したい場合に問題が発生します： "model.colorBlue"のようなメソッドを新しいモデルにコピーする必要があります。

そして、「青」の定義が変わったらどうなるでしょうか？2つのモデルに変更する必要があります。また、コントローラでは、データベースに直接書き込むべきではありませんが、Lie Ryanが指摘したように、を使用する必要がありますmodel.setColor。

ビューと同じです。ロジックまたは検証をビューに配置し始めた場合、ビューを変更したい場合は、そのすべての機能をコピーする必要があります。