मुझे पार्टी के लिए देर हो चुकी है, लेकिन मैंने सोचा कि आप जानना चाहेंगे कि फॉर-लूप अनावश्यक है। 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();
}
वैसे, आपको एक डेटा संदर्भ नहीं रखना चाहिए जिसे आप हर समय एक्सेस करते हैं; एक बनाना बेहतर है स्थानीय रूप से , एक यूजिंग स्टेटमेंट के अंदर, जो डेटाबेस डिस्कनेक्शन को ठीक से हैंडल करेगा।