Сам недавно случайно наткнулся, что при скачивании большого количества файлов с помощью javascript браузер виснет и падает. Правда в хроме нужно иногда разрешить сайту скачивать много файлов, но это не всегда помогает, т.к. хром качает их в фоне. Протестировать можно здесь: https://iliakonnov.github.io/chromeCrash/index.html
Вместо page_count = page_count+[get_page_count(get_html(page_count[count]),page_count)] лучше использовать page_count.append(get_page_count(get_html(page_count[count]),page_count))
Вместо str(input()) лучше использовать raw_input(), т.к. input выполняет eval(raw_input()), что позволяет выполнять произвольный код.
Вместо count = count + 1 лучше использовать count += 1
Да и функцию get_html можно переписать с использованием requests, для того, чтобы убрать один import:
def get_html(url):
r = requests.get(url)
return r.text
Однако я не уверен, что ничего не сломается, давно requests не пользовался.
ИМХО можно переименовать переменную perehod на redirect, а переменную a (используется для записи файла) на f. Так будет понятнее, что это именно файл.
Картинка для привлечения внимания. Ты картинка, что до ката.
Также я бы посоветовал для форматирования строк использовать не %, а .format. Поддерживается начиная с 2.6 (https://docs.python.org/2.6/library/string.html#string.Formatter.format)
Но если автор использует Python 3, то преобразование в строку излишне
Вместо
page_count = page_count+[get_page_count(get_html(page_count[count]),page_count)]
лучше использоватьpage_count.append(get_page_count(get_html(page_count[count]),page_count))
Вместо
str(input())
лучше использоватьraw_input()
, т.к. input выполняетeval(raw_input())
, что позволяет выполнять произвольный код.Вместо
count = count + 1
лучше использоватьcount += 1
Да и функцию get_html можно переписать с использованием requests, для того, чтобы убрать один import:
Однако я не уверен, что ничего не сломается, давно requests не пользовался.
ИМХО можно переименовать переменную
perehod
наredirect
, а переменнуюa
(используется для записи файла) наf
. Так будет понятнее, что это именно файл.