saeid23 Ответов: 1

Как использовать multi data reader в C#


Цитата:
при использовании нескольких datareader для повторного извлечения данных из более чем одной таблицы. у меня есть ошибка.. недопустимая попытка вызова CheckDataIsReady при закрытии считывателя.


Что я уже пробовал:

void btn_Click(object sender, EventArgs e)
       {

           Button source = (Button)sender;
           conn.Open();
           SqlCommand cmd = new SqlCommand("select cod , name , price , Unit from info_food where cod='"+source.Name+"'", conn);
           SqlDataReader dr = cmd.ExecuteReader();

           if(dr.HasRows && dataGridView1 == null || dataGridView1.Rows.Count == 0)
           {
               dr.Read();
               dataGridView1.Rows.Add(dr["cod"], dr["name"], dr["Unit"], dr["price"] , 1);
               dr.Close();
               conn.Close();

           }


           else if (dataGridView1.Rows.Count > 0)
           {
               foreach (DataGridViewRow row in dataGridView1.Rows)
               {
                   if (row.Cells[row.Index].Value.ToString().Equals(source.Name))
                   {
                       string mount = dataGridView1.Rows[Int32.Parse(source.Name)-1].Cells[4].Value.ToString();
                       int plus = Convert.ToInt32(mount) + 1;
                       dataGridView1.Rows[0].Cells[4].Value = plus;
                   }
                   else
                   {

                       dr.Read();
                       dataGridView1.Rows.Add(dr["cod"], dr["name"], dr["Unit"], dr["price"], 1);
                   }
               }

           }

         conn.Close();
       }

Richard MacCutchan

Где именно происходит ошибка?

saeid23

еще
{
др.читать();
dataGridView1.Строк.Добавить(д-р["код"], доктор["имя"], Доктор["подразделение"], доктор["цена"], 1);
}

Richard MacCutchan

Итак, доктор.HasRows, скорее всего, ложь.

Richard MacCutchan

Вам нужно изменить свою логику так, чтобы вы выполняли любой код только в том случае, если вызов dr.HasRows возвращается true Если он возвращает false, то делать нечего.

saeid23

хорошо спасибо

Gerry Schmitz

Взгляните внимательнее на свои первые 2 "Если".

saeid23

Ошибка чтения внутри еще

1 Ответов

Рейтинг:
2

OriginalGriff

Сразу несколько проблем:
1) Никогда не объединяйте строки для построения SQL-команды. Это оставляет вас широко открытыми для случайной или преднамеренной атаки SQL-инъекции, которая может уничтожить всю вашу базу данных. Вместо этого всегда используйте параметризованные запросы.

Когда вы объединяете строки, вы вызываете проблемы, потому что SQL получает такие команды, как:

SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
Цитата, добавленная пользователем, завершает строку в том, что касается SQL, и вы получаете проблемы. Но могло быть и хуже. Если я приду и наберу вместо этого: "x';DROP TABLE MyTable;--", то SQL получит совсем другую команду:
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Которые SQL видит как три отдельные команды:
SELECT * FROM MyTable WHERE StreetAddress = 'x';
Совершенно правильный выбор
DROP TABLE MyTable;
Вполне допустимая команда "удалить таблицу"
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?

2) Посмотрите на этот код:
if(dr.HasRows && dataGridView1 == null || dataGridView1.Rows.Количество == 0)
{
др.Читать();
dataGridView1.Строк.Добавить(д-р["код"], доктор["имя"], Доктор["подразделение"], доктор["цена"] , 1);
Так что ... это dataGridView1 является null вы идете вперед и пытаетесь использовать его?

Ошибка, которую вы показываете, там не очевидна - поэтому вам нужно будет использовать отладчик, чтобы следовать этому методу и точно выяснить, что происходит - я предполагаю, что у читателя закончились строки, но вы все равно вызываете Read - но без ваших данных это невозможно сказать, и у нас нет доступа к этому вообще. Отладчик позволит вам точно знать, что происходит.


saeid23

Сначала я отформатировал информацию столбца из таблицы в кнопку. Теперь я собираюсь поместить информацию о строке в datagrid для каждой нажатой кнопки.
А если вы нажмете на каждую кнопку несколько раз, то вместо этого дублируете информацию. Добавьте его в столбец сумма.
Таковы условия для рассмотрения этих вопросов.
Извините, если моя формулировка неверна.
Я иранец

OriginalGriff

Прочтите, что я сказал!

saeid23

хорошо спасибо