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

    −106

    1. 1
    "<br />".join(["%s" % template for template in rendered_templates ])

    моё... у меня вечная мания всё усложнить :)

    Запостил: Nemoden, 21 Апреля 2012

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

    • ava loststy 21.04.2012 23:55 # −1

      Не вижу говнокода. Совершенно нормальная конструкция для какого-то узконаправленного скрипта.
      Ответить

      • ava Nemoden 22.04.2012 11:59 # 0

        а то, что это - эквивалент
        "<br />".join(rendered_templates)
        ?
        Ответить

        • ava WGH 22.04.2012 18:17 # +1

          Не совсем.
          >>> "".join([1, 2, 3])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: sequence item 0: expected string, int found
>>> "".join("%s" % s for s in [1, 2, 3]) # а пример из поста
'123'
>>> "".join(map(str, [1, 2, 3])) # можно ещё так
'123'

          В общем, единственная проблема кода - неоправданное использование оператора форматирования. Надо было str вместо него. P.S. И предпочтительней generator expression вместо списка. Меньше памяти займет.
          Ответить

          • ava Nemoden 23.04.2012 05:21 # 0

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

        • ava loststy 22.04.2012 23:00 # 0

          Ну да, конечно, а каст где?
          Ответить

          • ava Nemoden 23.04.2012 05:19 # 0

            нужно было сказать, что у меня rendered_templates - список строк, хотя само имя переменной как бы намекает на это...

            согласен - не совсем ГК
            Ответить

    • ava guest 30.06.2012 17:12 # 0

      "<br />".join([ str(template) for template in rendered_templates ])
      Ответить

      • ava guest 14.08.2012 21:15 # 0 

        "<br />".join(map(str, rendered_templates ))

        Список абсолютно лишний.
        Ответить

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