C#で、DBへのクエリ発行用メソッドはどう管理するべきでしょうか

Question

前提・実現したいこと

クエリ発行用メソッドを分かりやすく管理したいのですが、
どうやって管理するのが一般的または王道なのでしょうか。

発生している問題

現在、以下のようにフォームやクラスごとに、それぞれが使用するクエリをまとめたクラスを作っています。

class MyForm : Form {
  public void Method() {
    MyFormQueries.InsertSomething()
  }
}
//MyFormで使うクエリは全部MyFormQueriesクラスに記述
class MyFormQueries {
  public static void InsertSomething() {
    //クエリ発行
  }
}


class MyClass {
  public void Method() {
    MyClassQueries.InsertSomething()
  }
}
//MyClassで使うクエリは全部MyClassQueriesクラスに記述
class MyClassQueries{
  public static void InsertSomething() {
    //クエリ発行
  }
}

問題点

得られたメリットは、各フォーム・クラスから送るクエリを変えたい時に、該当のフォーム・クラス名を見ればすぐにクエリが見つかり、
そのクエリを変えても他のフォーム・クラスに影響を与えない点です。

デメリットは、色々なクラスに、全く同じや似たようなクエリメソッドが出来てしまいました。
また、テーブル設計変更があった場合、該当のテーブルにアクセスするクエリが分散しすぎて、把握出来なくなりそうな点です。

試したこと

テーブル単位で、そのテーブル宛に発行するクエリを集中管理するクラスを作り、そこにクエリを全部入れました。

class MyForm : Form {
  public void Method() {
    UserDataTableQueries.InsertSomething()
  }
}

class MyClass {
  public void Method() {
    UserDataTableQueries.SelectSomething()
  }
}

//UserDataテーブルのデータにアクセスするクエリは全部ここに登録
class UserDataTableQueries
{
  public static void InsertSomething() {
    //クエリ発行
  }
  public static IEnumerable SelectSomething() {
    //クエリ発行
  }

  //大量のクエリメソッド
}

問題点

得られたメリットは、テーブル設計変更しても、そのテーブルにアクセスするクエリ用メソッドを容易に探せることです。
※JOINするクエリの場合は少し気を使いますが・・・

デメリットは、1つのクラスに(事実上、多数のフォーム・クラス固有の)メソッドが集中しすぎて見辛くなり、
使うクエリと使わないクエリが分かり難くなりました。

Answer 1

まず「どう管理すべき」という問いに対して正解はありません。

あなたのプロダクトが最も保守しやすく、無駄を減らしてバグを発生させないように作れるのであれば、それが「あなたにとっての正解」でしょうけど。

そもそもプロダクトの特性によって最適な構造は変わってくるでしょう。
データの入出力の単位が画面と完全に１：１になるようなアプリケーションであれば、１画面ごとに１つのデータアクセスモジュールを作成するのは良い方法かもしれません。

ただ、複数の画面から同じリソースを参照・更新するようなアプリケーションの場合は、リソース単位でデータアクセスモジュールが別れている方が明らかに重複を減らせます。

また、データアクセスモジュールとテーブルを１：１にするかどうかも、正規化されたデータとアプリケーションが利用したいデータが１：１であればそれでいいでしょうけど、そうでない場合は無駄が多くなるでしょう。

その場合、データベース構造とは切り離してもう１段階抽象化したアプリケーションの扱うリソースの単位でデータアクセスモジュールを作成した方が良いでしょう。

繰り返しますが、こういったアプリケーションの構造設計は「これが正解」というのはないので、あなたが判断して最適な構造を設計してくださいという回答になります。