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

    +159

    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
    function verifyType($type){
            switch ($type){
                case 'cs':
                    $this->serverType = $type;
                    require_once $this->serverType.'Inc.php';
                    break;
                case 'css':
                    $this->serverType = $type;
                    require_once $this->serverType.'Inc.php';
                    break;
                case 'cssold':
                    $this->serverType = $type;
                    require_once $this->serverType.'Inc.php';
                    break;
                case 'samp':
                    $this->serverType = $type;
                    require_once $this->serverType.'Inc.php';
                    break;
                default:
                    Error::add('Ошибка', 'Тип сервера указан неверно!');
            }
        }

    Очередной высер чудо-программистов. Еще одна панель управления игровыми серверами

    Запостил: Onotole, 18 Октября 2014

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

    • function verifyType($type){
      $type_list = array('cs','css','cssold','samp');
      if(in_array($type, $type_list)) $this->serverType = $type;
      else Error::add('Ошибка', 'Тип сервера указан неверно!');
      }
      Ответить
      • Какой ты умница. Но ты неправильно мыслишь.
        Ответить
        • Тут видимо принято говнокодом на говнокод отвечать =)
          Ответить
          • Я немного о другом. Автор говнокод запостил верно, потому что по правилам надо было в первом case перечислить все варианты и после двоеточия описать один блок кода, без дублирования кода. Например, case 'bla', 'blabla', 'blablabla': некий код; break;

            switch на то и switch, что легко в такой код вписать новые условия и новые варианты действий.

            А то, что гость предлагал, это как раз нарушение логики, ненужное усложнение читабельности и прямой путь к будущему говнокоду и рефакторингу.
            Ответить
            • Чёт я с первого раза не допрочел чтоли =) Такто да, он прально написал, за исключением того, что не приинклудил файл
              Ответить
              • Скажем так, на небольшой говнокод тут принято отвечать говнокодом помассивнее:)
                Ответить

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