Си диез / Говнокод #9124 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 A1mighty, (Updated )

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

  • вы так говорите, как будто это что-то плохое:(
    * я про написание класса на каждый чих
    Ответить
  • Написание класса на каждый чих это вполне правильно, я бы еще:
    1) интерфейс выделил для этого класса - для unit тестирования например
    2) вместо KeyValuePair<int,object> возвращал бы объект отдельного класс, который, в первой версии, тупо оборачивал бы KeyValuePair<int,object>
    3) выделение отдельного Exception'а нужно для того, чтобы в том коде который будет использовать класс SMSSender ловить только это исключение и что-то с ним делать
    Ответить
    • Смущает только
      const string API_URL = "http://api.sms****.ru/?";
      (надо его с другого места брать)
      Ответить
  • Вообще, предложенное альтернативное решение и есть говнокод.
    Ответить

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

Из-за тебя ушел bormand, guest!

    А не использовать ли нам bbcode?


    8