1. C# / Говнокод #9124

    +142

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    81. 81
    82. 82
    83. 83
    84. 84
    class SMSSender
    {
        const string API_URL = "http://api.sms****.ru/?";
        string base_URL = "";
        private string _email;
        private string _password;
        XmlDocument doc = new XmlDocument();
        Dictionary<string,string> parameters;

        public SMSSender(string email, string password)
        {
            _email = email;
            _password = password;
            base_URL = API_URL + "email=" + _email + "&password=" + _password + "&";
        }

        public bool LoginAttempt()
        {
            parameters = new Dictionary<string, string>();
            parameters.Add("method", "login");
            return APIRequest(parameters);
        }

        public KeyValuePair<int,object> GetCreditsLeft()
        {
            parameters = new Dictionary<string, string>();
            parameters.Add("method", "get_profie");
            APIRequest(parameters);
            return new KeyValuePair<int, object>(0, int.Parse(GetValueByName("credits")));
        }

        public int SendSMS(string senderName, string internationalNumber, string text)
        {
            parameters = new Dictionary<string, string>();
            parameters.Add("method", "push_msg");
            parameters.Add("text", text);
            parameters.Add("phone", internationalNumber);
            parameters.Add("sender_name", senderName);
            APIRequest(parameters);
            return int.Parse(GetValueByName("n_raw_sms"));
        }

        public KeyValuePair<int, object> GetLastError()
        {
            return new KeyValuePair<int, object>(int.Parse(doc.GetElementsByTagName("err_code")[0].InnerText), doc.GetElementsByTagName("text")[0].InnerText);
        }

        private string GetValueByName(string keyToReturn)
        {
            return doc.GetElementsByTagName(keyToReturn)[0].InnerText;
        }

        private bool APIRequest(Dictionary<string, string> param)
        {
            string URL = base_URL;
            foreach (KeyValuePair<string, string> p in param)
                URL = URL + p.Key + "=" + p.Value + "&";
            doc.Load(URL);
            if (GetLastError().Key == 0) return true;
            else throw new SMSSenderException(GetLastError().Key, GetLastError().Value.ToString());
        }
    }

    class SMSSenderException : Exception
    {
        int _errorCode;
        string _Message;
        public SMSSenderException(int errorCode, string Message)
        {
            _errorCode = errorCode;
            _Message = Message;
        }

        public int ErrorCode
        {
            get { return _errorCode; }
        }

        override public string Message
        {
            get { return _Message; }
        }
    }
}

    API сервер отправки принимает запросы вида http://api.****sms.ru?method=send_msg&phone=+79 123456789&text=abcdef, возвращает простейший XML с err_code и результатом выполнения запроса.
    Казалось бы, 20 строчек кода и проблема решена? Нифига, без специального класса для этого не обойтись. Совсем никак. И уж совсем ничего нельзя делать без специального Exception для этого дела.

    Запостил: A1mighty, 16 Января 2012

    Комментарии (5) RSS

    • ava anmiles 16.01.2012 14:23 # +3

      Недавно в дотнете?
      Ответить

    • ava akaDElpher 16.01.2012 14:32 # +3

      вы так говорите, как будто это что-то плохое:(
      * я про написание класса на каждый чих
      Ответить

    • ava depend86 16.01.2012 14:52 # +4

      Написание класса на каждый чих это вполне правильно, я бы еще:
      1) интерфейс выделил для этого класса - для unit тестирования например
      2) вместо KeyValuePair<int,object> возвращал бы объект отдельного класс, который, в первой версии, тупо оборачивал бы KeyValuePair<int,object>
      3) выделение отдельного Exception'а нужно для того, чтобы в том коде который будет использовать класс SMSSender ловить только это исключение и что-то с ним делать
      Ответить

      • ava akaDElpher 16.01.2012 15:02 # +1

        Смущает только
        const string API_URL = "http://api.sms****.ru/?";
        (надо его с другого места брать)
        Ответить

    • ava uaavg 16.01.2012 18:05 # 0

      Вообще, предложенное альтернативное решение и есть говнокод.
      Ответить

    Добавить комментарий