Primo Chalice Ответов: 2

Как обновить данные в SQL с помощью формы Windows C#, не затрагивая другие поля?


Привет,

Я создал форму Windows с помощью приложения C# Windows для добавления, обновления, удаления и отображения пользовательских данных из SQL.

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

У меня есть 3 текстовых поля:

Имя пользователя
Пароль
Имя

Поэтому, если я помещаю значения в имя пользователя и пароль и оставляю поле имени пустым, то это обновление имени пользователя и пароля, но делает поле имени пустым в SQL.

Я хочу, чтобы неизмененные поля оставались такими же, какими они были.

Пожалуйста помочь.

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

using (SqlConnection connection = new SqlConnection("Data Source=PRIMO-CHALICE;Initial Catalog=NewsClip;Integrated Security=SSPI"))
            {
                using (SqlCommand command = new SqlCommand())
                {
                    command.Connection = connection;            // <== lacking
                    command.CommandType = CommandType.Text;
                    command.CommandText = "UPDATE NewsClip_Login SET Username = @Username, Password = @Password, Name = @Name WHERE Username='" + this.EmployeeUsernameAdd.Text + "';";
                    command.Parameters.AddWithValue("@Username", EmployeeUsernameAdd.Text);
                    command.Parameters.AddWithValue("@Password", EmployeePasswordAdd.Text);
                    command.Parameters.AddWithValue("@Name", EmployeeNameAdd.Text);

                    try
                    {
                        connection.Open();
                        int recordsAffected = command.ExecuteNonQuery();
                    }
                    catch (Exception ex)
                    {
                        MessageBox.Show(ex.Message);
                    }
                    finally
                    {
                        connection.Close();
                    }
                }

2 Ответов

Рейтинг:
1

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) Никогда не храните пароли в открытом тексте - это серьезная угроза безопасности. Здесь есть некоторая информация о том, как это сделать: Хранение паролей: как это сделать.[^]
Чтобы дать вам представление о том, как мы относимся к такого рода вещам, смотрите здесь: Криминальный Кодекс 1[^]

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

И наконец: почему он меняет поле имени? Потому что ты сам так сказал...
Если вы хотите, чтобы имя оставалось нетронутым, когда оно пустое, вам нужно проверить свой код C# и либо использовать другой запрос, либо передать вместо него текущее значение. Если вы скажете SQL "set it to this", то это именно то, что он будет делать...


Primo Chalice

Пожалуйста, скажите мне, как реализовать это в моем коде.

{
byte[] hashedPassword = GetSHA1(userId, пароль);
if (MatchSHA1(hashedPassword, GetSHA1(userId, enteredPassword)))
{
Приставка.WriteLine("войдите в систему!");
}
еще
{
Приставка.WriteLine("не регистрируйте его!");
}
if (MatchSHA1(hashedPassword, GetSHA1(userId, notPassword)))
{
Приставка.WriteLine("не случится!");
}
еще
{
Приставка.WriteLine("не регистрируйте его!");
}
}

OriginalGriff

Копировать и вставлять?

Primo Chalice

Я имел в виду, что получаю данные из базы данных SQL. Так как же мне изменить код? Исходный код приведен в вопросе.

OriginalGriff

Первое, что вам нужно сделать, - это изменить свою базу данных SQL так, чтобы пароли хранились не в обычном тексте, а в хэше, что означает столбец VARBINARY вместо NVARCHAR или VARCHAR.

Primo Chalice

Да, я так и сделал. И я уже скопировал функции MatchSHA1 и GetSHA1.

OriginalGriff

Так что же создает вам проблему? Вы знаете, как выполнять параметризованные запросы, вы знаете, как выполнять команды SELECT, INSERT и UPDATE ... и у вас есть код для сравнения значений. Что тебе еще нужно?

Рейтинг:
0

jsc42

Далее к ответу ОГА...
Я предполагаю, что

this.EmployeeUsernameAdd.Text
это старое имя пользователя, которое вы заменяете, поэтому вы должны использовать (например) @OldUsername параметр.

Чтобы не менять имя, если пользователь оставил поле пустым, используйте что-то вроде ...
... Name = CASE WHEN @Name = '' THEN Name ELSE @Name END ...

Аналогично для полей имени пользователя и пароля, если они пусты, что означает "без изменений".
Это избавляет вас от необходимости либо динамически создавать SQL, либо иметь десятки вариантов.

Поместите SQL (с параметризацией) в хранимую процедуру. Это поможет разделить обязанности, так что, например, если вы меняете имя таблиц / реструктурируете столбцы, вы можете просто изменить хранимую процедуру без необходимости редактировать / компилировать / повторно развертывать исходный код.

Наконец: я надеюсь, что я неправильно понял ваш SQL, и вы на самом деле не храните пароли в обычном тексте. Я надеюсь, что образец-это просто выписка и что любые пароли сохраняются в виде зашифрованного текста, либо фактически посылаются encrypte (предпочтительно), либо вы используете шифрование SQL Server в самой базе данных.