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

    +163

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    ...
$str = preg_replace ( array_keys ( $new ), array_values ( $new ), $str );
$str = trim ( $str );
$str = str_replace ( "       ", "_", $str );
$str = str_replace ( "      ", "_", $str );
$str = str_replace ( "     ", "_", $str );
$str = str_replace ( "    ", "_", $str );
$str = str_replace ( "   ", "_", $str );
$str = str_replace ( "  ", "_", $str );
$str = str_replace ( " ", "_", $str );

    Запостил: develx0, 16 Сентября 2013

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

    • ava Vladiator 16.09.2013 19:42 # +1

      Разве этого достаточно?
      $prefix = "";
while (true) {
    $prefix += " ";
    $str = str_replace ( $prefix, "_", $str );
}
      Ответить

      • ava develx0 16.09.2013 19:51 # 0

        И этого достаточно чтобы стать следующим постом на этом сайте.
        Одна строка
        если уж совсем сохранить функционал:
        $str = preg_replace('/ {1,7}/', '_', $str);
        или если упростить:
        $str = preg_replace('/ +/', '_', $str);
        Ответить

        • ava 1024-- 16.09.2013 19:53 # 0

          Так то был зелёный.

          Хотя, автор кода шёл от 7 до 1, а Vladiator - от 1 до бесконечности, это немного в другую сторону.
          Ответить

          • ava kegdan 16.09.2013 20:50 # 0

            ну так если бы автор шел от 1 до семи пробелы бы остались, в этой части он прав)
            Ответить

            • ava 1024-- 16.09.2013 20:56 # +2

              > если бы автор шел от 1 до семи пробелы бы остались
              пробелов бы не осталось, но было бы много _________________
              Именно в этой части Vladiator неправ, поскольку на первой же итерации пробелы бы кончились :)

              Вот если б он пошёл от бесконечной строки с пробелами...
              Ответить

              • ava kegdan 16.09.2013 20:59 # 0

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

                • ava develx0 16.09.2013 21:01 # 0

                  здесь выше по коду автор использовал регулярки
                  $str = preg_replace ( array_keys ( $new ), array_values ( $new ), $str );
                  а с пробелами что-то лоханулся
                  Ответить

              • ava Vladiator 16.09.2013 21:05 # +3

                Фикс:
                $spaces = "";
while (true) {
    $spaces += " ";
}
while ($spaces) {
    $str = str_replace ( $spaces, "_", $str );
    $spaces = substr($spaces, 0, -1);
}
                Ответить

                • ava develx0 16.09.2013 21:07 # 0

                  Чем дальше, тем веселей %-)
                  Ответить

                • ava 1024-- 16.09.2013 21:20 # +2

                  Если я ничего не напутал,
                  function repl ( $str, $spaces = "", $down = FALSE) {
  return $down ?
    strlen($spaces) > 0 ? repl (str_replace ( $spaces, "_", $str ), substr($spaces, 1), $down) : $str
  : is_infinite(strlen($spaces)) ? repl($str, $spaces, TRUE) : repl($str, $spaces + ' ');
}
                  Ответить

                  • ava 1024-- 16.09.2013 21:43 # +1

                    Таки напутал. Надо было точку использовать и немного скобок.
                    function repl ( $str, $spaces = "", $down = FALSE) {
  return $down ?
    (strlen($spaces) > 0 ? repl (str_replace ( $spaces, "_", $str ), substr($spaces, 1), $down) : $str)
  : (is_infinite(strlen($spaces)) ? repl($str, $spaces, TRUE) : repl($str, $spaces . ' '));
}

                    Пример для малых чисел: http://ideone.com/1KaaRe
                    С вынесением алгоритма проверки достаточности в аргумент: http://ideone.com/Z3VzJz
                    Ответить

        • ava Vladiator 16.09.2013 19:53 # +1

          Тут же не занимаются улучшением кода.
          Ответить

          • ava bot 17.09.2013 07:50 # 0

            Если соединить говнокод и рефакторинг, получится говнофакторинг.
            Ответить

      • ava Dummy00001 16.09.2013 20:38 # +8

        Однозначно нет. С рекурсией:
        function TheBestReplaceEver($str, $prefix = " ")
{
	$str = str_replace ( $prefix, "_", $str );
	$str = TheBestReplaceEver( $str, $prefix." " );
	return $str;
}
TheBestReplaceEver( $str );

        Код не только никогда не завершится, но благодяря использованию рекурсии он так же и php процесс положит. А если повезет и сервак криво сконфигурирован - то и сервак посвопит несколько минут. В добавок, все это теперь оформлено в удобную переиспользуемую функцию, которую можно вызывать откуда угодно.
        Ответить

    • ava eth0 16.09.2013 21:11 # +2

      Никак не припомню, но тут подобное бывало-с, и не раз.
      Ответить

    • ava myaut 16.09.2013 22:02 # +3

      python:

      >>> '_'.join("1 2 3 4".split())
      '1_2_3_4'
      Ответить

      • ava bormand 16.09.2013 22:05 # +1

        perl: s/\s+/_/g
        Ответить

      • ava guest 17.09.2013 12:43 # +2

        '_'.join("1 2 3 4".split())
        наркоманы же. На руби порядок наоборот:
        "1 2 3 4".split().join('_')
        Ответить

        • ava myaut 17.09.2013 15:02 # +1

          Синтаксически это логичнее, да. Но почему список абстрактных элементов должен иметь join(), склеивающий строки?
          Какой это имеет смысл:
          [1, 0.5, None].join('_')

          ?

          Конечно можно для всех элементов применить __str__()...
          Ответить

          • ava myaut 17.09.2013 15:10 # +1

            Да, ну str.join в Python жрет любой итерируемый объект, а вот не всякий такой объект содержит метод join()
            Ответить

    • ava guest 17.09.2013 12:44 # −16

      показать все, что скрытоИщу партнёра для секса. Есть своя квартира. Анал, орал, группа. Пиши мне на [email protected]
      Ответить

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