データベース処理のプログラミングについて

質問

データベースを処理するためのプログラムを作成していますが、そのコーティングで悩んでいます。
説明が長くなってしまいますが、お付き合いいただける方、ご教示頂ける方がいらっしゃることを、切に願います。

まずは現在の状況説明ですが、下記のような感じでプログラムを作っています。

データベースに、Table1があります。
・Table1・・・fld1～fld10までの10個のフィールドがあり、fld1が主キー

また、このテーブルを処理するためのクラスがあります。
・データベース処理系のクラス・・・clsTable1

clsTable1では、各フィールドを、メンバ、プロパティとして作ってあります。
・メンバ：_fld1、　プロパティ：fld1　・・・　メンバ：_fld10、　プロパティ：fld10

また、Table1に対してSQLを発行するためのメソッドがあります。
・SelectRecord・・・主キーで検索して1件だけ返すメソッド（検索のためfld1が必要）
・InsertRecord・・・Insert用メソッド（fld1～fld10の全てが必要）
・UpdateRecord・・・主キーで検索し、該当するレコードのfld2を更新するメソッド（検索値のfld1と更新値のfld2が必要）

また、あるフィールドの範囲検索などをするメソッドとして、
・SelectRecords・・・fld3を範囲検索して複数レコードを返すメソッド（fld3をBETWEEN演算子で検索するためfld3minとfld3maxが必要）
を作成した場合は、fld3minとfld3max用のメンバとプロパティを追加しています。

これらのメソッドは全て引数なしになっていて、フォームなどから使う場合は必要なプロパティに値を代入してから各メソッドを呼んでいます。
そのため、メソッドを呼び出す際には、どのプロパティを設定する必要があるのかが分からないです。
（作っている時は記憶で、メンテの時はコメントやソースを見て判断しています。）

これを改善するために、設定が必要なフィールド全てを各メソッドの引数にしてしまえば良いのかとも思ったのですが、そうするとInsertRecoedの引数は10個になります。
仮にフィールド数が100列のテーブルだったりした場合は引数が100個になってしまうので、それもダメだろうと思っています。
Insert以外でも、詳細な検索などが必要な場合や、全フィールドのUpdateの場合にも、引数が多くなりすぎです。

どんな感じにすればよろしいのでしょうか？
また、みなさんは、どんな感じにされているのでしょうか？

開発環境：
SQL Server 2000 Standard
Visual Basic 2010 Express

回答No.1

めとろいと（@naktak） · ベストアンサー

正直、どの値を設定すべきかどうかは、メソッド名や詳細設計、テーブル設計を
見れば分かることかと思います。

それを前提とした上でDTOクラス、DAOクラスを作ってやりとりするとかすれば、
引数は最小限に抑えられますよ。
ビジネスロジックからはDTOクラスに必要な項目を設定し、DAOクラスに
行って欲しい処理を投げ、DAOクラスがDTOクラスを利用して検索・登録・更新・削除
を行うイメージです。

ここの図が分かりやすいかもです。
http://www.syboos.jp/sysdesign/doc/20080713213412806.html

いまからそんな大掛かりなこと出来ない！
って思っても、DTOはアクセサさえあればいいわけですから、やれないこともないです。

問合せ結果などもちゃんとDTOを使うかは別にして、引数を減らしたいというだけの
要件からは、DTOもどきのただのアクセサを設けて、3個以上引数が必要な場合は
そのアクセサクラスを引数にするとか決め事を作ればいいです。

但し、SELECT結果を条件に応じてUPDATE、などといった場合、DTOを使ってないと、
いちいちキー情報なども設定し直しが発生するので、それが面倒かつミスに繋がる
といったところでしょう。

Visual Studio 2010だと、デザイン時に予めデータソースを定義して利用することで
そういった実装を行うことが出来ます。確か。
ただ、接続先のDBやらの環境周りが決まりきった上での実装になると思いますが。

お礼

回答ありがとうございます。

> 正直、どの値を設定すべきかどうかは、メソッド名や詳細設計、テーブル設計を
> 見れば分かることかと思います。

基本的には作るのもメンテするのも自分だけなので問題は無いのですが、後任が見た時のことも考えた方がいいかなと思い始めました。
でも、それほど気にしなくてもいいみたいですね。

DTOクラス、DAOクラスについては、参照urlを見ても？？？です。
このキーワードで勉強してみます。

回答No.2

Siegrune

引数で作ったり、プロパティで作ったり、Publicの変数を使ったり、いろいろです。

引数で作ったときは、
>設定が必要なフィールド全てを各メソッドの引数にしてしまえば良いのかとも思ったのですが、
>そうするとInsertRecoedの引数は10個になります。
という問題点を防ぐために、配列を引数として渡すこともしていました。
・・・欠点は型がばらばらなときに、すべて文字列型にしていったん配列に格納し、
　SQLを組み立てするときに、Castをするのが面倒という点。
　例）日付型の項目にInsertしたいときは、
"Insert into ××× Values(・・・,cast('" & ParaData(12) & "',112),・・・"
　みたいにしていました。
　当時は１テーブル毎に作っていたので。
　（今ならテーブルの属性を読み込んで汎用的にできるかもとも思いますが。）

お礼

回答ありがとうございます。

配列で渡すってことは、必要な項目の数は分かっても、何を必要としているかまでは分からないですよね。
oop的に考えてメソッドの中身を意識せずに使えるようにしなきゃいけないのかなと思っていたのですが、naktakさんの回答も併せて見ると割り切ってしまってもいいような気がしてきました。
もう一度、何が必要か考えてみます。