Исключения или перечисления

@Nikolay_W · Регистрация: 07.04.2011

Студворк — интернет-сервис помощи студентам

Давно задавался вопросом, как лучше реализовывать обработку разных ответов от класса.
То есть допустим есть класс, реализующий работу с неким апи, есть несколько методов: авторизация и получении информации о определенном пользователе.
Я вижу несколько вариантов реализации.
1) Вариант на исключениях. Минусы: много классов-исключений.

Кликните здесь для просмотра всего текста

Api:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
 class SomeApi
    {
        private readonly string _login;
        private readonly string _password;
        private SomeHttpClient _httpClient;
 
        public SomeApi(string login, string password)
        {
            _login = login;
            _password = password;
            _httpClient = new SomeHttpClient();
        }
 
        public void Login()
        {
            string response = _httpClient.Post("http://somesite.ru/someloginendpoint", _login, _password);
            if (response == "invalid") throw new InvalidCredentialException();
            if (response == "banned") throw new UserIsBannedException();
            if (response == "needActivate") throw new UserIsNotActivatedException();
        }
 
        public UserInfo GetUserInfo(string id)
        {
            string response = _httpClient.Post("http://somesite.ru/somegetuserinfoendpoint", id);
            if (response == "private") throw new UserIsPrivateException(id);
            return new UserInfo(response);
        }
    }
 
    class UserInfo
    {
        public string Name;
        public UserInfo(string name)
        {
            Name = name;
        }
    }

Использование:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
 class Program
    {
        static void Main()
        {
            var api = new SomeApi("login", "password");
            try
            {
                api.Login();
                try
                {
                    var userInfo = api.GetUserInfo("someid");
                    Console.WriteLine(userInfo.Name);
                }
                catch (UserIsPrivateException)
                {
                    Console.WriteLine($"user {"someid"} is private");
                }
                catch (SomeNetworkException)
                {
                    Console.WriteLine("network error");
                }
            }
            catch (InvalidCredentialException)
            {
                Console.WriteLine("Incorrect credentials, please try again");
            }
            catch (UserIsBannedException)
            {
                //some code
            }
            catch (UserIsNotActivatedException)
            {
                //some code
            }
            catch (SomeNetworkException)
            {
                Console.WriteLine("network error");
            }
 
 
        }
    }

