DjangoBB LoFi version

Начало » Python для новичков » Сокращение кода

paraman

Июль 16, 2012 16:46:52

Посоветуйте, что почитать, чтобы в строчку писать условия.
На подобие:

for info in g1.xpath_list('//*[@class="info"]'):
    info1 = str(info.text).strip()
    if info1:
        info2.append(info1)

Одним из участников форума был сокращён до строчки вида:

info = [item.strip() for item in (g1.xpath_list('//*[@class="info"]/text()'))]

Предпологаю, что данное сокращение было по схеме:

(выражение for элемент in последовательность)

Пытался сократить данный код по схеме:

(выражение for элемент1 in последовательность1 if условие1)

for info in g.xpath_list('//*[@class="info"]'):
    info1 = str(info.text).strip().replace('None', '1-1')
    if str(info1):
	info2.append(info1)
    if len(info1) > 1:
       info3.append(int(result1.split(' - ')[0])+int(info1.split(' - ')[1]))
    else:
        info3.append(info1)

Сократить не смог, всё время хрень выходила. Как я понимаю, тут два цикла обычный(мой) и сокращённый(если не ошибаюсь его написал odnochlen).
Если не сложно покажите, как же на самом деле его сократить.

romis

Июль 16, 2012 17:01:58

info2 = [str(info.text).strip() for info in (g1.xpath_list('//*[@class="info"]/text()')) if str(info.text).strip()]

paraman

Июль 16, 2012 17:35:26

romis

info2 = [str(info.text).strip() for info in (g1.xpath_list('//*[@class="info"]/text()')) if str(info.text).strip()]

Убираем text d

g1.xpath_list('//*[@class="info"]/text()

тогда работает, а вот второе условие.
Написал так:

info2 = [str(info.text).strip() for info in (g.xpath_list('//*[@class="info"]')) if str(info.text).strip() if len(str(info.text)) > 1 info3.append(int(info.split(' - ')[0])+int(info.split(' - ')[1]))]

Но вываливается с ошибкой.

Lexander

Июль 16, 2012 17:44:04

А смысл в таком сокращении кода?
Он становится нечитаем и уже не соответствует требованиям PEP о длине строки.

paraman

Июль 16, 2012 17:48:50

ActiveState Komodo IDE 7 комодо ругается, на в строчке

info2.append(info1)

indentation contains tabs

fata1ex

Июль 16, 2012 17:49:23

Lexander, тут, кажется, бессмысленно что-то объяснять. Человек не хочет понимать, что нужно осознавать, что ты пишешь, знать базовые конструкции языка, которые называются не “сокращением кода”, а, например, “списочные выражения”. И тогда не будет вопросов, почему (… if a if b x.append) не работает. Ну а писать третий раз про смысл сокращения всего и вся я, пожалуй, не буду.

Хах, подумал, что это похоже на брутфорс языка :)

paraman

Июль 16, 2012 22:32:47

fata1ex
Lexander, тут, кажется, бессмысленно что-то объяснять. Человек не хочет понимать, что нужно осознавать, что ты пишешь, знать базовые конструкции языка, которые называются не “сокращением кода”, а, например, “списочные выражения”. И тогда не будет вопросов, почему (… if a if b x.append) не работает. Ну а писать третий раз про смысл сокращения всего и вся я, пожалуй, не буду.

Хах, подумал, что это похоже на брутфорс языка

fata1ex
Думаю, для начала нужно нормально изучить язык, а потом уже применять какие-то методологии.

От части с Вами соглашусь, но всем новичкам советовать одно и тоже. Какой смысл тогда вообще писать?
Я прекрасно понимаю, что без основы (теории), нет ничего. Но практика важна.

fata1ex

Июль 16, 2012 22:46:43

paraman, если вы начнёте учиться бегать стометровку на руках с криками “Практика важна!”, то вряд ли получится что-нибудь дельное ) Бессмысленно пытаться решать диффуры без умения брать производную. И так далее.

А то, что многим советуется набрать базу по языку, неудивительно. Мало кто хочет действительно разобраться в чем-нибудь, лишь бы побыстрее да попроще. И это нормально. Ну и, конечно, я вряд ли назову больше 2-3 человек, которые действительно прислушались к советам на этом форуме. И это тоже нормально. Но, может, что-нибудь и откладывается :)

Ну и насчет “всем советовать одно и тоже” - это неправда.

paraman

Июль 16, 2012 23:45:28

И всё же, я прислушиваюсь к Вам. Пока не прочту 200 страниц, не напишу более сюда за помощью.