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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2337)
4. JavaScript (2030)
5. C# (1929)
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. Java / Говнокод #12506

   +97

   1. 1
   2. 2
   3. 3

   Integer [] jh = new Integer [1];
   Integer j0 = new Integer(17);
   jh[0]= j0;

   Заполняем массив.

   Запостил: 3.14159265, 30 Января 2013

   Tweet

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

   • bormand 30.01.2013 21:06 # +1

     Правильно будет как-то так?

     Integer[] jh = new Integer[] {17};

     Ответить

     • 3.14159265 30.01.2013 21:08 # +1

       Да. В 1.4 было чуть сложнее из-за отсутствия автобоксинга, но в целом с сишком мало что изменилось.
       Самое смешное что этот массив потом идёт аргументом в варарг.
       И потом эти люди будут рассказывать о многословности жабы.
       Полагаю это приятней чем крестократкость #12488.

       Ответить

       • bormand 30.01.2013 21:21 # +1

         Ну вот кстати обидно, что как в с++11 нельзя:

         // хочется вот так
         vibrator.vibrate({0, 300, 300, 300}, -1);
         // но нужно так
         vibrator.vibrate(new long[] {0, 300, 300, 300}, -1);

         Ответить

         • defecate-plusplus 30.01.2013 22:02 # +12

           > new long vibrator
           иногда мне трудно даже вообразить, что программируют постояльцы уютненького

           Ответить

           • bormand 30.01.2013 22:07 # +3

             Всего лишь особый вибро-сигнал для говнотифи, который нельзя спутать со входящей смской или звонком.

             Ответить

             • absolut 30.01.2013 22:16 # +5

               коричневая частота?

               Ответить

               • bormand 30.01.2013 22:20 # 0

                 Жалко, что хоть ведроидскому светодиоду передается rgba, но работают только красный синий и зеленый. Умел бы он микшировать цвета - сделал бы коричневую индикацию ;(

                 Ответить

                 • absolut 31.01.2013 06:03 # 0

                   Дело не только в цвете. Главное, чтобы эффект был.

                   Ответить

         • 3.14159265 31.01.2013 15:47 # +1

           >// хочется вот так
           >{0, 300, 300, 300}
           Больше js напоминает. Да и зачем язык усложнять? Ну сделайте шаблонный варарг метод array, который вернет массив.
           vibrator.vibrate(Make::array(0, 300, 300, 300), -1);
           И не надо запоминать какие скобки что и зачем делают. И код читабельный.
           

           Ответить

           • defecate-plusplus 31.01.2013 16:47 # +1

             что делают {1, 2, 3, 4} интуитивно понятно еще с сишки
             в которой и 20 лет назад работало int array[] = {1, 2, 3, 4};
             просто в с++11 эту схему инициализации наконец-то обобщили и предоставили способ передачи {....} в конструктор/метод/функцию

             Ответить

             • 3.14159265 31.01.2013 22:54 # +3

               Да я не особо против. Не меньше 40 лет прошло как они очухались. Есть одна проблема: там везде знак равно забыли.
               PS Они через 10 лет проснутся и запилят кортежи в стиле js.
               

               {
                  a:"c++",becomes:"fucking",crap:"!"
                 ,doIt:[](...) { cout<<"А я метод!" }
               }

               
               Ну к тому времени всё равно никто не сможет его читать.

               Ответить

               • defecate-plusplus 31.01.2013 23:44 # +4

                 хватит ныть
                 в с++ всегда был высокий порог вхождения
                 илитарный же язык
                 
                 после школоло-паскаля он вообще казался нагромождением одних значков пунктуации

                 Ответить

                 • LispGovno 01.02.2013 01:26 # +1

                   после паскаля он казался нагромождением одних значков пунктуации школоло

                   Ответить

                 • absolut 01.02.2013 02:11 # +2

                   Помню как решил почитать Страуструпа (первое знакомство с си подобным) - вообще ничего не понял.

                   Ответить

                 • 3.14159265 01.02.2013 15:15 # 0

                   >запилят кортежи
                   Ошибочка, анонимные классы.
                   
                   Кортежи предлагаю так:
                   

                   c<'+',"+",2,complicated()>

                   
                   Зачем угловые? Чтоб никто не догадался!
                   
                   >илитарный же язык
                   Говноилитарный. Только пхп илитарнее: PHP (3188) C++ (1121). И кстати правильно что на третьем месте шарп. Но с таким серъезным соперником как кресты бороться будет трудно.

                   Ответить

                   • guest 01.02.2013 15:38 # +1

                     > Ошибочка, анонимные классы.
                     

                     struct {int f(){return 0;}}

                     Ответить

                   • guest 01.02.2013 15:39 # +2

                     >Кортежи предлагаю так:
                     

                     std::make_tuple('+',"+",2,complicated())

                     Ответить

                     • 3.14159265 01.02.2013 15:42 # 0

                       Слишком просто. Тогда и массивы следовало:
                       

                       std:make_array(1,2,3,4);

                       Ответить

                       • defecate-plusplus 01.02.2013 15:44 # 0

                         если очень надо - то можно и make_array
                         просто { .... } - это не array, это initializer_list

                         Ответить

                         • guest 01.02.2013 16:48 # +2

                           > просто { .... } - это не array
                           

                           #include <iostream>
                           using namespace std;
                           int main() {
                              auto a = {1,2,3,4};
                              cout<<a.begin()[2]<<endl;
                              return 0;
                           }

                           
                           http://liveworkspace.org/code/lEir4$7

                           Ответить

                           • defecate-plusplus 01.02.2013 16:59 # +1

                             http://liveworkspace.org/code/1H0IIF$1

                             Ответить

               • ISO 14.09.2023 09:13 # 0

                 Ну как, проснулись?

                 Ответить

         • TarasB 01.02.2013 17:05 # −1

           чё надо на куче создавать? Вот жеже говнище

           Ответить

           • roman-kashitsyn 01.02.2013 17:11 # +2

             в жабе всё на куче кроме примитивов

             Ответить

           • 3.14159265 01.02.2013 17:39 # +2

             Ты говоришь "куча" будто это что-то плохое.

             Ответить

           • bormand 01.02.2013 18:43 # 0

             Там куча выделяет память намноого быстрее сишной. Если объекты достаточно быстро подыхают, амортизированная скорость окажется чуть-чуть медленнее работы со стеком.

             Ответить

             • bormand 01.02.2013 19:06 # 0

               UPD: Подыхают = становятся никому не нужными. Копирующие гц с несколькими поколениями спасают только нужные объекты, забивая на все остальные. Поэтому скорость сборки первого поколения зависит от количества нужных объектов, которые гц приходится копировать во второе поколение, но не зависит от общего количества объектов, выделенных с момента прошлой сборки.
               
               P.S. Всю эту картину сильно портят финализаторы, поэтому их и не рекомендуют юзать без особой необходимости.

               Ответить

             • 3.14159265 01.02.2013 19:13 # 0

               Куча еще полезна при работе с замыканиями анонимными классами, чтобы захваченные параметры хранить.

               Ответить

               • bormand 01.02.2013 19:30 # 0

                 Ну в полезности кучи, я думаю, Тарас не сомневается. Просто ему не нравится то, что ее тут юзают чаще чем в сишке.
                 
                 P.S. Кстати хотспотовский JIT умеет убирать ненужные new и превращать объект тупо в пачку переменных на стеке, если ссылка на объект не утекает за пределы функции?

                 Ответить

                 • 3.14159265 01.02.2013 19:40 # +4

                   Я знаю одно: что не моча, то говно
                   CLR(.NET) и выделяет на стеке переменные, и замыкания тоже через стек реализованы, и передача в метод объектов и по значениям, и по ссылкам (ref, out), и много других фич, которые теоретически должны повысить скорость.
                   А общая скорость по бенчам ниже чем у JVM.

                   Ответить

                   • roman-kashitsyn 01.02.2013 21:54 # +4

                     Потому что спецы мелкософта оптимизируют итераторы, делая их неинтуитивными структурами, но случайно забывают включить в библиотеку Set, чтобы выполнять uniq для списка за амортизированное O(n).

                     Ответить

                     • LispGovno 01.02.2013 22:21 # +1

                       > амортизированное O(n)
                       Хештаблица?

                       Ответить

                       • bormand 01.02.2013 22:30 # +2

                         Она самая. Деревом uniq можно сделать только за O(n*log(n)), как и сортировкой.

                         Ответить

                     • LispGovno 01.02.2013 23:00 # 0

                       > забывают включить в библиотеку Set
                       http://msdn.microsoft.com/ru-ru/library/bb359438.aspx
                       Aлсо dick tionary

                       Ответить

                       • bormand 01.02.2013 23:10 # +1

                         Видимо Роман об этом: Поддерживается в версиях: 4.5, 4, 3.5.

                         Ответить

                       • 3.14159265 01.02.2013 23:13 # +2

                         >>bb359438.aspx
                         since 3.5
                         
                         > >спецы мелкософта оптимизируют итераторы, делая их неинтуитивными структурами
                         Два плюса серу, только за это. Там кстати споры были насчет "оптимизицации"
                         

                         for (v in enumer){
                         ()=>{use of v};
                         }

                         Ибо шарпообезьянки не привыкли к тому что мутабельная переменная v захватывается и при вызове лямбды там будет финальное значение.
                         
                         Хотя любой мало-мальски писавший на js по-любому наступал на это (или уже знал).
                         

                         Ответить

                         • bormand 01.02.2013 23:32 # 0

                           На js я на это налетел немного по другой причине, когда прикручивал маркеры к опенстритмаповской карте. Я знал, что лямбда будет захватывать значение переменной по ссылке, и переписал его в отдельную переменную. Код выглядел примерно так:

                           for (var i=0; i<n; i++) {
                               var id = i;
                               addSomething(..., function() {
                                   doSomething(id);
                               });
                           }

                           Но естественно в doSomething всегда попадало последнее значение. Но не потому, что я не знал о лямбде, а из-за богомерзкого скопинга js, о котором я тогда не знал, и по наивности думал, что var работает так же, как и в нормальных языках.

                           Ответить

                           • bormand 01.02.2013 23:39 # 0

                             P.S. Поэтому я теперь, если приходится писать на js, по-паскальски пишу все var'ы в первой строке функции, дабы не смущать читателя, пришедшего из языков с нормальными скопами.

                             Ответить

                           • TarasB 02.02.2013 00:02 # 0

                             А что там происходило? Мои императивные мозги не понимают подвоха.

                             Ответить

                             • bormand 02.02.2013 00:09 # 0

                               А в жс паскальский скопинг (переменные действуют на всю функцию) с богомерзким, вводящим в заблуждение псевдосишным синтаксисом, позволяющим писать вар в любом месте. Из за этого все лямбды захватили одну и ту же переменную, и клик по любому маркеру обрабатывался как клик по последнему, т.к. в конце цикла в ней окажется н-1.

                               Ответить

                               • LispGovno 02.02.2013 09:49 # −2

                                 Я ничего не понял. Можно пример на идеоне или лайворкспейсе для императивных питушков написать? (Что получилось и как по вашему должно быть? (можно даже примерчик в каком-нибудь сишарпике накатать))

                                 Ответить

                                 • bormand 02.02.2013 10:10 # +1

                                   > Я ничего не понял.
                                   Пример с багом из-за богомерзкого скопинга жс: http://ideone.com/MgVwAS А интуитивно должно быть как в си - описал переменную внутри блока - значит ее скоп - этот самый блок, а не вся функция. К лямбдам никаких претензий, претензии только к ёбнутым скопам.

                                   Ответить

                                   • bormand 02.02.2013 10:19 # 0

                                     P.S. К паскалю тоже никаких претензий, там хоть переменная и действует на всю функцию, но это интуитивно и наглядно, т.к. там var написан перед телом функции.

                                     Ответить

                                   • 3.14159265 06.02.2013 19:11 # +1

                                     >Пример с багом из-за богомерзкого скопинга жс
                                     
                                     Вот хороший пример, без функций
                                     

                                     var a='Taras';
                                     
                                         do{
                                             var a='Goomno';
                                         }while(false);
                                         alert(a);

                                     
                                     Угадайте что будет.
                                     
                                     И поэтому в кои-то веки внесли хорошее предложение расширить js словом let, что и сделали добрые дяди из мозиллы.
                                     https://developer.mozilla.org/en-US/docs/JavaScript/New_in_JavaScript/1.7?redirectslug=New_in_JavaScript_1.7#section_11

                                     Ответить

     • Lure Of Chaos 31.01.2013 00:30 # +2

       Integer[] jh = new Integer[] {Integer.valueOf(17)}; // w\o autoboxing

       Ответить

   • GK2012 25.08.2021 21:11 # 0

     Она встала с места, процокала тонкими каблучками по ступеньках и, встав возле меня, продолжила:

     Ответить

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

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

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

   Помни, guest, за тобой могут следить!

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

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

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