EL Marshal Boraee Ответов: 2

Пожалуйста, помогите в изменении кода

This code reduces the quantities of a particular item when it is sold, but I adjusted the schedule to have validity dates and I want it to reduce the quantities according to the oldest date and then delete it if the quantity is equal to zero and then reduce the latest date


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

For Each r As DataGridViewRow In dgvprodac.Rows
            Dim ad As Integer = Val(r.Cells(0).Value)
            Dim txP As String = Val(r.Cells(1).Value)
            Dim xt As Single = Val(r.Cells(2).Value)
            Dim txC As Integer = Val(r.Cells(3).Value)
            cmd.CommandText = String.Format(" UPDATE [store] SET [sanf_kem] = sanf_kem -{0}  WHERE [ID_sanf] = {1} and  [date_ex] as MinDate ", xt, ad)
            cmd.ExecuteNonQuery()
        Next

2 Ответов

Рейтинг:
6

EL Marshal Boraee 

This code is closer to the solution for me, but a mistake can be modified

sqlstr = "Select min(date_ex),sanf_kem from store where ID_sanf =@pass"
       cmd = New OleDbCommand(sqlstr, con)
       cmd.Parameters.Add(New OleDbParameter("@pass", ad))
       con.Open()
       Dim dr As OleDb.OleDbDataReader = cmd.ExecuteScalar()
       If dr.Read() Then
           DateTimePicker1.Value = dr.Item(0)
           TextBox5.Text = dr.Item(1)
       End If

Рейтинг:
2

OriginalGriff

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

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

Когда вы исправите это через все свое приложение, расскажите нам, что вы пытались сделать, что хотели, и что произошло, когда вы это сделали.