"PHP" / Говнокод #13624 Ссылка на оригинал

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
  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 oxser, (Updated )

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

  • ava DBdev # Ссылка на оригинал 0

    "по-украински" же...

    И не отдавайте на дешёвый фриланс (неважно в какую страну) свои проекты, если ожидаете качества
    Ответить

  • ava kegdan # Ссылка на оригинал 0

    Самое интересно - в первых 4 ретурн прямо в кейсе, а в последней - специальная переменная... Неужели писалось без копипасты?
    Ответить

    • ava 3.14159265 # Ссылка на оригинал 0 ⇈

      Обычно такое случается когда копипасту меняют, но в одном месте.
      Ответить

      • ava vistefan # Ссылка на оригинал 0 ⇈

        Причем меняют на более идиотский вариант? (Кейс-то всё равно не проваливающийся и я не могу представить ситуаций, когда прямые ретурны и введение переменной дали бы разные результаты)

        Да, и классика - бряки после ретурнов. Писал явно Пинки из "Пинки и Брейн" Спилберговского.
        Ответить

        • ava Lowezar # Ссылка на оригинал 0 ⇈

          Я тоже обычно оставляю бряки, просто чтоб была однородная структура (глазу проще видеть break; в одном отступе, чем несколько разных вариантов break/return/throw), да и если в процессе переделки потом return становится не нужен, меньше дописывать. Элементарное удобство. Что здесь криминального?
          Ответить

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

Где здесь C++, guest?!

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

    ↑
    8
    ↓