icavs Ответов: 1

Обновление базы SQL-запросов для записей datagridview в VB.NET

Привет Эксперты,

Хороший день.

Пожалуйста, помогите выяснить, что вызывает эту ошибку "преобразование из типа 'DBNull' в тип 'Decimal' недопустимо".

Заранее спасибо.

Практическое руководство записи:
emp_number | earned_hours
000065 | 100.00
000066 | 125.25

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

Вот мой код
Dim cn As New SqlConnection("Data Source = .\ SQLEXPRESS;Initial Catalog=DAILY_TEST;User ID=sa;")
       cn.Open()
       Try
           Dim cm As New SqlCommand
           With cm
               .Connection = cn
               For i As Integer = 0 To DataGridView1.RowCount - 1
                   .CommandText = "UPDATE a_earned_leave SET earned_hours ='" & CDec(DataGridView1.Rows(i).Cells(1).Value) & "' WHERE emp_number ='" & DataGridView1.Rows(i).Cells(0).Value & "'"
                   .ExecuteNonQuery()
               Next
           End With
           MsgBox("Records updated.")
           cm.Dispose()
           cm = Nothing
           cn.Close()
           cn.Dispose()
       Catch Ex As Exception
           MsgBox(Ex.ToString)
       End Try

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) не жестко кодируйте строки подключения: всегда храните их в конфигурационном файле или аналогичном файле. Если вы этого не сделаете, то при выпуске кода в производство вам придется изменять каждый экземпляр по отдельности, а это означает, что вы выпускаете код, который еще не тестировали. Еще хуже, когда вы начинаете вносить изменения позже, ваш код содержит ссылки на живую БД, и это является рецептом полной катастрофы.

3) Никогда не используйте sa пользователь для подключения к вашей БД: вы должны использовать пользователя с достаточным доступом только для выполнения своей работы, а не Пользователя с полным доступом ко всем БД. Это еще один рецепт повреждения или удаления БД. В сочетании с (2) это означает, что вы передаете каждому пользователю свой "мастер-ключ" ко всем базам данных, и многие люди в вашей компании будут очень расстроены, когда они узнают об этом ...

4) Никогда не оставляйте пароль пользователя sa пустым: это еще один способ оставить вашу БД незащищенной. Измените его на надежный пароль и открывайте только тем, кто в нем нуждается.

5) наугад, DataGridView1.Rows(i).Cells(1) содержит null - либо потому, что вы "сбежали с нижней части" данных DGV в "пустую строку" внизу, либо некоторые из ваших данных DGV содержат null. Используйте отладчик, выясните, какая это строка, и посмотрите на нее.

Но сначала исправьте первые четыре в вашем приложении, иначе ваша БД будет удалена или повреждена. Пропустите один, и вы в опасности ...

icavs

Спасибо за напоминание @OriginalGriff
Я работаю над тестовой средой, поэтому любые материалы о коде и использовании БД не являются окончательными. Будет заглядывать в код, чтобы проверить, не выглядит ли он пустой строкой в нижней части dgv. Спасибо

OriginalGriff

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

Например, моя строка подключения к БД даже не хранится в конфигурационном файле, я использую ее для извлечения:
https://www.codeproject.com/Articles/1203924/Instance-Storage-A-Simple-Way-to-Share-Configurati
Так что это сразу же уменьшает проблемы.

Никогда не полагайтесь на "Я исправлю это позже" - это уже в производстве, когда вы найдете время вернуться ...