Говнокод: по колено в коде.

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5703)
2. C++ (2739)
3. Куча (2337)
4. JavaScript (2030)
5. C# (1929)
6. Java (1503)
7. Си (1102)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (31)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   Tweet

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

   • guest 02.03.2010 19:47 # 0

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

     Ответить

     • ClintEastwood 02.03.2010 20:23 # 0

       от этого еще более суровый :)

       Ответить

   • guest 02.03.2010 21:07 # 0

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

     Ответить

     • 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;
       }

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

       Ответить

       • Amok 03.03.2010 14:27 # 0

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

         Ответить

         • guest 03.03.2010 15:00 # 0

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

           Ответить

         • guest 07.03.2010 11:28 # 0

           давайте развернем вашу проверку:
           

           if (text == undefined ) return undefined;
           if( text == "") return "";
           return text;

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

           Ответить

   • guest 03.03.2010 10:32 # 0

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

     Ответить

   • guest 04.03.2010 14:16 # 0

     if (text.charAt(0) == ' ') {
     return '';
     }
     
     Это типа отрубить голову до мозолей на пятках?

     Ответить

     • guest 04.03.2010 14:31 # 0

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

       Ответить

       • guest 07.03.2010 12:00 # +1

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

         Ответить

   • guest 09.03.2010 20:12 # +1

     return text.replace(/^\s+|\s+$/g, "")

     Ответить

     • guest 09.03.2010 23:24 # 0

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

       Ответить

       • guest 10.03.2010 12:33 # 0

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

         Ответить

     • ClintEastwood 10.03.2010 09:21 # 0

       return text.replace(/^\s*/, '').replace(/\s*$/, '');

       Ответить

       • guest 10.03.2010 12:26 # 0

         или в один проход
         

         return text.replace(/^\s*|\s*$/g, "");

         Ответить

         • babaika 11.03.2010 11:01 # 0

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

           Ответить

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

   Ошибка компиляции комментария:

   1. Гости могут высказаться только в понедельник, среду, четверг или воскресение

   Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

   А не использовать ли нам bbcode?

   • [b]жирный[/b] — жирный
   • [i]курсив[/i] — курсив
   • [u]подчеркнутый[/u] — подчеркнутый
   • [s]перечеркнутый[/s] — перечеркнутый
   • [blink]мигающий[/blink] — мигающий
   • [color=red]цвет[/color] — цвет (подробнее)
   • [size=20]размер[/size] — размер (подробнее)
   • [code=<language>]some code[/code] (подробнее)

   Проверочный код: *