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

    +163

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if ($type=='input') $params['class'] = 'styleInput';
if ($type=='password') $params['class'] = 'stylePassword';
if ($type=='checkbox') $params['class'] = 'styleCheckBox';
if ($type=='select') $params['class'] = 'styleSelect';
if ($type=='multipleSelect') $params['class'] = 'styleMultipleSelect';
if ($type=='textarea') $params['class'] = 'styleTextarea';
if ($type=='groupSelect') $params['class'] = 'styleGroupSelect';

    Конкатенация - враг! Учитесь трудолюбию!

    Запостил: ifeya, 02 Марта 2011

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

    • ava Lure Of Chaos 02.03.2011 13:25 # +3

      тут не просто конкатенация
      во-первых, каждая первая буква становится заглавной,
      а во-вторых, 'checkbox' исключение.

      Лично я бы переименовал стили, что бы они соответствовали $type
      Ответить

      • ava ifeya 02.03.2011 13:27 # +1

        ucfirst тоже враг народа? даже если чекбокс исключение, свича мы не знаем?
        Ответить

        • ava Lure Of Chaos 02.03.2011 13:33 # +2

          ну зачем такие сложности? проще систему сделать логичнее, чем городить исключения(которые из правил) из каждого чиха
          Ответить

          • ava ifeya 02.03.2011 13:34 # −1

            правильно, давайте говнокодить
            Ответить

            • ava Lure Of Chaos 02.03.2011 13:37 # 0

              то ли вы не поняли, то ли я чего-то не понимаю. Если делать так, как я предложил в первом комментарии, то сабж вообще не нужен. Или это, по-вашему, говнюче?
              Ответить

            • ava bugmenot 02.03.2011 18:17 # +1

              это же PHP
              Ответить

              • ava Lure Of Chaos 02.03.2011 18:35 # 0

                это не оправдание. А даже скорее отягчающее обстоятельство ))
                Ответить

      • ava gegMOPO4 03.03.2011 14:47 # 0

        Лучше массив. Мало ли какие исключения будут.
        Ответить

    • ava Skelet0n 02.03.2011 18:02 # −1

      можно просто сделать конструкцию из switch'а и case'ов.
      Ответить

      • ava Lure Of Chaos 02.03.2011 18:36 # +1

        можно. и выложить сюда )
        Ответить

      • ava Vasiliy 02.03.2011 20:08 # 0

        нужно 1 букву $type перевести в верхний регистр и что получится прилепить к "style". Всегда Ваш К.О.
        Ответить

        • ava Lure Of Chaos 02.03.2011 20:42 # 0

          К.О. проморгал вариант чекбокса
          Ответить

          • ava ZX_Spectrum 02.03.2011 23:01 # 0

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

            • ava Lure Of Chaos 02.03.2011 23:14 # 0

              тогда можно вообще ничего не апперкейсить
              Ответить

              • ava ZX_Spectrum 03.03.2011 00:27 # 0

                можно, ведь результат увидит только браузер, а он особых эмоций по этому поводу не испытает.
                поэтому для стилей все можно сократить до
                $params['class'] = $type; // даже без "style".
                Ответить

                • ava Lure Of Chaos 03.03.2011 15:51 # +1

                  поэтому даже можно везде писать $type, а не $params['class']
                  Ответить

                  • ava ZX_Spectrum 03.03.2011 16:18 # 0

                    если элементам потом не присваивается еще какой-нибудь класс
                    $params['class'] += ' ' + $some_class;
                    Ответить

        • ava TarasB 03.03.2011 15:43 # 0

          checkbox тогда не сработает.
          Вообще почему бы массив не завести?
          Ответить

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