2010-08-21 84 views
1

这是我的代码我有:在C#中的几个foreach循环中执行SQLite插入?

我读过设置CommandText应该只发生一次,而不是在一个循环中...但是,我如何从foreach中获取单独的项目数据?

有人聪明地重构代码,这将是不错的:)

using (SQLiteTransaction trans = DataAccess.ConnectionManager.BeginTransaction()) 
{ 
    using (SQLiteCommand com = new SQLiteCommand(DataAccess.ConnectionManager)) 
    { 
     // How can I add the parameters here if they are only known in the foreach loop? 
     com.Parameters.Add(new SQLiteParameter("@date", day.SchooldayDate)); 
     com.Parameters.Add(new SQLiteParameter("@periodnumber", period.PeriodNumber)); 
     com.Parameters.Add(new SQLiteParameter("@schoolclasscode", period.SchoolclassCode)); 


     foreach (var week in weekList) 
     { 
      foreach (var day in week.Days) 
      { 
       foreach (var period in day.Periods) 
       { 
        com.CommandText = "Insert into tablename (date,periodnumber,schoolclasscode) Values (@date,@periodnumber,@schoolclasscode)"; 

       } 
      } 
     } 

     com.ExecuteNonQuery(); 
    } 
    trans.Commit();     
} 

UPDATE:该解决方案!

using (SQLiteTransaction trans = DataAccess.ConnectionManager.BeginTransaction()) 
      { 
       using (SQLiteCommand com = new SQLiteCommand(DataAccess.ConnectionManager)) 
       { 
        com.CommandText = "Insert into lessonday (lessondate,lessonnumber,schoolclasscode) VALUES (@lessondate,@lessonnumber,@schoolclasscode)"; 

        SQLiteParameter p1 = new SQLiteParameter("@lessondate", DbType.DateTime); 
        SQLiteParameter p2 = new SQLiteParameter("@lessonnumber", DbType.Int32); 
        SQLiteParameter p3 = new SQLiteParameter("@schoolclasscode", DbType.String); 

        com.Parameters.Add(p1); 
        com.Parameters.Add(p2); 
        com.Parameters.Add(p3); 

        foreach (var week in weekList) 
        { 
         foreach (var day in week.Days) 
         { 
          p1.Value = day.SchooldayDate; 
          foreach (var period in day.Periods) 
          { 
           p2.Value = period.PeriodNumber; 
           p3.Value = period.SchooclassCode; 

           com.ExecuteNonQuery(); 
          } 
         } 
        } 
       } 
       trans.Commit(); 
      } 
     } 
+0

是的,这是正确的做法!使用参数化查询并将您的插入打包到一个大型事务中会使Sqlite快得多。 – TTT 2010-08-22 04:44:09

+0

是的,它是快速可怕的4000插入不到一个感觉第二:) – Elisabeth 2010-08-22 07:58:33

+0

它更加疯狂在0,488秒插入40行5列哈哈。 Sqlite是疯狂的快速。 – Elisabeth 2010-08-22 10:52:46

回答

0

好吧,你可以只移动循环:

using (SQLiteTransaction trans = DataAccess.ConnectionManager.BeginTransaction()) 
{ 
    foreach (var week in weekList) 
    { 
    foreach (var day in week.Days) 
    { 
     foreach (var period in day.Periods) 
     { 
     using (SQLiteCommand com = new SQLiteCommand(...)) 
     { 
      com.Parameters.Add(new SQLiteParameter(...)); 
      com.Parameters.Add(new SQLiteParameter(...)); 
      com.Parameters.Add(new SQLiteParameter(...)); 
      com.CommandText = "Insert into ..."; 
      com.ExecuteNonQuery(); 
     } 
     } 
    } 
    } 
    trans.Commit();     
} 

然而,这似乎有点丑陋的我。我不知道SQLite提供程序将如何表现,但我认为至少值得尝试创建SQLiteCommand根据您的原始代码,添加参数但不是它们的值 ...然后移动ExecuteNonQuery代码进入最内部的foreach循环。事情是这样的:

using (SQLiteTransaction trans = DataAccess.ConnectionManager.BeginTransaction()) 
{ 
    using (SQLiteCommand com = new SQLiteCommand(DataAccess.ConnectionManager)) 
    { 
    com.CommandText = "Insert into ..."; 
    SQLiteParameter p1 = com.Parameters.Add(new SQLiteParameter(...)); 
    SQLiteParameter p2 = com.Parameters.Add(new SQLiteParameter(...)); 
    SQLiteParameter p3 = com.Parameters.Add(new SQLiteParameter(...)); 
    foreach (var week in weekList) 
    { 
     p1.Value = week; 
     foreach (var day in week.Days) 
     { 
     p2.Value = day; 
     foreach (var period in day.Periods) 
     { 
      p3.Value = period;    
      com.ExecuteNonQuery(); 
     } 
     } 
    } 
    } 
    trans.Commit();     
} 

只是重申,我不知道这是否会工作 - 但我希望会。

+0

[+1,如果我可以!](http://meta.stackexchange.com/questions/5212/) – Timwi 2010-08-21 21:24:54

+0

@Lisa:我怀疑它是错误的类型。尝试使用适当的DbType创建参数。 – 2010-08-21 21:57:09

+0

是的,只是之前,你发布了这一点,我删除了我的评论哈哈看到它再次更新:P 好吧谢谢乔恩现在我可以去睡觉晚安;) – Elisabeth 2010-08-21 21:58:48