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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2340)
4. JavaScript (2030)
5. C# (1930)
6. Java (1503)
7. Си (1103)
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 (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   +111

   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

   // todo
   /// <summary>
   /// Генерация пароля из GUID
   /// </summary>
   /// <param name="guid">GUID</param>
   /// <returns>пароль</returns>
   public string PasswordByGuid(string guid)
   {
       return guid[33].ToString()
           + guid[28].ToString()
           + guid[2].ToString()
           + guid[10].ToString()
           + guid[21].ToString()
           + guid[15].ToString();
   }

   Коммерческий проект :)

   Запостил: ddv_demon, 08 Ноября 2012

   Tweet

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

   • bormand 08.11.2012 16:53 # 0

     Самое забавное, если этот guid это по совместительству является ключем в таблице юзверей...

     Ответить

   • absolut 08.11.2012 16:56 # +3

     Не оптимально! Где StringBuilder, блджад?

     Ответить

     • bormand 08.11.2012 17:39 # 0

       Как это где? В цепочке операторов + же он. Его не видно, но он есть.
       
       P.S. Кстати, а если я пишу "a"+b, то тут тоже генерится стрингбилдер?

       Ответить

       • absolut 08.11.2012 21:16 # +1

         посмотрел ILDASM'ом в ГК получился вызов System.String::Concat. Никаких StringBuilder'ов.

         Ответить

         • bormand 08.11.2012 21:24 # 0

           А. Это же шарп. Сорри... спутал с жабкой.

           Ответить

           • 3.14159265 08.11.2012 21:32 # −1

             Фу говно. Неужели они не могли своровать позаимствовать и этот нюанс.

             Ответить

             • guest 26.02.2013 02:41 # 0

               concat немного производительней чем в stringbuilder, особенно для аргументов меньше 5 штук, в таком случае буквально нулевой оверхед. при где-то 5 аргументах и более создаётся неявный массив строк (params Object[] как в яве в принципе), тоже по идее должен быть немного производительнее чем StringBuilder

               Ответить

           • LispGovno 09.11.2012 09:07 # +1

             StringBuilder в шарпе есть. Но невидимые дела такого рода под полом компилятор не творит.

             Ответить

   • mr.The 08.11.2012 19:23 # +2

     Вспоминается этот http://xkcd.com/221/ комикс.

     Ответить

     • roman-kashitsyn 08.11.2012 23:44 # +6

       ждём возмущений Тараса и ссылку на славянскую версию

       Ответить

       • Fai 09.11.2012 00:44 # 0

         int getRandom(int from, int to)
         {
              return (from + to) / 2
         }

         
         
         Так покруче.

         Ответить

         • bormand 09.11.2012 05:41 # 0

           Нет, тут нет случайности. А там все-таки кости бросали ;)

           Ответить

           • absolut 09.11.2012 09:14 # −1

             а если так:

             int getRealRandom(int from, int to)
             { 
                return getRandom( 100500, 42); 
             }

             ?

             Ответить

             • bormand 09.11.2012 09:38 # −1

               int getRealRandom(int from, int to)  
                   static int n = getRandom( 100500, 42)
                   return n;
               }

               
               Если конечно шарп умеет статики в функциях.

               Ответить

               • absolut 09.11.2012 09:50 # +3

                 >Если конечно шарп умеет статики в функциях.
                 Не умеет. Нужно поле делать.

                 Ответить

           • guest 09.11.2012 17:30 # +1

             этоже мат. ожидание случ. величины
             всё правильно!

             Ответить

       • TarasB 09.11.2012 15:23 # +2

         На хкцд.ру этого номера нет, поэтому я промолчал.

         Ответить

         • absolut 09.11.2012 15:44 # +3

           ну модеров-то посрамил хотя бы?

           Ответить

           • TarasB 09.11.2012 17:23 # +1

             модеров говнокода?

             Ответить

             • guest 09.11.2012 17:48 # +1

               Ну а кого же еще?
               moderator

               Ответить

             • absolut 09.11.2012 18:08 # +2

               модеров хкцд.ру

               Ответить

               • bormand 09.11.2012 18:30 # +1

                 Злые модераторы хкцд.рфру удалили столь нужный нам рисунок?

                 Ответить

               • TarasB 09.11.2012 18:39 # +1

                 Ну может они не смогли перевести.

                 Ответить

                 • bormand 09.11.2012 18:49 # +2

                   Функция ПолучитьСлучайноеЧисло()
                       Возврат 4; // получено броском правильной игральной кости
                                  // случайность гарантирована
                   КонецФункции

                   Ответить

                   • eth0 09.11.2012 19:00 # 0

                     Честной же.

                     Ответить

                     • Xtasy 09.11.2012 19:11 # 0

                       "честным броском кости" тогда уж.
                       И возвращаемый тип куда-то пропал (или в 1С он не указывается?)

                       Ответить

                       • bormand 09.11.2012 20:08 # 0

                         > "честным броском кости" тогда уж.
                         Окай. Не специалист в ролёвках ;(
                         
                         > И возвращаемый тип куда-то пропал
                         И в 1с тоже. Но судя по соседним говнокодам - таки не указывается.

                         Ответить

                       • eth0 09.11.2012 20:37 # 0

                         Да, немного протормозил, mea culpa. Решил придраться к одному слову, вот и вышло не ахти.

                         Ответить

     • LispGovno 09.11.2012 09:04 # +1

       int getRandom()
       {
           int a;
           return a;
       }

       Ответить

       • Steve_Brown 09.11.2012 10:17 # +2

         Вот это будет истинная случайность. Поскольку нельзя предсказать поведение (по крайней мере, без знания контекста до деталей). В отличие от псевдослучайного rand...

         Ответить

       • absolut 09.11.2012 10:23 # +2

         жалко только, что не скомпилируется. C# же

         Ответить

         • LispGovno 09.11.2012 10:38 # 0

           http://ideone.com/e89ici
           http://ideone.com/afMXNG

           Ответить

           • LispGovno 09.11.2012 10:40 # +1

             А блин, это же раздел не крестов...

             Ответить

             • LispGovno 09.11.2012 10:45 # 0

               http://ideone.com/OlbbTl
               Может как-нибудь так? У кого шарп под рукой с нужным ключем компиляции?

               Ответить

               • absolut 09.11.2012 11:20 # +3

                 во чо нагуглил на знакомого товарища по ГК:
                 

                 http://stackoverflow.com/questions/6054750/no-unassigned-local-variable-error

                 Ответить

                 • LispGovno 09.11.2012 11:22 # 0

                   Я думал так сделать, если первый вариант не сработает.

                   Ответить

           • Steve_Brown 09.11.2012 10:41 # 0

             http://ideone.com/8iE54O

             Ответить

           • absolut 09.11.2012 11:08 # 0

             ну а что ж ты само значение-то не вывел :)
             http://ideone.com/kIqyOg

             Ответить

       • Elvenfighter 09.11.2012 22:41 # 0

         int getRandom() {
             int a = (int) new int;
             delete (int*)a;
             return a;
         }

         
         
         Нужно больше энтропии!

         Ответить

         • Elvenfighter 09.11.2012 22:54 # 0

           Или даже так:
           

           int getRandom() {
               int *a = new int;
               int b = *a;
               delete a;
               return b;
           }

           Ответить

           • Fai 09.11.2012 23:14 # −1

             > int b = *a;
             > delete a;
             > return b;
             
             Неверно. В переменной b уже будет лежать 0, когда освободим память по a - ничего не произойдёт, следовательно вернётся 0.

             Ответить

             • Elvenfighter 09.11.2012 23:36 # −1

               a - указатель на случайный кусок памяти, new для надёжности.

               Ответить

           • absolut 09.11.2012 23:26 # +2

             а ещё можно перепочитать топик и название раздела

             Ответить

         • Fai 09.11.2012 23:13 # 0

           Хорошо работает. Только сколько не вызывай одно и тоже число на каждый запуск программы.

           Ответить

   • Steve_Brown 09.11.2012 10:24 # +6

     public static string PasswordByGuid(Guid guid)
     {
         string s = guid.ToString();
         return s[8] + s[13] + s[18] + s[23];
     }

     Ответить

     • bormand 09.11.2012 11:16 # +3

       ----?

       Ответить

       • Steve_Brown 09.11.2012 14:36 # +1

         Ага :)

         Ответить

     • absolut 09.11.2012 14:54 # +2

       отличный выбор, а главное, случайный

       Ответить

     • inkanus-gray 10.11.2012 01:01 # +2

       function PasswordByGuid(guid)
       {
           var s = guid.toString();
           var t = '0x' + s.replace(/-/g, '-0x');
           return eval(t);
       }

       Ответить

   • body90 24.11.2012 00:34 # +1

     Просто слов нет! :)

     Ответить

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

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

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

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

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

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

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