Должен ли AcceptChanges() вызываться каждый раз, когда добавляется новая строка?

Question

Что рекомендуется

while (reader.Read()) 
{ 
    table.Rows.Add(
      new object[] { reader[0], reader[1], reader[2], reader[3] } 
    ); 
    table.AcceptChanges(); 
} 

или

while (reader.Read()) 
{ 
    table.Rows.Add(
      new object[] { reader[0], reader[1], reader[2], reader[3] } 
    ); 
} 
table.AcceptChanges(); 

Примечание где table.AcceptChanges помещается.

---

EDIT 1

Вот код блока:

protected void Page_Load(object sender, EventArgs e) 
{ 
    IDataReader reader = cust.GetCustomerOrderSummary("99999"); 
    using (DataSet ds = new DataSet()) 
    { 
     using (DataTable table = 
       new DataTable { TableName = "OrderSummary" }) 
     { 
      DataColumn idColumn = table.Columns.Add("number", typeof(int)); 
      table.Columns.Add("name", typeof(string)); 
      table.Columns.Add("quantity", typeof(int)); 
      table.Columns.Add("prev_quantity", typeof(int)); 
      table.PrimaryKey = new DataColumn[] { idColumn }; 
      while (reader.Read()) 
      { 
       table.Rows.Add(
        new object[]{ reader[0], reader[1], reader[2], reader[3] } 
       ); 
       table.AcceptChanges(); 
      } 
      ds.Tables.Add(table); 
      rptCustomerOrder report = 
        new rptCustomerOrder { DataSource = ds }; 
      ReportViewer1.Report = report; 
     } 
    } 
} 

---

EDIT 2
После прочтения статьи MSDN here я решил поместить AcceptChanges () вне цикла на основе следующих s (из статьи):

Вызов AcceptChanges на уровне DataTable вызывает метод AcceptChanges для каждого вызова DataRow.

Answer 1

Это действительно зависит от того, что ваш объект таблицы и что делает AcceptChanges. В общем, я бы сказал, что подход secodn более эффективен, если вы можете вызвать AcceptChanges только один раз и заставить его выполнять свою работу.

Снова это вообще, потому что я не знаю, на каком объекте вы на самом деле имеете дело. Но делать вещи внутри циклов обычно дороже, потому что вы делаете это чаще, но ваш объект таблицы может потребовать, чтобы вы принимали изменения между каждой строкой, и в этом случае у вас нет выбора.

Answer 2

Я принимаю AcceptChanges, когда вы на самом деле фиксируете свои данные?

Первый способ менее эффективен в этом случае, чем второй, но более безопасен, так как вероятность потери данных меньше, если ваше чтение прерывается в какой-то момент.

Answer 3

Вы действительно хотите вызвать метод AcceptChanges? Я сделал подобное, и мне никогда не приходилось его использовать.

Answer 4

Вызов AcceptChanges после добавления новой строки будет на самом деле превратить DataRowState из ваших Добавленного DataRow от Added к Unchanged. Если вы поедете с ним, вы можете потерять отслеживание ваших новых добавленных строк и во время настойчивости. ADO.NET не сможет определить строки, которые необходимо вставить в базу данных. Поэтому выбирайте этот вариант с умом, вы даже не можете его требовать.

Answer 5

Вы принимаете изменения в таблице, но имейте в виду, что вы можете вызывать acceptchanges на каждой отдельной строке, таким образом, только влияя на эту строку.

dsDataset.Tables ["OrderSummary"]. Строки [currentrow] .acceptchanges();

Это действительно зависит от того, что вы пытаетесь сделать. Вы можете обновить все в конце таблицы или по отдельности в каждой строке.

Также имейте в виду, что принимающая таблица принимает это решение, изменяет datarowstate на неизменное. Таким образом, если вы пытаетесь обновить базу данных (SQL, Access, ect ...), это может помешать обновлению базы данных.

Из вашего примера видно, что вы используете эту таблицу только для отчета, поэтому не знаете, почему вам необходимо обновить datarowstate вообще. Не уверен, что отчет волнует. Насколько я знаю, acceptchanges/datarowstate касается только обновления бэкэнда.