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

    +173

    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
    $pattern = '/[^A-Za-zА-Яа-я0-9]/u';
$phrase = $this->data['ProjectsPhrase']['phrase'];

$phrase = preg_replace($pattern, ' ', $phrase);

$phrase = str_replace('  ', ' ', $phrase);
$phrase = str_replace('  ', ' ', $phrase);
$phrase = str_replace('  ', ' ', $phrase);
$phrase = str_replace('  ', ' ', $phrase);
$phrase = str_replace('  ', ' ', $phrase);
$phrase = str_replace('  ', ' ', $phrase);
$phrase = str_replace('  ', ' ', $phrase);
$phrase = trim($phrase);

    Вот такое говнокодище встретил в проекте. Паранойя такая паранойя…

    Запостил: govnouzver, 17 Февраля 2011

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

    • ava Lure Of Chaos 17.02.2011 17:23 # +4

      это не паранойя, это элементарное невежество
      Ответить

      • ava GhOsTMZ 17.02.2011 17:32 # +1

        не, это чтоб наверняка)
        Ответить

        • ava gegMOPO4 17.02.2011 18:21 # 0

          'a b' -> 'a b' -> 'a b' -> 'a b' -> 'a b'

          Это именно невежество (особенно в контексте первых строчек).

          Кстати, некоторые русские буквы -- йок.
          Ответить

    • ava nepster 17.02.2011 18:15 # −5

      показать все, что скрытоtrim()
      Ответить

      • ava Int 17.02.2011 18:52 # −2

        Нифига, /\s+/ -> ' '
        Ответить

        • ava gegMOPO4 17.02.2011 21:20 # 0

          -> ' '. А потом всё же trim() нужен.
          Ответить

          • ava Int 18.02.2011 11:25 # 0

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

            • ava govnouzver 18.02.2011 14:44 # 0

              Да... судя по коду, семь раз :) что бы наверняка :)
              Ответить

              • ava gegMOPO4 18.02.2011 14:52 # +1

                Данный код сжимает последовательности до 128 пробелов.
                Ответить

            • ava gegMOPO4 18.02.2011 14:48 # 0

              Символы кроме допустимых заменить пробелами, повторяющиеся пробелы сжать в один, пробелы в начале и конце отбросить.

              Первые два шага можно объединить в один (достаточно поставить + после ] в шаблоне), третий делается trim-ом.
              Ответить

            • ava govnouzver 18.02.2011 16:00 # 0

              Кэп?
              Ответить

    • ava Govnocoder#0xFF 18.02.2011 19:01 # 0

      На 1С последовательность команд 6-12 можно было бы заменить одной командой, но с префиксом "НАВЕРНЯКА".
      Ответить

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