1. JavaScript / Говнокод #2704

    +166.8

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    function  trim(text) {
  if (text.length == 0) {
    return text;
  }
  if (text.charAt(0) == ' ') {
    return '';
  }
  if (text.charAt(text.length-1) == ' ') {
    text = text.substring(0, text.length-1);
  }
  return text;
}

    суровый китайский trim...

    Запостил: ClintEastwood, 02 Марта 2010

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

    • ava guest 02.03.2010 19:47 # 0

      Не просто китайский, а ещё и криво работающий. Мде...
      Ответить

    • ava guest 02.03.2010 21:07 # 0

      Он был бы суров, если бы был рекурсивен.
      Ответить

      • ava guest 02.03.2010 23:33 # +1 

        function trim(text)
{
  if (text.charAt(0) == " ")
  {
    return trim(text.substr(1));
  }
  
  if (text.charAt(text.length - 1) == " ")
  {
    return trim(text.substr(0, text.length - 1));
  }
  
  return text;
}

        Рекурсивно, но не китай.
        Ответить

        • ava Amok 03.03.2010 14:27 # 0

          Кошерно было бы сделать проверку на undefined.

          К тому же могут возникнуть проблемы на пустой строке ("").

          т.е. добавить проверку:
          if (text == undefined || text.length == 0)
          {
          return text; //можно по желанию return "";
          }
          Ответить

          • ava guest 03.03.2010 15:00 # 0

            Да, не сообразил как-то.
            Ответить

          • ava guest 07.03.2010 11:28 # 0

            давайте развернем вашу проверку:
            if (text == undefined ) return undefined;
if( text == "") return "";
return text;

            вот теперь проблем не будет точно ))
            Ответить

    • ava guest 03.03.2010 10:32 # 0

      А может он вызывается в бесконечном цикле до тех пор пока что вернулось не равно тому что ушло в функцию.
      Ответить

    • ava guest 04.03.2010 14:16 # 0

      if (text.charAt(0) == ' ') {
      return '';
      }

      Это типа отрубить голову до мозолей на пятках?
      Ответить

      • ava guest 04.03.2010 14:31 # 0

        Потому и криво работающий, что при наличии начального пробела глушится под корень вся строка. А ещё концевой пробел удаляется только один.
        Ответить

        • ava guest 07.03.2010 12:00 # +1

          Ничего, скоро автор узнает что кроме "чистых" пробелов еще много чего надо отрезать.... и допишет наверное ещё строк 20))
          Ответить

    • ava guest 09.03.2010 20:12 # +1 

      return text.replace(/^\s+|\s+$/g, "")
      Ответить

      • ava guest 09.03.2010 23:24 # 0

        Неправильно. А текстовое поле?
        Ответить

        • ava guest 10.03.2010 12:33 # 0

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

      • ava ClintEastwood 10.03.2010 09:21 # 0

        return text.replace(/^\s*/, '').replace(/\s*$/, '');
        Ответить

        • ava guest 10.03.2010 12:26 # 0

          или в один проход
          return text.replace(/^\s*|\s*$/g, "");
          Ответить

          • ava babaika 11.03.2010 11:01 # 0

            или набор велосипедов (аж 13 штук) с бенчмарком
            http://blog.stevenlevithan.com/archives/faster-trim-javascript
            Ответить

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