8

LinqtoSQLを使用してテーブルに複数の行を追加したい

    public static FeedbackDatabaseDataContext context = new FeedbackDatabaseDataContext();
    public static bool Insert_Question_Answer(List<QuestionClass.Tabelfields> AllList)
    {
          Feedback f = new Feedback();
          List<Feedback> fadd = new List<Feedback>();
            for (int i = 0; i < AllList.Count; i++)
            {
                f.Email = AllList[i].Email;
                f.QuestionID = AllList[i].QuestionID;
                f.Answer = AllList[i].SelectedOption;
                fadd.Add(f);
            }
            context.Feedbacks.InsertAllOnSubmit(fadd);
            context.SubmitChanges();
        return true;            
    }

リストオブジェクトにレコードを追加すると、つまりfaddすると、レコードはAllListの最後の値で上書きされます

4

2 に答える 2

17

パーティーに遅れましたが、forループは不要だということを知りたいと思いました。foreachをより適切に使用します(インデックスは必要ありません)。

LINQ(わかりやすくするために名前を変更したメソッド)を使用すると、さらに興味深いものになります。

public static void InsertFeedbacks(IEnumerable<QuestionClass.Tabelfields> allList)
{
    var fadd = from field in allList
               select new Feedback
                          {
                              Email = field.Email,
                              QuestionID = field.QuestionID,
                              Answer = field.SelectedOption
                          };
    context.Feedbacks.InsertAllOnSubmit(fadd);
    context.SubmitChanges();
}

ちなみに、常にアクセスする1つのデータコンテキストを保持するべきではありません。usingステートメント内に、データベースの切断を適切に処理するものをローカルに作成することをお勧めします。

于 2012-08-24T07:55:02.627 に答える
12

forループのスコープでFeedbackのオブジェクトを作成する必要があるため、メソッドを次のように変更します。

public static bool Insert_Question_Answer(List<QuestionClass.Tabelfields> AllList)
{
      List<Feedback> fadd = new List<Feedback>();
        for (int i = 0; i < AllList.Count; i++)
        {
            Feedback f = new Feedback();
            f.Email = AllList[i].Email;
            f.QuestionID = AllList[i].QuestionID;
            f.Answer = AllList[i].SelectedOption;
            fadd.Add(f);
        }
        context.Feedbacks.InsertAllOnSubmit(fadd);
        context.SubmitChanges();
    return true;            
}
于 2012-08-24T07:32:22.130 に答える