Member 13713259 Ответов: 2

Справка, система заимствований

привет ребята я новичок в программировании пожалуйста потерпите со мной
у меня есть система заимствований, у меня есть список предметов. как я могу вернуть товар после его заимствования??

im использует vb 2015 с sql 2008 в качестве своей базы данных

срочно нужна ваша помощь. спасибо ребята

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

With cmd
            .Connection = ProjectConnect
            .CommandText = "UPDATE tblItems SET Quantity = @quan + 1  WHERE ItemID = " & txtID.Text
            .Parameters.AddWithValue("@quan", txtQuan.Text)
            .ExecuteNonQuery()
End With

попробовал сделать это, но этого недостаточно, мне нужно, чтобы вся запись была удалена в gridview

Richard MacCutchan

Вам нужно использовать команду DELETE.

Member 13713259

как мне увеличить количество, если я собираюсь использовать delete??

Richard MacCutchan

Ну и что вы пытаетесь сделать: обновить запись или удалить ее?

Member 13713259

я хочу, чтобы запись в gridview исчезла после нажатия кнопки возврата, но вместо этого она остается там. как мне это сделать?

2 Ответов

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

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

Но даже тогда это странный код.Whey вы устанавливаете количество (предположительно количество на складе) на количество товаров плюс один? Если он возьмет один предмет из десяти, которые у вас есть на складе, и вернет его, то количество instock будет равно 2 - что произойдет с другими предметами?
Возможно, то, что вам нужно сделать, это
... SET Quantity = Quantity + @quan WHERE ...

Member 13713259

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

OriginalGriff

... SET Quantity = Quantity + 1 WHERE ...

Рейтинг:
2

Patrice T 

.CommandText = "UPDATE tblItems SET Quantity = @quan + 1 WHERE ItemID = " & txtID.Text

Почему вы смешиваете параметры и конкатенацию?
Это опасно.

Никогда не создавайте SQL-запрос путем объединения строк. Рано или поздно вы сделаете это с помощью пользовательских вводов, и это откроет дверь к уязвимости под названием "SQL-инъекция", она опасна для вашей базы данных и подвержена ошибкам.
Одна кавычка в имени - и ваша программа выйдет из строя. Если пользователь вводит имя, например "Брайан О'Коннер", может привести к сбою вашего приложения, это уязвимость SQL-инъекции, и сбой-это наименьшая из проблем, вредоносный пользовательский ввод, и он продвигается к командам SQL со всеми учетными данными.
SQL-инъекция - Википедия[^]
SQL-инъекция[^]
Атаки SQL-инъекций на примере[^]
PHP: SQL-инъекция - руководство пользователя[^]
Шпаргалка по предотвращению инъекций SQL - OWASP[^]

Member 13713259

спасибо добрый человек