Estou atrasado para a festa, mas achei que você gostaria de saber que o loop for é desnecessário. Melhor usar foreach (você não precisa do índice).
Fica ainda mais interessante quando você usa LINQ (método renomeado para maior clareza):
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();
}
A propósito, você não deve manter um contexto de dados que você acessa o tempo todo; é melhor criar um localmente , dentro de uma instrução using, que tratará adequadamente a desconexão do banco de dados.
