yoecon Ответов: 2

Nullreferenceexception был необработан в C#

Я попытался вставить выбранные данные из Datatable в Datagridview. Однако он показал, что ошибка типа NullReferenceException не была обработана. Может ли кто-нибудь посоветовать, как решить эту ошибку. мой код выглядит следующим образом. Спасибо.

DataTable LoadData = ConnectionStr.executsql("Select * from ช0210$ WHERE MS_INS_NAME = '" + Combo_CompName.Текст + "'");

если (метод loadData.Строк.Count > 0)
{
по каждому элементу (объекта datarow Dr в метод loadData.Строк)
{
dgv.Строк.Добавить(др.ItemArray);
}
}

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

Я попробовал поставить если проверить
если (метод loadData.Строк.Count > 0)

2 Ответов

Рейтинг:
1

MadMyche

Вам нужно проверить, чтобы увидеть, если Метод loadData не является нулевым; потому что если это так, то не будет ни а Строки собственность, ни Рассчитывать свойство строк.

Вы можете либо обернуть вещь while в другую if блок или вы можете комбинировать нулевой проверьте с помощью Строки.Рассчитывать проверять. Вот оба варианта; выберите один

if (LoadData != null)
{
   if (LoadData.Rows.Count > 0)
   {
      foreach (DataRow dr in LoadData.Rows)
      {
         dgv.Rows.Add(dr.ItemArray);
      }
   }
}
/* =====< OR >===== */
if ((LoadData != null) && (LoadData.Rows.Count > 0))
{
   foreach (DataRow dr in LoadData.Rows)
   {
      dgv.Rows.Add(dr.ItemArray);
   }
}

yoecon

Привет Мадмыч,
Спасибо Вам за совет. Это работа. Ошибка исчезла.
Однако моя работа еще не закончена и нуждается в дальнейшей помощи.
Я создал ComboBox с именем Combo_CompName, который уже был заполнен названием компании.
То, что я хочу сделать, это загрузить данные из datatable с условием только выбранного названия компании в ComboBox, чтобы быть showm в gridview при нажатии кнопки загрузки. Я попытался создать функцию, которая следует за кодом, и поместить ее в кнопку загрузки (нажмите событие), но это не работает. пожалуйста, adivse.


общественного недействительными loadshareholder()
{
выбранная строка = CompName.GetItemText(это.Combo_CompName.Элемент);
DataTable LoadData = ConnectionStr.executsql("SELECT * FROM ช0210$ WHERE MS_INS_NAME = '" + selected + "'");

если ((метод loadData != нулем) и усилитель; & (метод loadData.Строк.Count > 0))
{
по каждому элементу (объекта datarow Dr в метод loadData.Строк)
{
dgv.Строк.Добавить(др.ItemArray);
}
}

MadMyche

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

Рейтинг:
1

OriginalGriff

К тому, что сказала Мадмич, нужно добавить еще две вещи:
1) Это одна из самых распространенных проблем, которые нам задают, и это также та, на которую мы меньше всего готовы ответить, но вы больше всего готовы ответить сами.

Позвольте мне просто объяснить, что означает ошибка: вы пытались использовать переменную, свойство или возвращаемое значение метода, но оно содержит null - что означает, что в переменной нет экземпляра класса.
Это немного похоже на карман: у вас есть карман в рубашке, который вы используете, чтобы держать ручку. Если вы сунете руку в карман и обнаружите, что там нет ручки, вы не сможете подписать свое имя на листе бумаги - и вы получите очень смешные взгляды, если попытаетесь! Пустой карман дает вам нулевое значение (здесь нет ручки!), поэтому вы не можете сделать ничего, что обычно делали бы, когда вы извлекли свою ручку. Почему он пуст? Вот в чем вопрос - может быть, вы забыли взять ручку, когда уходили из дома сегодня утром, или, возможно, вы оставили ручку в кармане вчерашней рубашки, когда снимали ее вчера вечером.

Мы не можем сказать, потому что нас там не было, и, что еще важнее, мы даже не можем видеть вашу рубашку, не говоря уже о том, что находится в кармане!

Вернемся к компьютерам, и вы каким - то образом сделали то же самое-и мы не можем увидеть ваш код, а тем более запустить его и узнать, что содержит null, когда это не должно быть.
Но вы можете - и Visual Studio поможет вам здесь. Запустите свою программу в отладчике, и когда она выйдет из строя, VS покажет вам строку, в которой она обнаружила проблему. Затем вы можете начать просматривать различные его части, чтобы увидеть, какое значение равно null, и начать просматривать свой код, чтобы узнать, почему. Поэтому поставьте точку останова в начале метода, содержащего строку ошибки, и снова запустите программу с самого начала. На этот раз VS остановится перед ошибкой и позволит вам изучить, что происходит, пройдя через код, глядя на ваши значения.

Но мы не можем этого сделать - у нас нет вашего кода, мы не знаем, как его использовать, если бы он у нас был, у нас нет ваших данных. Так что попробуйте - и посмотрите, сколько информации вы можете узнать!

2) Никогда не объединяйте строки для построения команды 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;
Вполне допустимая команда "удалить таблицу" 
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

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