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

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

   −406

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7

   Код конца 2014 года
   
       CGSize iOSDeviceScreenSize = [[UIScreen mainScreen] bounds].size;
       if (iOSDeviceScreenSize.height == 480)
           isRetina = true;
       else
           isRetina = false;5

   Запостил: Headless, 23 Марта 2015

   Tweet

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

   • guest 24.03.2015 08:08 # +1

     Это позор и смерть для народа того, кто писал это

     Ответить

   • TarasB 24.03.2015 11:46 # 0

     А что значит цифра после точки с запятой?

     Ответить

     • Headless 24.03.2015 12:02 # 0

       Видимо случайно добавил, цифры там конечно нет. Жалко нельзя редактировать.

       Ответить

       • TarasB 24.03.2015 12:15 # +5

         Жаль, я думал, фишки новые. Типа выровнять команду по адресу, кратному 5.

         Ответить

         • kegdan 24.03.2015 12:45 # +1

           присвоить 5 раз, что уж наверняка

           Ответить

           • chtulhu 24.03.2015 19:29 # −1

             я ловил забавные баги в clang/llvm: компилятор просто пропускал некоторые выражения. Причем ловил и в крестах и на objc.
             Так что хотелось бы иметь такую инструкцию :)

             Ответить

     • guest 24.03.2015 19:32 # +3

       Встроенный юнит-тест, проверяющий длину false

       Ответить

   • chtulhu 24.03.2015 19:26 # 0

     Я вот замечаю, что многие до сих пор не могут(или не хотят) в autolayout, хотя ему уже точно пару лет есть

     Ответить

   • Gerchicov-bp 25.03.2015 10:19 # +1

     когда-то айфонов было 2 - ретина и неретина. Так что вполне себе нормальный код из старого проекта, который поддерживал только ифоны и не поддерживал смену ориентации

     Ответить

     • kegdan 25.03.2015 10:55 # 0

       >>Так что вполне себе нормальный код
       
       Для говнокодера - да. Нормальные люди пишут код, который годен больше месяца

       Ответить

       • Gerchicov-bp 25.03.2015 11:47 # 0

         чел, я не спорю, что это плохохо для 2014 года, но код работает на ВСЕХ ios кроме 8, причем без дополнительных проверок. Напиши вариант лучше. Точнее работал бы, если б автор поста не допустил ошибку.
         Расскажешь почему именно ios 8?

         Ответить

         • Headless 25.03.2015 11:51 # +4

           Там написана дата написания кода) Конец 2014 - тогда уже были шестые айфоны.
           
           Кроме того, проверка изначально не верна, height = 480 был равен и на айфоне 3gs, и 3g, которые не имели ретина экранов.
           
           Также на UIScreen есть проперти scale, в котором будет 2.0 или 3.0, если девайс ретина.
           
           Ну а помимо всех перечисленных пунктов могу добавить, что этот код я встретил в десятке контроллеров этого легаси проекта.

           Ответить

           • Gerchicov-bp 25.03.2015 12:00 # 0

             так вот в этом и есть ошибка, причем скорее всего твоя, потому что height = 480 - это неретина, причем работало бы во всех ios и даже на ios 8 при определенных условиях
             
             Scale? Ок, открываем документацию и читаем когда он появился. А после этого читаем когда появился bounds.
             
             И дата 2014 кстати ни о чем не говорит. Ты лучше напиши, какую минимальную версию ios должно поддерживать приложение и iphone или ipad тоже?

             Ответить

             • Headless 25.03.2015 12:02 # 0

               Да что ты говоришь))) height 480 и на айфоне 3gs и на айфоне 4/4c.
               
               Scale - iOS 4.0. Deployment target проекта - iOS 7. iPhone only.

               Ответить

               • Gerchicov-bp 25.03.2015 12:03 # 0

                 ну так вот с этого и надо было начинать - про ios 7

                 Ответить

                 • Headless 25.03.2015 12:04 # 0

                   А тут нечего проверять - мы всегда работаем с поинтами а не с пикселями. О чем неоднократно написано в документации.

                   Ответить

                   • Gerchicov-bp 25.03.2015 12:09 # 0

                     да, наверное ты прав. А scale у тебя в ios 8 нормально работает? Не доходили руки проверить

                     Ответить

                     • Headless 25.03.2015 12:11 # 0

                       Честно говоря никогда не юзал - не вижу смысла, когда есть автолейаут. Кроме того, с iOS 7 iPhone не ретина не осталось, так что 1x мы фактически дропнули, и даже не добавляем в ассеты.
                       
                       Если приложение запускается на айпаде мини(единственный не ретина девайс, оставшийся в природе), то 2x ассеты просто даунскейлятся.

                       Ответить

                       • Gerchicov-bp 25.03.2015 12:13 # 0

                         lol, а ipad 2? А accessibility?

                         Ответить

                         • Headless 25.03.2015 12:14 # 0

                           iPad 2 идентичен айпаду мини - разрешение одинаковое. А что с accessibility?

                           Ответить

                           • Gerchicov-bp 25.03.2015 12:15 # 0

                             ну например это:
                             http://www.paintcodeapp.com/news/ultimate-guide-to-iphone-resolutions

                             Ответить

                             • Headless 25.03.2015 12:16 # 0

                               И что?

                               Ответить

                               • Gerchicov-bp 25.03.2015 12:17 # 0

                                 вместо 5 экранов (или сколько их там) получили на выходе 15 - больше чем устройств? Да не, ничего не произошло

                                 Ответить

                                 • Headless 25.03.2015 12:26 # 0

                                   Я не понимаю, о чем ты говоришь. С автолейаутом мы можем поддерживать все разрешения, и не интересоваться вопросом ретина они или нет. Вопрос намного важнее - это ширина и высота экрана. Сейчас фактически нужно поддерживать 4 размера экрана айфонов, и один размер айпада.
                                   
                                   Айфоны - 320 x 480, 320 x 568, 375x667, 414x736
                                   Айпад - 1024 x 768.
                                   
                                   Все в пойнтах.

                                   Ответить

                             • bormand 25.03.2015 12:31 # 0

                               По сравнению с зоопарком андроидов - фигня. У вас на iOS их хотя бы конечное число (всего 15?)...

                               Ответить

                               • Gerchicov-bp 25.03.2015 12:41 # +3

                                 на андроид просто к этому вопросу умнее подошли (хотя бы для приложений со стандартным UI) - на манер разметки html.
                                 А тут - сначала одно разрешение будет всегда, потом вот вам еще одно недоразрешение (ретина, которая улучшает четкость, но не изменяет физически размеры). Вам это слишком просто? А нате вам еще "ретина плюс один сантиметр". Что и этого мало? Нате вам еще ипадов и ифонов.
                                 И из-за этой неопределенности имеем over9000 способов выделить нужное разрешение (и поведение в зависимости от него) - начиная с bounds и определения разрешения по модели и заканчивая autolayout и size classes. И все эти способы ущербны. Даже если рассматривать только последние технологии - storyboard и так уже перегружен элементами, а с size classes вообще превращается в ад

                                 Ответить

                                 • chtulhu 25.03.2015 15:18 # +2

                                   В эппл просто не умеют программировать. У них все продукты - гламурное говно.

                                   Ответить

   • Gerchicov-bp 25.03.2015 11:51 # +3

     сколько обсирают этот код - а до сих пор не нашли ошибку в посте
     
     плюсану тому, кто найдет

     Ответить

     • guest 25.03.2015 12:41 # +1

       ты про false;5 ?

       Ответить

       • Gerchicov-bp 25.03.2015 12:42 # 0

         уже ответили выше

         Ответить

     • volter9 25.03.2015 22:09 # 0

       CGSize iOSDeviceScreenSize = [[UIScreen mainScreen] bounds].size;
       isRetina = iOSDeviceScreenSize.height == 480;

       
       или вообще:
       

       isRetina = [[UIScreen mainScreen] bounds].size.height == 480;

       Ответить

       • guest 25.03.2015 23:46 # 0

         Это ошибка?

         Ответить

       • Gerchicov-bp 26.03.2015 09:30 # 0

         Ок, скажу. Ошибка в том, что даже если бы size выдавал информацию о количестве пикселей, а не точек, то размер 480 - у НЕретина экранов. А вы пишете наоборот
         А поскольку выдает информацию в точках, то все равно придется проверять что-то еще, потому что повторяя высказавшегося выше - у 4/4s тоже размер 480

         Ответить

         • guest 26.03.2015 09:47 # 0

           >о количестве пикселей, а не точек
           А это не одно и то же?

           Ответить

           • bormand 26.03.2015 09:54 # 0

             Нет. Пиксели (px) - физические пиксели экрана. Поинты (pt) - 1/72 дюйма.

             Ответить

             • guest 26.03.2015 10:23 # +2

               Тогда уже пункт https://ru.wikipedia.org/wiki/Типографский_пункт

               Ответить

               • bormand 26.03.2015 10:32 # 0

                 Да, он самый.

                 Ответить

       • bormand 26.03.2015 09:39 # +1

         > [[UIScreen mainScreen] bounds].size.height == 480;
         А если айфун набок повёрнут?

         Ответить

         • inkanus-gray 26.03.2015 12:59 # +1

           То неретина по-любому. Очевидно же.

           Ответить

           • kegdan 26.03.2015 17:43 # −2

             кретина

             Ответить

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

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

   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] (подробнее)

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