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

    +162.5

    1. 1
    2. 2
    $require = 'email|name|phone';
foreach (explode('|', $required) as $field)

    найдено при рефакторинге

    Запостил: Stormherz, 12 Января 2010

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

    • ava Stormherz 12.01.2010 11:00 # 0

      $required в начале, синтаксически все верно, случайно дел ткнул
      Ответить

    • ava guest 12.01.2010 14:13 # 0

      в чем говнокод?
      Ответить

      • ava Stormherz 12.01.2010 14:27 # +3

        ну, к примеру, можно было бы не делать строку, чтобы потом разбить её на массив, а сделать сразу массив
        Ответить

    • ava guest 12.01.2010 15:45 # 0

      Perl like код, я за такое руки рублю, сэкономил на коде, потратил на производительности.
      Ответить

    • ava homakov 12.01.2010 18:08 # 0

      -1.
      Порой такие конструкции удобнее, чем писать Array("word") etc, говно код, но НЕОДНОЗНАЧНЫЙ, может быть и полезен.
      Ответить

      • ava Stormherz 12.01.2010 18:59 # +1

        согласен, но дело не в удобстве. во-первых читаемость хуже, во-вторых, как правильно выше сказали, производительность ниже, хоть и песчинка по размеру, но если человек привык так делать, в других местах будет не лучше. Я дальше по коду проглядывал, думал может функционально необходимо было такую строку иметь, так нет, только для этого и была использована =)
        Ответить

    • ava guest 13.01.2010 11:36 # −2

      где смеятся?
      Ответить

    • ava guest 13.01.2010 13:44 # 0

      Я понимаю, что это медленнее работает. Но иногда так делаю и нисколько не переживаю по этому поводу.
      Ответить

    • ava hardcoder 13.01.2010 19:33 # +1

      Всё-таки скорее говнокодно, чем кодогодно ;)
      Ответить

    • ava nolka4 14.01.2010 07:34 # 0

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

    • ava guest 14.01.2010 12:14 # 0

      В JavaScript я частенько использую такое для сокращения объёма JS-файла (когда, допустим, в JS захардкоден большущий массив, и писать 'elem1|elem2|...|elemN'.split('|') оказывается заметно выгоднее, чем выписывать каждый элемент отдельно). Но JS-то ладно, ну выполнится такой код у юзера на компе за сколько-то миллисекунд, ну и не жалко. Но в PHP-то? Вы действительно хотите, чтобы эта дрянь выполнялась тысячи, миллионы раз на сервере, которому и без того есть чем заняться?
      Ответить

    • ava Suor 14.01.2010 13:58 # 0

      Нашли на чём оптимизировать, это вообще стоит нисколько.
      Просто уродская конструкция array('', '', ...) достала человека.
      На перле это вообще будет:
      for my $field (qw(email name phone)) {
      ...
      }
      Так что не говнокод, говноязык.
      Ответить

    • ava mrbig66 14.01.2010 18:19 # 0

      Не говнокод.
      Ответить

    • ava guest 14.01.2010 22:39 # 0

      не говнокод +1
      Ответить

    • ava danxill 16.01.2010 14:49 # 0

      Это не говнокод, согласен, но по правилам для того и создавались массивы чтоб использовать их. А не использовать explode('|'); как сделанно в данном случае!
      Ответить

    • ava AHaHkacmHbIu_nemyx 25.08.2021 17:53 # 0

      vanished
      Ответить

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