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

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

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

1. PHP (5705)
2. C++ (2739)
3. Куча (2357)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1115)
8. Pascal (647)
9. Python (583)
10. 1C (538)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

1. C# / Говнокод #11800

   +133

   1. 1

   string[] res = value.Split("$".ToCharArray(), StringSplitOptions.None);

   Запостил: shtaff, 19 Сентября 2012

   Tweet

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

   • movaxbx 19.09.2012 16:07 # 0

     value.Split("new char[] {'$'}, StringSplitOptions.None);

     
     Говно в этом?

     Ответить

     • shtaff 19.09.2012 16:10 # +2

       Я бы даже написал value.Split('$').
       А вообще переводить односимвольную строчную константу в массив символов - это что-то связаное с магией, по моему.

       Ответить

       • movaxbx 19.09.2012 16:13 # 0

         Так там либо массив чаров, либо строк нужен.
         http://msdn.microsoft.com/ru-ru/library/system.string.split.aspx

         Ответить

         • shtaff 19.09.2012 16:18 # +5

           public string[] Split( рarams char[] separator )
           
           Ключевое слово params даёт возможность передавать элементы массива просто через запятую.

           Ответить

           • movaxbx 19.09.2012 16:26 # +4

             >рarams
             Буду знать

             Ответить

             • abatishchev 20.09.2012 12:37 # +1

               params только в перегрузке без StringSplitOptions
               
               http://msdn.microsoft.com/en-us/library/b873y76a.aspx
               http://msdn.microsoft.com/en-us/library/ms131448.aspx

               Ответить

           • LispGovno 22.09.2012 22:35 # +1

             Проще вызвать ToCharArray, чем перечислять 'с','и','м','в','о','л','ы' через запятую. После ввода ".To" IDE сама дополнит.

             Ответить

             • wvxvw 23.09.2012 01:20 # 0

               (defun string-to-chars (start end)
                 (interactive "r")
                 (let ((replacement
                        (mapconcat #'identity
                         (mapcar #'char-to-string
                                 (buffer-substring start end)) "' + '")))
                   (kill-region start end)
                   (insert (concat "'" replacement "'"))))

               
               
               Это смотря какая ИДЕ.

               Ответить

   • пъянь 22.09.2012 20:36 # 0

     А такой код норм:
     string[] res = value.Split("$!%:?*()_+".ToCharArray(), StringSplitOptions.None);
     
     ?

     Ответить

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

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

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

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

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

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

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