Говнокод #4466 — Python — Говнокод.ру

Python / Говнокод #4466

−165
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
16. 16
17. 17
18. 18
19. 19
20. 20
21. 21
22. 22
23. 23
24. 24
25. 25
26. 26
27. 27
```
def getrows(c):
    return \
        sum( # суммируем по категориям
            map(
                lambda category:
                    sum( # суммируем по itemid
                        map(
                            lambda itemidset:
                                sum( # суммируем по itemstring
                                    map(
                                        lambda itemstring:
                                            map(
                                                lambda info:
                                                    transaction(category, itemstring, info),
                                                itemidset[itemstring].values()
                                            ),
                                        itemidset.keys()
                                    ),
                                []),
                            c[category].values()
                        ),
                        []
                    ),
                ("completedAuctions", "failedAuctions", "completedBidsBuyouts")
            ),
            []
        )
```
Залез в свой старый скрипт, генерирующий отчеты, думал кое-что подправить...
Увидел одну из функций (эту) и решил лучше забить тут что-то править хД

Причины такого ужаса - наверно отсутствие явной типизации в питоне и средств рефакторинга (rename variable, extract method, ...) в "IDE"

Запостил: burdakovd, 27 Октября 2010

Tweet
Комментарии (17) RSS
- KoirN 27.10.2010 23:59 # 0
  
  А мне нравиться )
  Ответить
  - seonull 28.10.2010 05:38 # 0
    
    пирамидка такая, пирамидка.
    Ответить
- user654321 28.10.2010 06:49 # +8
  
  ехал лямбда через лямбда
  Ответить
- Altravert 01.11.2010 08:28 # +3
  
  Так нельзя писать. Это очень плохо. Это невозможно поддерживать. И я не пишу так.
  Но от этого просто десткий восторг наступает. Запишите в одну строчку.
  Это что-то из http://www.nestor.minsk.by/sr/2006/02/sr60201.html, только очень красиво.
  Ответить
  - burdakovd 01.11.2010 13:57 # +1
    
    Ну вообще говоря когдя я писал это оно было в одну строчку, точнее разрывы строки были каждые 80 символов. И даже всё понятно было.
    
    Это сейчас чтоб хоть как-то разобраться что происходит - отформатировал.
    Кстати помогло, нужное исправление сделал и рефакторить не пришлось. В конце концов это не ПО а скрипт=)
    Ответить
    - burdakovd 01.11.2010 13:57 # +2
      
      Вот ещё кусочек, столь же монструозный, но не столь симметричный:
      
      ul( map( lambda realm: "%s %s" % ( realm, ul( map( lambda (character, f): u"%s, запас %s, данные от %s" % ( link( "%s-%s-data-1.html" % (realm, character), faction(f, character), "character" ), gold(BeanCounterDB.settings[realm][character]["wealth"]), time.strftime( "%Y-%m-%d %H:%M:%S", time.localtime( max( map( lambda transaction: int(transaction.time), getrows(data[realm][character]) ) ) ) ) ), realchars[realm] ) ) ), realchars.keys() ) )
      Ответить
      
      Altravert 02.11.2010 07:08 # 0
      
      Красиво, чуви, красиво)
      Ответить
      
      gegMOPO4 30.11.2010 20:20 # +1
      
      И это можно упростить:
      
      ul("%s %s" % ( realm, ul(u"%s, запас %s, данные от %s" % ( link( "%s-%s-data-1.html" % (realm, character), faction(f, character), "character" ), gold(BeanCounterDB.settings[realm][character]["wealth"]), time.strftime( "%Y-%m-%d %H:%M:%S", time.localtime(max(int(transaction.time) for transaction in getrows(data[realm][character]))) ) ) for character, f in realchars[realm] ) ) for realm in realchars )
      
      Ох, недаром лямбду собирались выпилить.
      
      Интересно, к чему относится этот код?
      Ответить
      
      burdakovd 30.11.2010 22:09 # 0
      
      Не надо лямбду выпиливать, я больше так не буду! :D
      
      > Интересно, к чему относится этот код?
      Есть такая игра - WoW, в нем есть аукцион.
      Есть такой аддон - http://auctioneeraddon.com/ , для анализа цен на игровом рынке и ведения логов торговли.
      
      А я писал скрипт для внеигрового анализа "базы данных" этого аддона и представления html отчета по торговле своих персонажей, с графиками, топами и т.д.
      Ответить
      
      gegMOPO4 30.11.2010 23:11 # −1
      
      Надеюсь, что к четвёртому Питону выпилят. Это одна из ошибок Гвидо, вроде целочисленного деления или строк-исключений.
      
      1) Лямбда в Питоне не нужна. Есть удобнее средства. Более 90% случаев использования лямбды можно записать более ясно и понятно с помощью list comprehensions или yield (вот ещё добавят yield from (PEP 380) и будет вообще хорошо). Остальное делается с помощью локальных или глобальных именованных функций. Или готовых кирпичиков operator.
      
      2) Лямбда в Питоне убога. Допускаются только простые выражения, ничего серьёзного. И не может быть, учитывая особенности питоновского синтаксиса. Вот лямбды, предлагаемые для C++ и Java лишены этого недостатка.
      Ответить
