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

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

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

1. PHP (5698)
2. C++ (2733)
3. Куча (2323)
4. JavaScript (2027)
5. C# (1924)
6. Java (1501)
7. Си (1095)
8. Pascal (647)
9. Python (582)
10. 1C (531)
11. SQL (431)
12. ActionScript (292)
13. Objective C (202)
14. bash (196)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (28)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

1. Python / Говнокод #27290

   0

   1. 1
   2. 2

   def IsZIPFile(filename):
       return filename.lower().endswith('.zip')

   ибааааать!

   Запостил: Petro-san, 10 Марта 2021

   Tweet

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

   • 6oHo6o 11.03.2021 01:37 # 0

     поразительно хуёвая функция, и одновременно с этим соврешенно ненужная. Единство формы и содержания!

     Ответить

     • JloJle4Ka 11.03.2021 02:21 # 0

       Что в ней хуёвого? И что в ней ненужного? Как бы ты это сделал? Будет ли решение на рагулярках хорошим?

       Ответить

       • 6oHo6o 11.03.2021 02:22 # 0

         хуевый в ней кодстайл
         
         я бы просто написал
         filename.lower().endswith('.zip')
         
         алсо
         import pathlib
         pathlib.Path(filename).suffix == zip

         Ответить

         • JloJle4Ka 11.03.2021 02:27 # 0

           Какое-то магическое выражение. К тому же, с высокой долей вероятности IsZIPFile превратится в IsArchive в будущем, так что этот код очен хорош своей расширяемостью, читаемостью и возможностями рефакторинга.

           Ответить

           • 6oHo6o 11.03.2021 02:28 # 0

             >IsArchive
             https://www.python.org/dev/peps/pep-0008/#function-and-variable-names

             Ответить

             • JloJle4Ka 11.03.2021 02:31 # 0

               Да, мне тоже не понравилось.
               
               Ну вдруг это код каких-нибудь сишников, и у них в компании свой кодстайл.
               
               Лично я вот так пишу: variable_name, functionName, ClassName, и мне нормально .

               Ответить

               • 6oHo6o 11.03.2021 02:36 # 0

                 >functionName
                 фу, нахуя нарушать конвенции?
                 
                 Всё таки питух реднкая вербозня, нахуя вот тут две строчки?

                 Ответить

                 • JloJle4Ka 11.03.2021 02:43 # 0

                   > фу, нахуя нарушать конвенции?
                   
                   https://youtu.be/HGutEFXo9os
                   
                   > Всё таки питух реднкая вербозня, нахуя вот тут две строчки?
                   
                   Напиши на одной, мне не жалко. Потом придётся переделывать это, заебёшься искать по коду такие конструкции.
                   
                   В данном конкретном случае в любом ЯП можно это обернуть в функцию, чтобы выглядело пижже. И не такой уж это и вербознух.

                   Ответить

                   • 6oHo6o 11.03.2021 03:02 # +2

                     мочи манту, шатай систему
                     
                     В нормальном языке даже эту функцию можно написать в одну строку.
                     И не надо filename делать строкой, кстати. Будешь потом слешики неправильные выкавыривать.
                     
                     

                     //Одна строка, внятная семантика, никакого lower
                     // и никаких дурацких скобочек при использовании
                     val File.isZip get() = extension.equals("zip", ignoreCase = true)
                     
                     
                     fun main() {
                         println(File("/home/rms/gpl.txt").isZip) //false
                         println(File("/home/rms/cp.zip").isZip) //true
                     }

                     Ответить

                     • bormand 12.03.2021 14:28 # 0

                       Что ты будешь делать когда обезьяны закончатся?

                       Ответить

                       • 6oHo6o 14.03.2021 00:18 # 0

                         У меня циклический буфер. Начну с начала

                         Ответить

               • bormand 12.03.2021 14:18 # +1

                 > Лично я вот так пишу
                 
                 Да там половина стандартной либы с нарушением PEP8 написана...

                 Ответить

           • Fike 11.03.2021 02:33 # +5

             > превратится в IsArchive в будущем
             
             в IsBukva

             Ответить

           • Petro-san 11.03.2021 03:08 # 0

             .3un

             Ответить

       • j123123 11.03.2021 06:06 # +1

         https://govnokod.ru/23672

         Ответить

         • Petro-san 13.03.2021 15:40 # +1

           это же низкий уровень
           
           Джордейна все помнят?

           Ответить

           • booratihno 13.03.2021 15:43 # 0

             ебать ты олд :))
             
             мне казалось, что только я ту книжку сюда приносил

             Ответить

   • Steve_Brown 12.03.2021 12:21 # +1

     Говнокод в определении типа файла по расширению, а не по содержимому.
     

     def IsZIPFile(filename):
         return open(filename, "rb").readall().startsswith(b'PK♥')

     Ответить

     • bormand 12.03.2021 12:30 # 0

       > PK♥

       Ответить

       • Desktop 12.03.2021 13:18 # +2

         PKН♥

         Ответить

     • bormand 12.03.2021 12:38 # 0

       > по содержимому
       
       А вдруг это docx или сейв от какой-нибудь игрушки?

       Ответить

       • Soul_re@ver 12.03.2021 21:43 # 0

         А нам не пофиг? Если нужно, чтобы следующий юзер скачал и у него всё открылось как надо, придётся
         и расширение проверять. А если нужно чисто для нас, то пофиг на расширение, главное чтобы открывалось, дальше разберёмся.
         
         Паттерн "переименованый zip" — лидер на рынке хранения ресурсов.

         Ответить

         • guest6 12.03.2021 21:48 # +1

           Kitty.scr.exe
           Telki.mp4.exe
           Windows10.iso.exe
           и даже
           Windows10.torrent.exe

           Ответить

           • Soul_re@ver 12.03.2021 21:58 # 0

             На сервере пофиг, там не двойным кликом открывают. Проверяем магические байты, если хотим, дальше скармливаем unzip.

             Ответить

             • MAKAKA 12.03.2021 22:00 # +1

               а вдруг мы вообще не храним это на сервере в виде файла?
               вдруг мы ебанутые, и храним это в базе, как BLOB, а расширение (вместе с контент диспозишином) приделываем сами, когда отдаем данные?

               Ответить

           • Petro-san 13.03.2021 15:55 # 0

             > .scr.exe
             помножил на 0

             Ответить

             • booratihno 13.03.2021 15:55 # 0

               чойто?

               Ответить

               • Petro-san 13.03.2021 16:00 # 0

                 scr этож и есть exe с аргументами --help, --version и --fullscreen

                 Ответить

                 • JloJle4Ka 13.03.2021 16:08 # 0

                   Поскольку трояны и вирусы, установленные с использованием файлов SCR, трудно обнаружить и удалить, не рекомендуется открывать неизвестные файлы SCR, найденные на жестком диске.
                   
                   Как часто вы находите неизвестные файлы на жёстком диске? Кто их вам подкладывает?

                   Ответить

                   • Desktop 13.03.2021 16:36 # +2

                     поработай с емаксом
                     
                     офигеешь от всех этих ~1~

                     Ответить

                   • booratihno 13.03.2021 16:39 # +2

                     а ты типа знаешь все файлы у себя в system32?

                     Ответить

                   • Petro-san 14.03.2021 07:18 # +1

                     все правильно же написано, беспроигрышный вероянт для мольваре - винда рано или позно сама найдет цє:/шиндоушс/*.src и запустит с клютчем --preview

                     Ответить

                 • booratihno 13.03.2021 16:14 # 0

                   не знал) доберусь до виндокомпа, посмотрю заголовки

                   Ответить

                 • 6oHo6o 14.03.2021 00:16 # +1

                   Ты совершенно прав.
                   
                   Проверил .scr файл. Заголовок MZ. Заглянул через Impex плагин в фаре, вижу обычное 64битное приложение. В секции IMPORTS вижу .dllки. В ресурсах вижу картинки.

                   Ответить

                   • bormand 14.03.2021 00:18 # 0

                     Ну было бы странно новый формат изобретать для скринсейверов.

                     Ответить

                     • 6oHo6o 14.03.2021 00:24 # 0

                       Интересно, а как это работает у иксов?
                       Там вроде был экстешнен для скринсейверов, нужно почитать

                       Ответить

                       • bormand 14.03.2021 00:25 # 0

                         Да тоже поди обычная прога.

                         Ответить

                         • 6oHo6o 14.03.2021 00:26 # 0

                           какой секурити ))
                           
                           зы: да, программа, общающаяся с сервером через тот самый экстиешншен
                           http://manpages.ubuntu.com/manpages/bionic/man3/Xss.3.html

                           Ответить

     • bootcamp_dropout 12.03.2021 12:41 # +1

       Против трапов тоже хорошо работает

       Ответить

       • bormand 12.03.2021 16:13 # 0

         Проблема в том, что ты его уже открыл и прочитал полностью.

         Ответить

         • Desktop 12.03.2021 16:15 # +1

           что упало у студента, то упало на газету

           Ответить

     • Desktop 12.03.2021 13:20 # +2

       Лучше всего попытаться прочитать заголовки соответствующей либой, ну или доверять пользователю и обламываться на открытии, если что
       
       Руками байтики читать, ещё из Пистона, это сорт пиздеца

       Ответить

       • Soul_re@ver 12.03.2021 21:49 # +1

         libmagic, кстати, в питон завезли. Так что читать байтики ручками это вдвойне пиздец.

         Ответить

         • Petro-san 14.03.2021 07:43 # 0

           любопытно, как этот магик проверяет сигнатурки, там же хитро и не все соблюдают спеку

           Ответить

           • Petro-san 14.03.2021 09:31 # 0

             лолушки, file и даже trid объебались на моих файлах
             даже несмотря на то что я "зип" вытащил из внешнего котейнера

             Ответить

     • booratihno 12.03.2021 14:57 # +2

       >readall
       а там 4 гб

       Ответить

       • bormand 12.03.2021 15:00 # 0

         Именно поэтому я за хаски с его ленивыми списками. Питон тоже мог бы вернуть ленивый итератор, в принципе.

         Ответить

         • booratihno 12.03.2021 15:02 # 0

           а неужели нельзя в пиздоне делать read() пока EOF не придет? неужто нужно сразу всё в память?
           
           Вообще если тебя вольнуют первые 4 байта, то помоему лучше именно их и считать, а не readall

           Ответить

           • bormand 12.03.2021 15:04 # 0

             Можно read(3), но некрасиво же, магическое число.

             Ответить

             • booratihno 12.03.2021 15:09 # 0

               prefix = b'pkz'
               read(len(prefix)) == prefix

               Ответить

               • bormand 12.03.2021 15:10 # 0

                 Уже 2 строчки получится.

                 Ответить

                 • bootcamp_dropout 12.03.2021 15:19 # 0

                   В питон завезли := он вроде может позволить одной строкой присвоить и использовать

                   Ответить

               • Desktop 12.03.2021 15:27 # +2

                 YAGNI rate is low

                 Ответить

                 • bormand 12.03.2021 15:48 # +1

                   По YAGNI эта функция вообще нинужна. Рано или поздно ты пойдёшь читать что-то из этого zip'а, там либа всё и проверит. Если юзер какую-то хуйню залил -- ну ССЗБ, заново закинет правильный файл.
                   
                   Так же, как и с проверкой мыла: собачка есть и ладно.

                   Ответить

                   • Desktop 12.03.2021 16:07 # +2

                     Особенно круто, если эта функция на клиенте, а открывать мы будем на серваке после аплоада.
                     
                     Но доктор сказал в морг, значит, в морг

                     Ответить

                     • bormand 12.03.2021 16:09 # 0

                       Угу, на клиенте вообще достаточно фильтра по zip в окошке выбора файлов.
                       
                       Именно поэтому я за "YAGNI".

                       Ответить

                 • MAKAKA 12.03.2021 16:17 # 0

                   да, YAGNI запрещает читать не целиком, я забыл

                   Ответить

                   • Desktop 12.03.2021 16:20 # 0

                     а голову ты дома не забыл?
                     
                     ты переменную зачем завёл, изверг? она тебе не нужна.
                     
                     напиши два раза строковый литерал.
                     
                     правда... тогда ты нарушишь DRY! сложно быть читателем википедии.
                     
                     но виноваты конечно авторы питоньячьей либы, сложно было что ли зафигачить метод readAndCheck(str)??

                     Ответить

                     • JloJle4Ka 12.03.2021 16:23 # 0

                       Кстати, даже на Википедии написано, что Dry может соснуть в веб-программировании, потому что иногда имеет смысл вылидировать форму на клиенте, а потом на сервере, чтобы сервер лишний раз не нагружать хуитой.
                       
                       Ты в этом случае за «веб-програмиирование» или «DRY»?

                       Ответить

                       • Desktop 12.03.2021 16:25 # +1

                         ой ну это сложный вопрос
                         
                         тут нужна экспертиза Соломона Самсоновича и Африкана Свиридовича

                         Ответить

                       • MAKAKA 12.03.2021 16:32 # +2

                         DRY соснет только если ты два раза напишешь одну и ту же логику валидации.
                         
                         Если же твой фреймворк позволяет написать её один раз, и запустить на беке и на фронте, то DRY не нарушен.

                         Ответить

                         • JloJle4Ka 12.03.2021 16:44 # 0

                           Т.е. если у меня фреймворк не позволяет один раз написать логику валидации формы, а потом её вставить на сервер и перевести на JS на клиенте, то фреймворк заведомо куриный?

                           Ответить

                           • MAKAKA 12.03.2021 16:49 # +2

                             Ну я считаю, что хороший фреймворк должен такое позволять.
                             Хотя бы на уровне HTMLя расставлять type и pattern.
                             
                             Валидировать на сервере нужно всегда, это вопрос безопасности
                             Валидировать на клиенте очень желательно: это вопрос удобства
                             
                             Можно отсылвать всё на сервер, и валидировать там.
                             Но если фреймворк может логику валидации запустить и на сервере, и на клиенте, причем из одного и того же сурса, то это вообще классно.
                             
                             Писать одну и ту же логику два раза на двух языках это же уныло

                             Ответить

                             • JloJle4Ka 12.03.2021 16:53 # 0

                               Да, это так. В питоне, кстати, есть вот такая красота:
                               
                               https://pypi.org/project/WTForms-ParsleyJS/
                               
                               Но это для WTForms из фласка. Не знаю, есть ли такая скилльная штука в джанге...

                               Ответить

                           • Desktop 12.03.2021 16:56 # +1

                             Какой «Идеальный код» МАКАКоннелла )))

                             Ответить

                     • MAKAKA 12.03.2021 16:31 # 0

                       Когда вы говорите, Иван Васильевич, такое ощущение, что вы бредите.

                       Ответить

                       • Desktop 12.03.2021 16:40 # 0

                         (с) википедия

                         Ответить

                     • bormand 12.03.2021 16:33 # +1

                       > ты переменную зачем завёл
                       > тогда ты нарушишь DRY
                       
                       read(3) == b"PK♥"
                       
                       И YAGNI живы и DRY цел.

                       Ответить

                       • MAKAKA 12.03.2021 16:34 # 0

                         DRY не цел, бо 3 это повторение PK♥

                         Ответить

                         • bormand 12.03.2021 16:38 # +2

                           Где ты повторение увидел? Три это три, PK♥ это PK♥. Это разные сущности, у них даже тип разный.

                           Ответить

                           • MAKAKA 12.03.2021 16:41 # +1

                             3 зависит от PK♥
                             Изменение одного должно приводить к изменению второго.
                             Если одно поменять, а про другое забыть -- будет неконсистентно.
                             
                             Это главная проблма не DRY кода: ты в одном месте поменял, а в другом забыл.
                             
                             В то время как одно можно вывести из другого просто на этапе компиляции.

                             Ответить

                             • bormand 12.03.2021 16:42 # 0

                               Ну ок.
                               
                               read(1000).startswith("PK♥")

                               Ответить

                               • MAKAKA 12.03.2021 16:43 # 0

                                 Плохо поддерживает файлы размером менее 1000 байт вроде, а еще magic number

                                 Ответить

                                 • bormand 12.03.2021 16:54 # 0

                                   Да нет, для коротких файлов просто вернёт кусок, как в сишке.
                                   
                                   Ок, убрал магические числа: read(math.exp(math.pi)).startswith("PK♥" )

                                   Ответить

                                   • MAKAKA 12.03.2021 17:00 # +1

                                     здесь понадобится документация, так как не очень понятно, причем тут pi

                                     Ответить

                                     • JloJle4Ka 12.03.2021 17:02 # +1

                                       """
                                       «pi» говорит цыплёнок.
                                       """

                                       Ответить

                                       • MAKAKA 12.03.2021 17:05 # +3

                                         /** Performs computation and returns the result, or throws some exception. */

                                         Ответить

                                         • bormand 12.03.2021 17:07 # +3

                                           Это документация к методу Do()?

                                           Ответить

                                           • MAKAKA 12.03.2021 17:11 # +2

                                             ну да, там еще такие методы есть
                                             

                                             /**
                                             * sets id
                                             */
                                             void setId(int id);
                                             
                                             /**
                                             * sets name
                                             */
                                             void setName(String name);

                                             Ответить

                                             • MAKAKA 12.03.2021 17:22 # +2

                                               Кстати, о борьбе с копипастой
                                               

                                               NUM_OF_NA_SYLLABLES = 2
                                               NA_SYLLABLE = "NA"
                                               BA_SYLLABLE = "BA"
                                               
                                               banana = BA_SYLLABLE + NA_SYLLABLE * NUM_OF_NA_SYLLABLES
                                               print(banana)

                                               Ответить

                                           • Desktop 13.03.2021 16:37 # 0

                                             https://developer.apple.com/documentation/foundation/nsoperation/1416837-start

                                             Ответить

                                         • JloJle4Ka 12.03.2021 17:08 # 0

                                           Забыл добавить «if compiled with O1 level at most».

                                           Ответить

         • j123123 12.03.2021 16:10 # 0

           а я за mmap()

           Ответить

           • bormand 12.03.2021 16:10 # +1

             А там 4 гигабайта и 32-битная ось.

             Ответить

             • j123123 12.03.2021 16:14 # 0

               маппить разные куски файла можно
               https://man7.org/linux/man-pages/man2/mmap.2.html
               

               The contents of a file mapping (as opposed to an anonymous
               mapping; see MAP_ANONYMOUS below), are initialized using length
               bytes starting at offset offset in the file (or other object)
               referred to by the file descriptor fd.  offset must be a multiple
               of the page size as returned by sysconf(_SC_PAGE_SIZE).

               Ответить

               • bormand 12.03.2021 16:18 # 0

                 Ну это уже не так прозрачно. Придётся как-то переключать маппинги на границе.

                 Ответить

             • j123123 12.03.2021 16:18 # +1

               Кстати на x86-64 там далеко не всё адресное пространство работает:
               https://en.wikipedia.org/wiki/X86-64#Virtual_address_space_details

               Ответить

               • bormand 12.03.2021 16:23 # +1

                 Да, но это же просто деталь реализации нынешних процов. В будущем разработают, добавив новый уровень в пейджтейбл.

                 Ответить

         • CHayT 12.03.2021 16:34 # +2

           Ленивое IO в Haskell впало в немилость, кстати, ибо освобождение дескрипторов с ним тоже ленивое ^____~

           Ответить

           • bormand 12.03.2021 16:37 # +1

             Да и ошибки парсинга конфига получать через час работы сервака как-то не весело.

             Ответить

             • Desktop 12.03.2021 16:41 # +1

               у вас последний день, зайдите в бухгалтерию за логами

               Ответить

       • guest6 13.03.2021 17:25 # 0

         Какой багор

         Ответить

       • guest6 13.03.2021 17:26 # 0

         Вроде можно не весь файл читать.

         Ответить

     • bormand 12.03.2021 16:24 # +1

       У вас файл утёк, кстати.
       
       Какой высокоуровневый язык с автоматическим управлением памятью )))

       Ответить

       • Desktop 12.03.2021 16:26 # +1

         автоматический от автоматическое оружие
         
         выстрелил и забыл

         Ответить

     • Rooster 12.03.2021 18:04 # 0

       А ежели тама SFX?

       Ответить

       • MAKAKA 12.03.2021 18:06 # 0

         А SFX это .exe внутри? Там должно начинаться с MZ? Миша Збыковски, или как там его

         Ответить

         • Rooster 12.03.2021 18:09 # +1

           Сам архив же не обязательно в начале фала.

           Ответить

           • bormand 12.03.2021 18:16 # +1

             Вообще zip надо тестить с конца файла, а не с начала. Главный хедер у него в конце.

             Ответить

             • MAKAKA 12.03.2021 18:19 # 0

               как работает "file" ?

               Ответить

               • Rooster 12.03.2021 18:22 # +1

                 file tsyplyonka broilera

                 Ответить

                 • MAKAKA 12.03.2021 18:25 # 0

                   короче, если в "басх" набрать например "филе а.оут" , то выводится типа "ЕЛФ"
                   
                   Генерал Файлюра читал мой диск, и нашел там еггог

                   Ответить

               • bormand 12.03.2021 18:28 # +2

                 Как анскилльная лалка, по началу (по хедеру первого файла). Я нулей туда хуйнул, перестал определять. А unzip'у норм.

                 Ответить

                 • Soul_re@ver 12.03.2021 21:50 # +2

                   jpeg туда лучше хуйни. Не потеряется функциональность ни изображения, ни архива.

                   Ответить

                 • Desktop 12.03.2021 23:19 # +1

                   > Я нулей туда хуйнул
                   
                   - стильное радио

                   Ответить

     • Petro-san 13.03.2021 15:52 # 0

       > startsswith(b'PK♥')
       этож не меньшее говно
       3un, ебнип, надо сканить с конца файла ради сигнатурки

       Ответить

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

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

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

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