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

    +112

    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
    public static class EntryPoint
    {
        private class NotFoundException : ApplicationException
        {
            public FoundException(int id, SourceType type)

            private SourceType Type { get; set; }
            private int Id { get; set; }
        }

      private class InProgressException : ApplicationException
        {
            public InProgressException(int id, SourceType type)
            {
                Type = type;
                Id = id;
            }

            private UserViewSourceType Type { get; set; }
            private int Id { get; set; }
        }

     public static void Analyze(SourceType type, IEnumerable<int> dataId)
        {
            try
            {
                initialize(type, dataId);
            }
            catch (InProgressException)
            {  
// немного говнокода
            }
            catch (NotFoundException)
            {
// немного говнокода
            }
}

    Для того что бы внутри статического класса между двумя процедурами наладить связь (Analyze - initilize) человек определил два новых типа exceptions (private) и дал им дополнительные свойства которые нигде не используются. Exceptions эти больше нигде кроме initialize не кидаются и нигде кроме Analyze не ловятся. Ну и initialize больше ниоткуда кроме Analyze не вызывается.
    В итоге имеем два лишних класса и тучу кода который можно без помех удалить, чем я собственно и занимаюсь :)

    Запостил: eval_2009, 05 Января 2011

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

    • ava guest 05.01.2011 20:29 # −1 

      class NotFoundException
        public FoundException(int id, SourceType type)
      Фейк.
      Ответить

      • ava eval_2009 05.01.2011 20:32 # 0

        Как угодно.
        Я слегка переименовываю некоторые классы что бы убрать несущественную либо специфичную для компании информацию
        Ответить

    • ava guest 05.01.2011 20:32 # 0

      >InProgressException
      Очень интересное исключение... Оповещает о прогрессе? о_О
      Ответить

      • ava absolut 05.01.2011 23:54 # 0

        > Оповещает о прогрессе?
        Скорее всего о незавершенной операции
        Ответить

    • ava Анонимус 06.01.2011 16:26 # +2

      за использование исключений в неисключительных ситуациях надо вешать вниз головой
      Ответить

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