C# と VB.NET の質問掲示板

お世話になっております。

今回「データベースからデータを取得」「データベースの更新」
を行うクラスを作成してみたのですが、少し使いにくいので、皆さんにアドバイスを頂きたく質問致しました。

環境 windows7 visual C# SQLServer2008 Expressになります。

以下、クエリを送ってDataSetを返すメソッドと、DataSet渡してデータベースを更新するメソッドを持つクラスになっています。

namespace ×××
{
    class SQLServerConnection
    {
        readonly static string DataSource = @".\SQLEXPRESS;";
        readonly static string IntegratedSecurity = "True;";
        readonly static string ConnectTimeOut = "30;";

        private string connectionString;

        #region Constructor
        /// <summary>
        /// 接続文字列を作成する。
        /// </summary>
        /// <param name="fileName">mdbファイルのファイル名(フルパス)</param>
        public SQLServerConnection(string fileName)
        {
            this.connectionString =
                @"Data Source = " + DataSource +
                @"AttachDbFilename = " + fileName + ";" +
                @"Integrated Security = " + IntegratedSecurity +
                @"Connect Timeout = " + ConnectTimeOut;
        }
        #endregion

        #region public Method
        /// <summary>
        /// データセットにクエリ結果をセットする。
        /// </summary>
        /// <param name="dataset">空のデータセット</param>
        /// <param name="queryString">クエリ文</param>
        /// <param name="tableName">データテーブル名</param>
        /// <returns>データセット</returns>
        public DataSet SelectRows(string queryString, string tableName)
        {
            using (SqlConnection connection = new SqlConnection(connectionString))
            {
                SqlDataAdapter adapter = new SqlDataAdapter();
                adapter.SelectCommand = new SqlCommand(queryString, connection);
                DataSet dataset = new DataSet();
                adapter.Fill(dataset, tableName);

                return dataset;
            }
        }

        /// <summary>
        /// クエリ結果を元にデータベースの更新を行う。
        /// </summary>
        /// <param name="dataset">空のデータセット</param>
        /// <param name="queryString">クエリ文</param>
        /// <param name="tableName">データテーブル名</param>
        /// <returns>データセット</returns>
        public void SelectSqlRows(DataSet dataset, string queryString, string tableName)
        {
            using (SqlConnection connection = new SqlConnection(connectionString))
            {
                SqlDataAdapter adapter = new SqlDataAdapter();
                adapter.SelectCommand = new SqlCommand(queryString, connection);
                SqlCommandBuilder builder = new SqlCommandBuilder(adapter);
                adapter.Update(dataset, tableName);
            }
        }
        #endregion
    }
}

ただこのクラスですと、DataSetにクエリ結果を取得するまではいいのですが、
もし既にDataSetにクエリ結果が取得されている場合でも、もう一度同じクエリをデータベース更新用メソッドに
渡してやらなければならず、あまり汎用性がないため、クラスを上手変更出来ないものかと思い質問しました。
よろしくお願い致します。

■No56471 (マサヤ さん) に返信
> 呼び出し側の処理をみないとなんともいいずらいのですが、SqlConnection はOpenしなくても問題ありませんでしたか？

マサヤ様お世話になります。

呼び出し側のコードは以下のような感じになっています。

private 何らかのイベント(object sender, EventArgs e)
{
    SampleConnection scn = new SampleConnection(DBFileName);
    DataSet ds = scn.SelectRows("select * from sample", "sample");
    DataTable dTbl = ds.Tables["sample"];
    
    DataRow dRow = dTbl.NewRow();
    dRow["id"] = 1;
    dRow["name"] = "×××";

    dTbl.Rows.Add(dRow);
    scn.SelectSqlRows(ds, "select * from sample", "sample");
}

取りあえずデータベースに更新はされていますが、openしないと問題がありますでしょうか？

Visual Studio 2010
WindowsXP SP3

質問させてください。

下記手順にて画像の「StretchDIBits(MFC)」による
拡大処理を行ったところアルファ値が全ピクセル０になってしまいました。
「StretchDIBits」にてアルファ値の画像を拡大することは可能でしょうか。

[C++/CLI]側
　アルファ付画像を読み込む(Bitmap/Format32bppArgb)
　「Bitmap」から「BitmapData」を取得し、「BitmapData」の「Scan0」を取り出す。
　「Scan0」をMFC側に渡す。

