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

Question

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

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

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 – объект, осуществляющий взаимодействие с базой данных.
Оцените данный код: его преимущества и недостатки. Каким образом необходимо переписать указанный код? Можете немного подсказать

#dot #programming #russian

0

03.10.2020

7 ответов

32 просмотра

Dmitry

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

0

03.10.2020

Tilek Cherikbaev Автор вопроса

Dmitry
и там самая главная ошибка. строка, как параметр х...

спасибо большое

0

03.10.2020

Пользователь 6196d

Dmitry
и там самая главная ошибка. строка, как параметр х...

Ну не факт

0

03.10.2020