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

    −99.7

    1. 1
    length = int((re.findall(r"top:[1-9]\d*",temp))[-1][4:])*2.54/72-0.6

    Запостил: wackum, 07 Декабря 2009

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

    • ava guest 07.12.2009 21:04 # −1

      Нормально. Лишние скобки убрать, использовать квантификатор + вместо *, взятие по индексу заменить на группировку в рег. выражении (по вкусу), и промежуточную переменную ввести. Тогда совсем всё классно будет :-)
      Ответить

      • ava guest 09.12.2009 19:59 # +1

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

        • ava Billy the Kidd 09.12.2009 20:38 # +1

          Инчи на ПукселыПерИнч - переходим почти что к СИ - пикселей на сантиметр. 6 миллиметров, видимо, скроллбар или менюбар. Штоб наверняка влезло ;)
          Добротное, качественное гэ, а не какое-то там унылое говнецо.

          Неплохо доставляет, что переменная length рассчитывается от якоря top. Что-то этот length шибко смахивает на height.
          Ответить

      • ava Billy the Kidd 09.12.2009 20:40 # −0.8

        Не силен в Питоне. Если "\d" === "[0-9]", то нельзя. Автор берет только показатель со значащим первым разрядом, в т.ч., и состоящий из ровно одного разряда.
        Ответить

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