Thomas Nielsen - getCore
Id, скажем, делает объект для централизации доступа к данным (как таковой может быть более легко протестирован и высмеян в модульных тестах), и в этом случае у вас есть ваша средняя проверка параметров, проверка escape-последовательности, а затем передать в качестве параметра, и вы должны быть своего рода поясом и подтяжками и можете оптимизировать использование ваших подключений к данным и т. д.
using (var cn = GetConnection())
{
using(var cmd = new SqlCommand("spMyProcedure", cn))
{
cmd.CommandType = System.Data.CommandType.StoredProcedure;
cmd.Parameters.AddWithValue("@id", invoiceNumber);
cmd.Parameters.AddWithValue("@comment", FlareOnBadChars(comment));
cmd.Parameters.AddWithValue("@note", FlareOnBadChars(note));
var affect = (int)cmd.ExecuteScalar();
if (affect != 1)
throw new DataConcernsException(string.Format("Expected to insert data but failed with data: '{0}','{1}','{2}'", invoiceNumber, comment, note));
}
} //Implicitly dispose already calls close on your connection
хороший аддон-это проверка параметров типа строки на наличие escape-последовательности, подобной упрощенной ниже ... на самом деле вам не нужны странные исключения из привязки параметров или sql ... гораздо лучше точно знать, что не так, если это произойдет
private string FlareOnBadChars(string source)
{
if (!Regex.IsMatch(source, @"\A[^\\]+\Z")) //TODO: add other things that you don't like before the ]
throw new DataConcernsException("That's not a nice string");
return source;
}
В качестве альтернативы рассмотрите возможность использования структуры репозитория, такой как entity framework, это позаботится об обработке параметров для вас, и вы не можете избежать того, чтобы сделать это правильно, так сказать ;)
Graeme_Grant
"можно ли легче протестировать и высмеять в модульных тестах" - неужели вы действительно думаете, что он может это сделать, учитывая этот вопрос?
Thomas Nielsen - getCore
Ну, как выразился один болгарский программист: величие ждет тех, кто пытается, никто не может научить вас. это все внутри, просто лезь!