Как считаете, нормально ли, что у меня встречаются в коде такие строки? Имеет ли смысл разбить строку на несколько строк?

output.write('MMS upload time (sec)\t'+'\t'.join(str(avg_upload_time) for ms_key in sorted(avg_upload_time.keys()))+'\n')

Думаю стоит

я бы так написал:
или так:
2.7 кажись уже отсортированными ключи отдает.

pill
for key in avg_upload_time

Разве можно без .keys() получить список ключей из avg_upload_time?

Проверил, вижу что можно. Удивлен честно говоря.

>>> sorted_values = map(avg_upload_time.get, sorted(avg_upload_time))
>>> output.write('\t'.join( + sorted_values))
>>> output.write('\n')

..bw

bw
Тоже красиво. Спасибо. Привет Сибири!

раньше мне тоже очень нравилось выравнивать весь код в одну строчку
правда то был пхп, а питон дает ещё большие возможности в этом плане

но, потом, где-то прочитал что именно это и зовется говнокодом :)
поэтому, с опытом, я стал придавать большее значение понятности кода, чем его кратости

Игнат
Дело в том что мне код кажется совершенно понятным и я достаточно часто встречаю подобные строки в чужом коде и они не пугают. Но вот, обнаружил, что мои сотрудники впадают в ступор при виде таких строк. Вариант pill'а мне кажется самым разумным. Просто вынести сложную часть в отдельную строку.

Пишите код так, как будто сопровождать его будет склонный к насилию психопат, который знает, где вы живете. Стив Макконнелл :)