パーティーに遅れましたが、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つのデータコンテキストを保持するべきではありません。 1つ作成することをお勧めしますローカル 、usingステートメント内で、データベースの切断を適切に処理します。