このクエリで何が問題になっていますか？

Question

この関数がレコードをデータベースに挿入しない理由がわかりません。 :( は、私は一切エラーメッセージや、データベース内だけで何も得る

EDIT：これは私のクエリは、今どのように見えるかです..まだ何も..

connection.Open(); 
XmlNodeList nodeItem = rssDoc.SelectNodes("/edno23/posts/post"); 

foreach (XmlNode xn in nodeItem) 
{ 
    cmd.Parameters.Clear(); 
    msgText = xn["message"].InnerText; 
    C = xn["user_from"].InnerText; 
    avatar = xn["user_from_avatar"].InnerText; 
    string endhash = GetMd5Sum(msgText.ToString()); 
    cmd.Parameters.Add("@endhash",endhash); 
    cmd.CommandText = "Select * FROM posts Where hash=@endhash"; 
    SqlCeDataReader reader = cmd.ExecuteReader(); 

    while (reader.Read()) 
    { 
     string msgs = reader["hash"].ToString(); 

     if (msgs != endhash || msgs == null) 
     { 
      sql = "INSERT INTO posts([user],msg,avatar,[date],hash) VALUES(@username,@messige,@userpic,@thedate,@hash)"; 
      cmd.CommandText = sql; 
      cmd.Parameters.Add("@username", C); 
      cmd.Parameters.Add("@messige", msgText.ToString()); 
      cmd.Parameters.Add("@userpic", avatar.ToString()); 
      cmd.Parameters.Add("@thedate", dt); 
      cmd.Parameters.Add("@hash", endhash); 
      cmd.ExecuteNonQuery();// executes query 
      adapter.Update(data);// saves the changes 
     } 
    } 

    reader.Close(); 
} 

connection.Close(); 

Answer 1

を役に立てば幸いnodeItemは実際にそれですべてのアイテムを持っていますか？そうでなければ、foreachループの内容は実行されていません。

どのようなアダプタとデータが使用されていますか？クエリと更新は、他のコマンドやリーダーを介して行われるようです。

「ハッシュ」には実際に何が含まれていますか？それがハッシュの場合、なぜあなたはwhileループの中でハッシュの内容をハッシュしていますか？そうでない場合、クエリ内のハッシュと比較されるのはなぜですかSELECT * FROM posts WHERE hash = @endhash？

ループを終了する前に接続を閉じないと、ループを制御するリーダーが無効になりますか？

Answer 2

プログラムをデバッグする方法： http://www.drpaulcarter.com/cs/debug.php

SQL CEの代わりにSQL Server Expressを使用すると機能するのでしょうか？もしそうなら、SQLプロファイラを分解してSQLコマンドを見てみることができますか？実行？

Answer 3

同じSqlCeCommandインスタンスを再利用しようとしていることが問題であると思われます。

whileループ内で新しいSqlCeCommandを作成してみてください。また、usingステートメントを使用してデータオブジェクトを閉じることもできます。

---

なぜデータセットをまったく変更していないので、adapter.Update(data)に電話するのはなぜですか？私はadapter.Fill(data)に電話したいと思う。 Updateメソッドは、データセット内のすべての変更をデータベースに保存します。あなたがDataReaderオブジェクトを持つレコードをループに「cmdを」コマンドを使用して、挿入を実行するwhile文の内部で同じ「CMD」コマンドを使用しているここで起こって物事の

Answer 4

たくさん...

ステートメント。以前は別のコマンド 'cmdAdd'を宣言していましたが、どこにも使用していないようです。それはあなたが挿入ステートメントのために使用しようとしていたものですか？

また、データループをwhileループ内で閉じて、データレデイアを繰り返し処理します。あなたは1つのレコードを読んで、その方法であなたのデータベースへの接続を閉じます。挿入条件が満たされていない場合は、データベースに何も書き込まないことになります。

EDIT：

あなたは本当にXMLNodeの上のforeachの外のデータベースへの接続を開閉する必要があります。ループするノードが10個ある場合、db接続は10回開いて閉じます（接続プーリングはおそらくそれを妨げるでしょうが...）

また、表をデータセットに変換する必要はありません。データセット内の値を変更していなくても、そのデータセットの更新を繰り返し呼び出すようにしています（「save shanges」）。 'posts'テーブルが遠隔であっても、これは何の理由もない（ハンドヘルドデバイスでは、それ以上ではない）多くのメモリを吸うことになります。

Answer 5

なぜデータベース接続を終了しますか whileループ？
投稿したコードcmd.ExecuteNonQuery（）を開こうとしていないDB接続オブジェクトで呼び出すときに、が例外をスローする必要があります。

SqlCeCommand.ExecuteNonQuery()メソッドは、影響を受ける行の数を返します。
以下に示すように、デバッガで1が戻ってきているかどうかをチェックしないのはなぜですか？

int rowsAffectedCount = cmd.ExecuteNonQuery(); 

はそれが:-)

Answer 6

「* FROM posts where where hash = @ endhash」から返されたものはありますか？あなたがブロック「を使用して、」実装していないといくつかの問題を持っている

Answer 7

ない場合は、whileループの事項の内部は何も...。私はあなたの内部コードにいくつかを追加しました。接続と選択コマンドのブロックは、私の考えでもっとうまく考えています。私はあなたがデータアダプタで同じことをしていることを願っています。

using (var connection = new SqlCeConnection(connectionString)) 
{ 
    connection.Open(); 
    var nodeItem = rssDoc.SelectNodes("/edno23/posts/post"); 

    foreach (XmlNode xn in nodeItem) 
    { 
     using (
      var selectCommand = 
       new SqlCeCommand(
        "Select * FROM posts Where hash=@endhash", 
        connection)) 
     { 
      var msgText = xn["message"].InnerText; 
      var c = xn["user_from"].InnerText; 
      var avatar = xn["user_from_avatar"].InnerText; 
      var endhash = GetMd5Sum(msgText); 
      selectCommand.Parameters.Add("@endhash", endhash); 
      selectCommand.CommandText = 
       "Select * FROM posts Where hash=@endhash"; 
      using (var reader = selectCommand.ExecuteReader()) 
      { 
       while (reader.Read()) 
       { 
        var msgs = reader["hash"].ToString(); 

        if (msgs == endhash && msgs != null) 
        { 
         continue; 
        } 

        const string COMMAND_TEXT = 
         "INSERT INTO posts([user],msg,avatar,[date],hash) VALUES(@username,@messige,@userpic,@thedate,@hash)"; 
        using (
         var insertCommand = 
          new SqlCeCommand(
           COMMAND_TEXT, connection)) 
        { 
         insertCommand.Parameters.Add("@username", c); 
         insertCommand.Parameters.Add(
          "@messige", msgText); 
         insertCommand.Parameters.Add(
          "@userpic", avatar); 
         insertCommand.Parameters.Add("@thedate", dt); 
         insertCommand.Parameters.Add(
          "@hash", endhash); 
         insertCommand.ExecuteNonQuery(); 
          // executes query 
        } 
        adapter.Update(data); // saves teh changes 
       } 

       reader.Close(); 
      } 
     } 
    } 

    connection.Close(); 
} 

もちろん、追加の入れ子を使用すると、部分は別の方法として分割する必要があります。