2) Вариант с использованием перечислений. (немного перемудрил возможно, вообще надо бы еще добавить перечисление GetUserInfoResult с возможными ответами (кроме приватного юзера возможно что то еще может в дальнейшем появиться), но тогда еще третьим параметром ApiRequestResult надо будет передавать какой нибудь object Argument

Кликните здесь для просмотра всего текста

Api:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
class SomeApi
    {
        private readonly string _login;
        private readonly string _password;
        private SomeHttpClient _httpClient;
 
        public SomeApi(string login, string password)
        {
            _login = login;
            _password = password;
            _httpClient = new SomeHttpClient();
        }
 
        public ApiRequestResult<LoginResult> Login()
        {
            string response;
            try
            {
                response = _httpClient.Post("http://somesite.ru/someloginendpoint", _login, _password);
            }
            catch (SomeNetworkError)
            {
                return new ApiRequestResult<LoginResult>(false, LoginResult.None);
            }
            if (response == "invalid")return new ApiRequestResult<LoginResult>(true,LoginResult.Invalid);
            if (response == "banned") return new ApiRequestResult<LoginResult>(true,LoginResult.Banned);
            if (response == "needActivate") return new ApiRequestResult<LoginResult>(true, LoginResult.NeedActivate);
            return new ApiRequestResult<LoginResult>(true, LoginResult.Ok);
        }
 
        public ApiRequestResult<UserInfo> GetUserInfo(string id)
        {
            string response;
            try
            {
                response = _httpClient.Post("http://somesite.ru/somegetuserinfoendpoint", id);
            }
            catch (SomeNetworkError)
            {
                return new ApiRequestResult<UserInfo>(false, null);
            }
            if (response == "private") return new ApiRequestResult<UserInfo>(true, UserInfo.Private);
            return new ApiRequestResult<UserInfo>(true, new UserInfo(response));
        }
    }
 
    class ApiRequestResult<T>
    {
        public bool IsOk;
        public T Result;
 
        public ApiRequestResult(bool isOk, T result)
        {
            IsOk = isOk;
            Result = result;
        }
    }
 
    enum LoginResult
    {
        None,
        Ok,
        Invalid,
        Banned,
        NeedActivate
    }
 
    class UserInfo
    {
        public static UserInfo Private = new UserInfo();
        public bool IsPrivate { get; private set; }
        public string Name;
 
        private UserInfo()
        {
            IsPrivate = true;
        }
        public UserInfo(string name)
        {
            Name = name;
        }
    }

Использование:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
 class Program
        {
            static void Main()
            {
                var api = new SomeApi("login", "password");
                var loginResult = api.Login();
                if (!loginResult.IsOk)
                {
                    Console.WriteLine("network error");
                    return;
                }
                //process response
                switch (loginResult.Result)
                {
                    case LoginResult.None:
                        break;
                    case LoginResult.Ok:
                        break;
                    case LoginResult.Invalid:
                        break;
                    case LoginResult.Banned:
                        break;
                    case LoginResult.NeedActivate:
                        break;
                    default:
                        throw new ArgumentOutOfRangeException();
                }
 
            }
        }

Есть еще третий вариант, скомбинировать первые да, то есть исключения, связанные с сетью пробрасывать вызывающей стороне, а ответы просто так возвращать.

@insite2012 · 08.11.2016, 15:22

Сообщение от Nikolay_W

третий вариант, скомбинировать первые да

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

@Nikolay_W · 08.11.2016, 15:24 **[ТС]**

Сообщение от insite2012

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

Получается, тогда будет одно перечисление на все возможные методы, и значений в нем может быть большое количество. Это нормально?

upd: я понял, вы имели в виду generic переменную. Надо попробовать, выглядит интересно

@insite2012 · 08.11.2016, 15:37

Сообщение от Nikolay_W

значений в нем может быть большое количество

А зачем особенно большое? Конечным пользователям вовсе не нужны тонкости. Краткое указание что не так, и хватит. Так что вполне три-пять членов перечисления должно хватить.

Добавлено через 10 минут

Сообщение от Nikolay_W

я понял, вы имели в виду

Типа вот такой конструкции.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
using System;
using System.Collections.Generic;
using System.Text;
 
namespace ConsoleApplication23 {
    class Program {
        static void Main(string[] args) {
            try {
                throw new MyException(Result.NO_NETWORK);
            }
            catch (MyException ex) {
                Console.WriteLine(ex.Result);
            }
            Console.ReadLine();
        }
    }
    public class MyException : Exception {
        public readonly Result Result;
        public MyException(Result r) {
            Result = r;
        }
    }
    public enum Result {
        OK,
        NO_NETWORK,
        GENERIC
    }
}

Новые блоги и статьи Все статьи Все блоги /
Подстановка значения реквизита справочника в табличную часть документа Maks 10.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: при выборе сотрудника (справочник Сотрудники) в ТЧ документа. . .	Очистка реквизитов документа при копировании Maks 09.04.2026 Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .	модель ЗдравоСохранения 8. Подготовка к разному выполнению заданий anaschu 08.04.2026 https:/ / github. com/ shumilovas/ med2. git main ветка * содержимое блока дэлэй из старой модели теперь внутри зайца новой модели 8ATzM_2aurI	Блокировка документа от изменений, если он открыт у другого пользователя Maks 08.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в конфигурации КА2. Задача: запретить редактирование документа, если он открыт у другого пользователя. / / . . .
Система безопасности+живучести для сервера-слоя интернета (сети). Двойная привязка. Hrethgir 08.04.2026 Далее были размышления о системе безопасности. Сообщения с наклонным текстом - мои. А как нам будет можно проверить, что ссылка наша, а не подделана хулиганами, которая выбросит на другую ветку и. . .	Модель ЗдрввоСохранения 7: больше работников, больше ресурсов. anaschu 08.04.2026 работников и заданий может быть сколько угодно, но настроено всё так, что используется пока что только 20% kYBz3eJf3jQ	Дальние перспективы сервера - слоя сети с космологическим дизайном интефейса карты и логики. Hrethgir 07.04.2026 Дальнейшее ближайшее планирование вывело к размышлениям над дальними перспективами. И вот тут может быть даже будут нужны оценки специалистов, так как в дальних перспективах всё может очень сильно. . .	Горе от ума kumehtar 07.04.2026 Эта мне ментальная установка, что вот прямо сейчас, мол, мне для полного счастья не хватает (нужное вписать), и когда я этого достигну - тогда и полный кайф. Одна из самых сильных ловушек на пути. . . .