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

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

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

1. PHP (5708)
2. C++ (2740)
3. Куча (2389)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1117)
8. Pascal (648)
9. Python (591)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (37)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   0

   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
   14. 14
   15. 15
   16. 16

   public class HttpQueryNameValueCollection : NameValueCollection
       {
           private ParallelLoopResult _parallelLoopResult;
   
           public override string ToString()
           {
               List<string> result;
               result = new List<string>();
               _parallelLoopResult = Parallel.ForEach(AllKeys, p =>
               {
                   if (BaseGet(p) != null)
                       result.Add(p + "=" + Get(p));
               });
               return string.Join("&", result);
           }
       }

   Формирование ссылки с query-параметрами недостаточно быстро, поэтому нужно это сделать в параллель :D

   Запостил: adoconnection, 24 Октября 2019

   Tweet

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

   • guest8 24.10.2019 14:51 # −999

     показать все, что скрытоvanished

     Ответить

   • MAPTbIwKA 24.10.2019 15:32 # 0

     параллельный код в toString это неожиданно:)

     Ответить

   • Her 24.10.2019 16:14 # 0

     а где lock, synchronized, critical section и вот это вот все для доступа из разных тредов?

     Ответить

     • MAPTbIwKA 24.10.2019 17:50 # 0

       действительно
       
       тут либо гонки, Либо неебические расходы на синхронизацию

       Ответить

     • guest8 24.10.2019 17:56 # −999

       показать все, что скрытоvanished

       Ответить

       • MAPTbIwKA 24.10.2019 18:12 # 0

         зависит от потокобезопасности List
         вероятно, может и распидорасить, если он не потокбезопасен

         Ответить

         • superhackkiller197 24.10.2019 23:28 # 0

           Если делать по уму - ничего не распидорасит. Для тебя, глупая мартышка, ничто не является безопасным, и в руки тебе ничего давать нельзя.

           Ответить

         • guest8 24.10.2019 23:30 # −999

           показать все, что скрытоvanished

           Ответить

           • gost 29.10.2019 14:29 # 0

             Запили бота, который будет под каждым комментарием уёбков оставлять такую пометку. В наше трудное время хрен поймёшь, кто прикалывается, а кто реально уёбок.

             Ответить

       • adoconnection 29.10.2019 13:43 # 0

         В лист добавится меньше записей, чем планировал кастующий параллелизм.
         
         Пруф:
         

         static void Main(string[] args)
         {
             List<string> result = new List<string>();
         
             Parallel.ForEach(Enumerable.Range(1, 100000), p =>
             {
                     result.Add(p.ToString());
             });
         
             Console.WriteLine(result.Count);
             Console.ReadKey();
         }

         Ответить

         • adoconnection 29.10.2019 14:13 # 0

           еще можно поймать IndexOutOfRangeException при попытке записи в лист

           Ответить

         • guest8 29.10.2019 14:33 # −999

           показать все, что скрытоvanished

           Ответить

     • guest8 29.10.2019 14:52 # −999

       показать все, что скрытоvanished

       Ответить

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

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

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

   Где здесь C++, guest?!

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

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

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