169 похожих чатов

Рассмотрим следующий код, который осуществляет открытие нового счета в системе

(счет может быть мультивалютным – в этом случае валюты перечисляются через запятую)

public class Account
{

public void OpenAccount()
{
var dbManager = new DbManager();
dbManager.BeginTransaction();

string currencies = GetCurrencies ();
if (string.IsNullOrEmpty(currencies))
return;

dbManager.ExecuteSql("EXEC dbo.OpenAccount " + currencies);
dbManager.CommitTransaction();
}

private string GetCurrencies()
{
string str;
str = Console.ReadLine();
return str;
}

}

В данном примере dbManager – объект, осуществляющий взаимодействие с базой данных.
Оцените данный код: его преимущества и недостатки. Каким образом необходимо переписать указанный код? Можете немного подсказать

7 ответов

15 просмотров

sql injection читай

и там самая главная ошибка. строка, как параметр хранимой процедуры должна быть обернута в одинарные кавычки. иначе будет ошибка. ну и переделать не на executeSql(string), а на выполнение запроса через параметры.

Я думаю что неплохо бы еще добавить код на случай ерора и ролбек делать там

> счет может быть мультивалютным – в этом случае валюты перечисляются через запятую еретики и очищающий огонь

Похожие вопросы

Обсуждают сегодня

Господа, а что сейчас вообще с рынком труда на делфи происходит? Какова ситуация?
Rꙮman Yankꙮvsky
29
А вообще, что может смущать в самой Julia - бы сказал, что нет единого стандартного подхода по многим моментам, поэтому многое выглядит как "хаки" и произвол. Короче говоря, с...
Viktor G.
2
30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Карта сайта