1. PHP / Говнокод #13624

    +157

    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
    85. 85
    86. 86
    87. 87
    88. 88
    public function getIcon()
        {
            switch($this->name)
            {
                case 'Пятигорск':
                    return 'inn_pyatigorsk';
                    break;
                case 'Южная':
                    return 'inn_uzhnaya';
                    break;
                case 'Кавказ':
                    return 'inn_kavkaz';
                    break;
            }
        }
    
    
        public function getBlockId()
        {
            switch($this->name)
            {
                case 'Пятигорск':
                    return 'pyatigorsk';
                    break;
                case 'Южная':
                    return 'uzhnaya';
                    break;
                case 'Кавказ':
                    return 'kavkaz';
                    break;
            }
        }
    
    
    
        public function getImageBlockId()
        {
            switch($this->name)
            {
                case 'Пятигорск':
                    return 'box-pyatigorsk';
                    break;
                case 'Южная':
                    return 'box-uzhnaya';
                    break;
                case 'Кавказ':
                    return 'box-kavkaz';
                    break;
            }
        }
    
    
    
        public function getSliderId()
        {
            switch($this->name)
            {
                case 'Пятигорск':
                    return 'slider_pyatigorsk';
                    break;
                case 'Южная':
                    return 'slider_uzhnaya';
                    break;
                case 'Кавказ':
                    return 'slider_kavkaz';
                    break;
            }
        }
        
        
    
        public function getBooKId()
        {
            $result = '';
            switch($this->name)
            {
                case 'Пятигорск':
                    $result = 'book_pyatigorsk';
                    break;
                case 'Южная':
                    $result = 'book_uzhnaya';
                    break;
                case 'Кавказ':
                    $result = 'book_kavkaz';
                    break;
            }
            return $result;
        }

    рефакторинг "по Украински"

    Запостил: oxser, 19 Августа 2013

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

    • И давно хохлы в Пятигорске говношлёпают?
      Ответить
    • "по-украински" же...

      И не отдавайте на дешёвый фриланс (неважно в какую страну) свои проекты, если ожидаете качества
      Ответить
    • Самое интересно - в первых 4 ретурн прямо в кейсе, а в последней - специальная переменная... Неужели писалось без копипасты?
      Ответить
      • Обычно такое случается когда копипасту меняют, но в одном месте.
        Ответить
        • Причем меняют на более идиотский вариант? (Кейс-то всё равно не проваливающийся и я не могу представить ситуаций, когда прямые ретурны и введение переменной дали бы разные результаты)

          Да, и классика - бряки после ретурнов. Писал явно Пинки из "Пинки и Брейн" Спилберговского.
          Ответить
          • Я тоже обычно оставляю бряки, просто чтоб была однородная структура (глазу проще видеть break; в одном отступе, чем несколько разных вариантов break/return/throw), да и если в процессе переделки потом return становится не нужен, меньше дописывать. Элементарное удобство. Что здесь криминального?
            Ответить
    • Я бы захуячил вложенную структуру данных, типа data['Пятигорск']['book'].
      Ответить
    • Зато в класс завернули!
      Ответить
    • и бряк после ретурна. ехоу!
      Ответить
    • Классический индусский код as is
      Ответить

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