0

この SqlCe コードは、私には非常に奇妙に見えます。

cmd.CommandText = "INSERT INTO departments ( account_id, name) VALUES (?, ?)";
foreach(DataTable tab in dset.Tables)
{
    if (tab.TableName == "Departments")
    {
        foreach(DataRow row in tab.Rows)
        {
            Department Dept = new Department();
            if (!ret)
                ret = true;
            foreach(DataColumn column in tab.Columns)
            {

                if (column.ColumnName == "AccountID")
                {
                    Dept.AccountID = (string) row[column];
                }
                else if (column.ColumnName == "Name")
                {
                    if (!row.IsNull(column))
                        Dept.AccountName = (string) row[column];
                    else
                        Dept.AccountName = "";
                }
            }
            List.List.Add(Dept);
            . . .
            dSQL = "INSERT INTO departments ( account_id, name) VALUES ('" + Dept.AccountID + "','" + Dept.AccountName +"')";

            if (!First)
            {
                cmd.Parameters[0].Value = Dept.AccountID;
                cmd.Parameters[1].Value = Dept.AccountName;
            }

            if (First)
            {
                cmd.Parameters.Add("@account_id",Dept.AccountID);
                cmd.Parameters.Add("name",Dept.AccountName);
                cmd.Prepare();
                First = false;
            }

            if (frmCentral.CancelFetchInvDataInProgress)
            {
                ret = false;
                return ret;
            }

            try
            {
                dbconn.DBCommand( cmd, dSQL, true );
            } 
            . . .

    public void DBCommand(SqlCeCommand cmd, string dynSQL, bool Silent)
    {
        SqlCeTransaction trans = GetConnection().BeginTransaction();
        cmd.Transaction = trans;
        try
        {
            cmd.ExecuteNonQuery();
            trans.Commit();
        }
        catch (Exception ex)
        {
            try 
            {
                trans.Rollback();
            }
            catch (SqlCeException) 
            {
                // Handle possible exception here
            }
            MessageBox.Show("DBCommand Except 2"); // This one I haven't seen...
            WriteDBCommandException(dynSQL, ex, Silent);
        }
    }

私の質問は次のとおりです。

1) 「?」cmd.CommandText への代入で実際に使用するか、代わりに「@」を使用する必要がありますか?

2) 「cmd.Parameters.Add()」の 1 つ (account_id) は「@」を使用し、他の (name) は使用しません。どちらが正しいですか、それとも「@」はオプションですか?

3) DBCommand() がそのまま書かれている理由について、頭も尻尾もわかりません - 最後の 2 つの引数は、例外が発生した場合にのみ使用されます...???

とても奇妙に思えるので、このコードを根本的にリファクタリングしたくなりますが、よく理解していないので、それは災害のレシピになる可能性があります...

4

2 に答える 2

1

この記事があなたの質問のいくつかに答えると確信しています。

http://msdn.microsoft.com/en-us/library/yy6y35y8.aspx

2 番目のグラフは、名前付きパラメーターと位置 (?) パラメーター (OleDb および ODBC で使用される) の違いを説明しています。

私は？が使用されている場合、@ はオプションですが、これについてはよくわかりません。それが機能している場合、私はそれが事実であると言うでしょう.

DBCommand の内容は、単にログ記録のために存在しているように見えます。実行が失敗すると、ロールバックが試行され、sql コマンドで例外がログに記録されます ( をdynSQL参照)。

于 2013-04-01T18:55:40.980 に答える
0

?パラメータは古い Access 構文です。

私の推測では、これは以前は Access データベースでしたが、ある時点で誰かが SQL CE に変換したものです。

一般に、SQL はその?パラメーターを理解しますが、より理解できるように、そこにいる間だけ変更することをお勧めします。

私はまだこれらすべての変数の頭と尾を作ろうとしています。整理できたら、コンパイル可能な (sp?) コードを投稿します。

編集:これをメソッドに入れて、すべてのREDエラーを解決して、コンパイルできないものを提供していないことを確認する必要がありました。

たくさんのコメントを追加して、次のように DataSet を渡しました。

private bool StrangeSqlCeCode(DataSet dset) {
  const string ACCOUNT_ID = "AccountID";
  const string DEPARTMENTS = "Departments";
  const string NAME = "Name";
  const string SQL_TEXT = "INSERT INTO departments (account_id, name) VALUES (@account_id, @name)";
  bool ret = false;
  //bool First = false; (we don't need this anymore, because we initialize the SqlCeCommand correctly up front)
  using (SqlCeCommand cmd = new SqlCeCommand(SQL_TEXT)) {
    // Be sure to set this to the data type of the database and size field
    cmd.Parameters.Add("@account_id", SqlDbType.NVarChar, 100);
    cmd.Parameters.Add("@name", SqlDbType.NVarChar, 100);
    if (-1 < dset.Tables.IndexOf(DEPARTMENTS)) {
      DataTable tab = dset.Tables[DEPARTMENTS];
      foreach (DataRow row in tab.Rows) {
        // Check this much earlier. No need in doing all the rest if a Cancel has been requested
        if (!frmCentral.CancelFetchInvDataInProgress) {
          Department Dept = new Department();
          if (!ret)
            ret = true;
          // Wow! Long way about getting the data below:
          //foreach (DataColumn column in tab.Columns) {
          //  if (column.ColumnName == "AccountID") {
          //    Dept.AccountID = (string)row[column];
          //  } else if (column.ColumnName == "Name") {
          //    Dept.AccountName = !row.IsNull(column) ? row[column].ToString() : String.Empty;
          //  }
          //}
          if (-1 < tab.Columns.IndexOf(ACCOUNT_ID)) {
            Dept.AccountID = row[ACCOUNT_ID].ToString();
          }
          if (-1 < tab.Columns.IndexOf(NAME)) {
            Dept.AccountName = row[NAME].ToString();
          }
          List.List.Add(Dept);
          // This statement below is logically the same as cmd.CommandText, so just don't use it
          //string dSQL = "INSERT INTO departments ( account_id, name) VALUES ('" + Dept.AccountID + "','" + Dept.AccountName + "')";
          cmd.Parameters["@account_id"].Value = Dept.AccountID;
          cmd.Parameters["@name"].Value = Dept.AccountName;
          cmd.Prepare(); // I really don't ever use this. Is it necessary? Perhaps.
          // This whole routine below is already in a Try/Catch, so this one isn't necessary
          //try {
          dbconn.DBCommand(cmd, true);
          //} catch {
          //}
        } else {
          ret = false;
          return ret;
        }
      }
    }
  }
  return ret;
}

レガシ コードで動作する DBCommand メソッドのオーバーロードを作成しました。

public void DBCommand(SqlCeCommand cmd, string dynSQL, bool Silent) {
  cmd.CommandText = dynSQL;
  DBCommand(cmd, Silent);
}

public void DBCommand(SqlCeCommand cmd, bool Silent) {
  string dynSQL = cmd.CommandText;
  SqlCeTransaction trans = GetConnection().BeginTransaction();
  cmd.Transaction = trans;
  try {
    cmd.ExecuteNonQuery();
    trans.Commit();
  } catch (Exception ex) {
    try {
      trans.Rollback(); // I was under the impression you never needed to call this.
      // If Commit is never called, the transaction is automatically rolled back.
    } catch (SqlCeException) {
      // Handle possible exception here
    }
    MessageBox.Show("DBCommand Except 2"); // This one I haven't seen...
    //WriteDBCommandException(dynSQL, ex, Silent);
  }
}
于 2013-04-01T18:50:44.583 に答える