Tasks

[psilentium]

No description provided.

[matklad]

str(''.join([str(i) for i in a])) это масло масляное, ''.join([str(i) for i in a])это уже строка.

[matklad]

''.join([str(i) for i in a]) это почти тоже самое, что и ''.join((str(i) for i in a)), что можно сократить до ''.join(str(i) for i in a)

[matklad]

Больше пробелов

По PEP8 нужно ставить пробелы вокруг бинарных операторов: a + b а не a+b.

После запятых пробелы надо ставить всегда, даже в русском языке =)

Pycharm умеет делать реформат кода

[matklad]

seq объявляется тут...

[matklad]

лучше вместо цикла записать

sum += sum(i for i in values if i > 0)

где первый sum это переменная, а второй -- та самая build-in функция

[matklad]

В целом хорошо =)

• Обратить внимание на работу с файлами(https://github.com/effect/bii-14s/blob/master/RECOMMENDATIONS.md)
  • Меньше глобальных переменных
  • Лучше использовать 'более функциональный' стиль -- писать функции, которые возвращают значения, а не функции-процедуры, которые изменяют аргументы/глобальные переменные