Jestem spóźniony na imprezę, ale pomyślałem, że możesz chcieć wiedzieć, że pętla for jest niepotrzebna. Lepiej używaj foreach (nie potrzebujesz indeksu).
Robi się jeszcze ciekawiej, gdy używasz LINQ (metoda o zmienionej nazwie dla jasności):
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();
}
Nawiasem mówiąc, nie powinieneś trzymać jednego kontekstu danych, do którego masz dostęp przez cały czas; lepiej je utworzyć lokalnie , wewnątrz instrukcji using, która prawidłowo obsłuży rozłączenie bazy danych.