- krushi 03.11.2010 10:46 # 0
  
  > Залез в свой старый скрипт
  
  Насколько стар этот скрипт -- точнее, версия питона, под которую он был написан? Лет пять назад альтернатив было куда меньше, чем сейчас. Если на чем-то глаз и спотыкается -- то на пустых листах [] в качестве второго аргумента map и sum.
  Ответить
  - burdakovd 03.11.2010 14:37 # 0
    
    Полтора года скрипту. Для меня пока и такой срок "старый"=)
    Python 2.5 или 2.6, не помню какой тогда у меня стоял.
    
    А там суммируются списки, и так как по умолчанию он суммирует с 0, то пришлось указать стартовое значение []. Хотя вероятно такое суммирование не эффективно (в памяти создаётся много временных списков), но пока не нашёл стандартной функции для объединения произвольного количества списков.
    Ответить
    - krushi 03.11.2010 14:55 # +1
      
      > стандартной функции для объединения произвольного
      А так?
      
      >>> [1,2,3] + [4,5,6] [1, 2, 3, 4, 5, 6]
      Ответить
      
      burdakovd 03.11.2010 16:04 # 0
      
      sum(lists, []) это и было использование оператора "+"
      
      Но это квадратичная сложность, для линейной я уже нашёл itertools.chain
      http://stackoverflow.com/questions/716477/join-list-of-lists-in-python
      Ответить
- gegMOPO4 30.11.2010 19:51 # +3
  def getrows(c): return (transaction(category, itemstring, info) for info in itemvalues for itemstring, itemvalues in itemidset.iteritems() for itemidset in c[category].itervalues() for category in ("completedAuctions", "failedAuctions", "completedBidsBuyouts") )
  
  или
  
  def getrows(c): for category in ("completedAuctions", "failedAuctions", "completedBidsBuyouts"): for itemidset in c[category].itervalues(): for itemstring, itemvalues in itemidset.iteritems(): for info in itemvalues: yield transaction(category, itemstring, info)
  Ответить
  - burdakovd 30.11.2010 21:42 # 0
    
    Восхищен.
    Код делает то же самое, что у меня, но намного красивее. И видимо эффективнее, так как у меня там форсировалось вычисление списков в sum.
    
    Как-то не пришло в голову использовать ни множественный for в list comprehensions, ни yield, хотя знал о них.
    Ответить
- TheHamstertamer 28.10.2011 19:59 # 0
  
  >Причины такого ужаса - наверно отсутствие явной типизации в питоне и средств рефакторинга (rename variable, extract method, ...) в "IDE"
  Нет, милейший, проблема в другом...
  Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только во вторник, пятницу или субботу
Где здесь C++, guest?!

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

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

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

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

Python / Говнокод #4466

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

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