Что факторы должны быть учтены при записи пользовательского класса исключений?

Question

Что факторы должны быть учтены при записи пользовательского класса исключений?

Я согласен с TKK

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

Первый оператор switch будет устанавливать sqlcommand и параметры, а второй останется неизменным в цикле while.

Объявите их в начале вашего метода:

`SqlConnection conn = new SqlConnection("connection string");
 SqlCommand cmd = AdoBase.GetSqlCommand("", conn);`

Измените первый оператор switch, чтобы добавить параметры в команду sql:

switch (model.Attribute)
{
    case "Users":
        sqlCommand = "GeneralUserSearch";
        if (!string.IsNullOrWhiteSpace(model.Name))
        {
            cmd.Parameters.AddWithValue("Name", model.Name);
        }

        if (!string.IsNullOrWhiteSpace(model.Name))
        {
            cmd.Parameters.AddWithValue("Username", model.Username);
        }

        break;
    case "Favorites":
        sqlCommand = "UserFavorites";
        cmd.Parameters.AddWithValue("Favorites", model.Favorites);
        break;
    case "Email":
        sqlCommand = "EmailSearch";
        cmd.Parameters.AddWithValue("Email", model.Email);
        break;
}

Удалите этот блок кода:

switch (model.Attribute)
{
    case "Users":
        if(!string.IsNullOrWhiteSpace(model.Name)) {
            cmd.Parameters.AddWithValue("Name", model.Name);
        }
        if(!string.IsNullOrWhiteSpace(model.Name)) {
            cmd.Parameters.AddWithValue("Username", model.Username);
        }                                                           
        break;
    case "Favorites":
        cmd.Parameters.AddWithValue("Favorites", model.Favorites);
        break;
    case "Email":
        cmd.Parameters.AddWithValue("Email", model.Email);
        break;                                             
}

И добавьте, наконец, после улова, чтобы очистить объекты соединения и команды:

 finally
 {
    conn.Dispose();
    cmd.Dispose();
 }

6

oop exception

задан Community 23 May 2017 в 12:01

7 ответов

Я - немного минималист и только создам пользовательское исключение, если существует код вызова, который должен явно реагировать на конкретное условие, которое произошло. Для всех других ситуаций я буду использовать самое соответствующее исключение библиотеки.NET. Например, ArgumentNullException, InvalidOperationException

1