[MFC側]
　伸縮モードの変更「SetStretchBltMode(copyDC, HALFTONE)」
　「Scan0」を「StretchDIBits」を使用して拡大
		srcBMP.bmiHeader.biSize = sizeof(BITMAPINFOHEADER);
		srcBMP.bmiHeader.biBitCount = 32;
		srcBMP.bmiHeader.biPlanes = 1;
		srcBMP.bmiHeader.biWidth = Width;
		srcBMP.bmiHeader.biHeight = Height;

		StretchDIBits(
			copyDC, 0, 0, ZoomWidth, ZoomHeight,
			0, 0, Width, Height,
			srcScan0, &srcBMP, DIB_RGB_COLORS,
			SRCCOPY
		);

　「GetDIBits」を使用して拡大した画像配列を取り出し

		dstBMP.bmiHeader.biSize = sizeof(BITMAPINFOHEADER);
		dstBMP.bmiHeader.biBitCount = 32;
		dstBMP.bmiHeader.biPlanes = 1;
		dstBMP.bmiHeader.biWidth = ZoomWidth;
		dstBMP.bmiHeader.biHeight = ZoomHeight;

		GetDIBits(
			copyDC, dstHBMP, 0, ZoomHeight,
			dstScan0,
			&dstBMP, DIB_RGB_COLORS);


　「dstScan0」のRGBは設定されていますがアルファ値が
　全ピクセル「０」になってしまいます。

　
以上、よろしくお願い致します。

        private void Form1_FormClosed(Object sender, FormClosedEventArgs e)
        {
            try
            {
                System.IO.FileStream fs = new System.IO.FileStream("data.bin", System.IO.FileMode.Create, FileAccess.Write);
                BinaryFormatter bf = new BinaryFormatter();
                bf.Serialize(fs, members);
                fs.Close();
            }
            catch (Exception exception)
            {
                MessageBox.Show(exception.Message);
            }

        }


これを実行すると{"アセンブリ 'report, Version=1.0.0.0, Culture=neutral, PublicKeyToken=null' の型 'report.Member' はシリアル化可能として設定されていません。"}とでるのですが、何が原因なのでしょうか？
ご教授おねがします。

VC++7.1以降のSTLのiteratorに関しての質問です。

std::vectorやstd::mapなどのcontainerを使用する際の
containerのelementのeraseのときは（std::vectorに限りませんが）、たとえば
vector<T>::iterator i = container.begin();
while ( i != container.end() ) {
  if ( 要素を削除したいなら ) {
    i = container.erase( i );
  }
}
とイディオムのように、呪文のようにずっとコードを書いてきました（勉強不足は承知しています）。
しかしstd::vectorの場合、codeを見ていて、i = container,erase( i )は必要がないと理解しました。
std::mapやstd::listは彼のcodeが必要なのも想像できますし、traceしていて確信も得たと思っています。
あくまでVC++7.1以降の話ですが…。

そこで質問なのですが、上記のcodeのi = container.erase( i )は
すべてのSTLのcontainerに通用するものなのでしょうか？
それとも
  if ( 要素を削除したいなら ) {
    container.erase( i );
    ++i;
  }
と書くのがSTLを扱う上でbetterなのでしょうか？

条件を満たすものをコンテナから削除するなら、僕はこうする:

#include <iostream>
#include <algorithm>
#include <numeric>
#include <vector>

using namespace std;

int main() {
  vector<int> iv(10);
  iota(iv.begin(), iv.end(), 0); // iv = { 0, 1, 2 ... 9 }

  // 使用前
  for_each(iv.begin(), iv.end(), [](int n) { cout << n << ' ';});
  cout << endl;

  // 偶数を削除
  iv.erase(remove_if(iv.begin(),iv.end(),[](int n) { return n % 2 == 0;}), iv.end());

  // 使用後
  for_each(iv.begin(), iv.end(), [](int n) { cout << n << ' ';});
  cout << endl;
}

...てか、こんな話じゃないのかしら?

■No56489 (επιστημη さん) に返信

あー…。
そうですね。containerを操作する適切なメソッドを使えば
containerの枝葉の違いを吸収してくれますね。
基本的なことに目を向けていませんでした…。

目が覚めました。
ありがとうございました。