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

    • ava guest 24.10.2014 21:50 # 0

      function verifyType($type){
      $type_list = array('cs','css','cssold','samp');
      if(in_array($type, $type_list)) $this->serverType = $type;
      else Error::add('Ошибка', 'Тип сервера указан неверно!');
      }
      Ответить

      • ava barbarbar 14.11.2014 04:58 # 0

        Какой ты умница. Но ты неправильно мыслишь.
        Ответить

        • ava guest 14.11.2014 09:08 # 0

          Тут видимо принято говнокодом на говнокод отвечать =)
          Ответить

          • ava barbarbar 14.11.2014 09:59 # 0

            Я немного о другом. Автор говнокод запостил верно, потому что по правилам надо было в первом case перечислить все варианты и после двоеточия описать один блок кода, без дублирования кода. Например, case 'bla', 'blabla', 'blablabla': некий код; break;

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

            А то, что гость предлагал, это как раз нарушение логики, ненужное усложнение читабельности и прямой путь к будущему говнокоду и рефакторингу.
            Ответить

            • ava guest 14.11.2014 10:07 # 0

              Чёт я с первого раза не допрочел чтоли =) Такто да, он прально написал, за исключением того, что не приинклудил файл
              Ответить

              • ava barbarbar 14.11.2014 10:19 # 0

                Скажем так, на небольшой говнокод тут принято отвечать говнокодом помассивнее:)
                Ответить

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