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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• 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 / Говнокод #14510

   +68

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11

   public static String elvis(String value, String ifNull) {
           return value == null ? ifNull : value;
       }
   
       public static Boolean elvis(Boolean value, Boolean ifNull) {
           return value == null ? ifNull : value;
       }
   
       public static Object elvis(Object value, Object ifNull) {
           return value == null ? ifNull : value;
       }

   - Objects#firstNotNull()?
   - нет, не слышал

   Запостил: myzone, 05 Февраля 2014

   Tweet

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

   • Dummy00001 06.02.2014 01:35 # +3

     минуснул. потому что Элвиса плюсовать это грех.

     Ответить

   • someone 06.02.2014 02:04 # +3

     О дженериках автор, видимо, тоже не слышал.

     Ответить

     • bormand 06.02.2014 07:07 # +1

       public static <T> T elvis(T value, T ifNull) {
           return value != null ? value : ifNull;
       }

       Как-то так?

       Ответить

       • myzone 07.02.2014 01:02 # 0

         именно так...

         Ответить

     • kegdan 07.02.2014 09:04 # 0

       Единственное, что слышал автор - то что он крут

       Ответить

       • VseGovnoOdinYaKrut 07.02.2014 17:00 # +4

         Ответить

   • bormand 06.02.2014 07:07 # +2

     > Objects#firstNotNull()
     Внешняя зависимость же... Не всегда айс. Или там guava/commons уже юзается?

     Ответить

     • roman-kashitsyn 06.02.2014 08:33 # +3

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

       Ответить

       • VseGovnoOdinYaKrut 06.02.2014 20:53 # 0

         Вот если бы ещё обратносовместимость была на уровне, а то её грамотность привносящая популязиционный фактор, приводит к адовому jar hell от зависимостей в третьем колене.

         Ответить

         • myzone 07.02.2014 01:04 # 0

           jar hell happends не хороших дипендерсов, так что лишний повод избавиться от аудейтед хлама ;)

           Ответить

           • VseGovnoOdinYaKrut 07.02.2014 18:20 # +2

             Порой аутдейтед хлам является основополагающей либой "от партнёров".

             Ответить

             • myzone 07.02.2014 20:28 # +1

               Уходите вы оттуда...

               Ответить

               • VseGovnoOdinYaKrut 07.02.2014 22:42 # 0

                 Не надо паники. Тешит то, что этвсёхерня в сравнении со стыковкой с тем тырпрайзом, который крутится на старопне под мсдосом. А переписывать никто софт не собирается ибо оно "и так хорошо работает".

                 Ответить

         • roman-kashitsyn 07.02.2014 10:25 # 0

           Можно поподробнее? Где не соблюдается обратная совместимость?

           Ответить

           • VseGovnoOdinYaKrut 07.02.2014 16:59 # +1

             Да пожалуйста, http://docs.guava-libraries.googlecode.com/git-history/release/jdiff/changes.html. Туева хуча @Deprecated режется в следующих релизах, чем доставляет попаболь от необходимости использования разных класслоадеров для библиотек, зависимых от разных версий гуавы или же подключать м'OSGi.

             Ответить

     • myzone 07.02.2014 01:06 # +1

       Гуавы там не было, а зря...
       Там люди сделали NativeStringCache, например, кто догадается как он работает?

       Ответить

       • VseGovnoOdinYaKrut 07.02.2014 05:11 # +3

         String.intern()?

         Ответить

         • myzone 07.02.2014 20:28 # +1

           Правильно!

           Ответить

           • bormand 07.02.2014 20:53 # +2

             Зато потом через == можно сравнивать :) Темная сторона силы.

             Ответить

             • 3.14159265 07.02.2014 22:33 # 0

               Только с 1.7. Там уже не в пермгене хранится.

               Ответить

               • bormand 07.02.2014 22:38 # 0

                 А в 1.6 и ниже через == разве нельзя сравнивать?
                 
                 Или ты о сборке мусора из пула интернированных строк?

                 Ответить

                 • 3.14159265 07.02.2014 22:45 # 0

                   >о сборке мусора из пула интернированных строк
                   OutOfMemoryError: PermGen space error
                   Это сильно заябывает. А куча больше.

                   Ответить

               • myzone 08.02.2014 03:20 # 0

                 кстати, там в этом коде рефлексией брался филд "offset" из стринга и мутились "указатели" на эти строки, или что-то типа того, я не сильно заморачился логикой, ибо наш проект переехал на последний релиз JDK, а там стринг перепилили :D

                 Ответить

                 • VseGovnoOdinYaKrut 08.02.2014 03:55 # +4

                   Дай дураку рефлектор, как говорится.

                   Ответить

                   • 3.14159265 08.02.2014 03:57 # +5

                     ... он и немутабельное изменит, и рефлектор поломает.

                     Ответить

                   • myzone 08.02.2014 03:59 # 0

                     Эта 5+, спасибо, поржал :)

                     Ответить

             • VseGovnoOdinYaKrut 07.02.2014 22:45 # +1

               Да, если только перед сравнением строки уже не надо интернировать, а то equals поди дешевле будет интернирования.

               Ответить

               • 3.14159265 07.02.2014 22:48 # +1

                 Нихера подобного.
                 Интернируем - раз при создании, как и расчёт хеш-кода. А equals нужно много раз на немутабельном объекте (те же мапы, они везде).
                 
                 Если конечно строки не одноразовые, типа такого гогна:
                 while(it.hasNext()) s+=it.next();
                 
                 Короче от контекста зависит.

                 Ответить

                 • Stertor 07.02.2014 22:59 # −3

                   Кому что...

                   Ответить

                   • Stertor 07.02.2014 23:20 # −3

                     Спасибо за минус. Я стану стражем твоей тени. (LEXX, надеюсь, смотрели)

                     Ответить

                 • VseGovnoOdinYaKrut 08.02.2014 00:20 # 0

                   Я не понял, что тут нихера. Вот в пример, о чём я выше писал, что менее накладнее, internedString == userString.intern() или internedString.equals(userString)? Контекст тут такой, юзер ввёл своё имя и мы проверяем, не "олень" ли юзер.

                   Ответить

                   • 3.14159265 08.02.2014 01:47 # 0

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

                     Ответить

                     • VseGovnoOdinYaKrut 08.02.2014 02:20 # 0

                       Получается, что например, если путешествие занимает 10 мап в роли ключа, то мы экономим >= 10 полных String.equals, в зависимости от коллизий. Оптимизация, однако!

                       Ответить

       • bormand 07.02.2014 05:29 # +1

         Ну неужели через JNI, раз native?

         Ответить

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

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

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

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

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

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

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