Search
Write a publication
Pull to refresh
4
0.1
Илья @Sulerad

User

Send message

Картинка для привлечения внимания. Ты картинка, что до ката.

Также я бы посоветовал для форматирования строк использовать не %, а .format. Поддерживается начиная с 2.6 (https://docs.python.org/2.6/library/string.html#string.Formatter.format)

Сам недавно случайно наткнулся, что при скачивании большого количества файлов с помощью javascript браузер виснет и падает. Правда в хроме нужно иногда разрешить сайту скачивать много файлов, но это не всегда помогает, т.к. хром качает их в фоне. Протестировать можно здесь: https://iliakonnov.github.io/chromeCrash/index.html

Но если автор использует 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:


def get_html(url):
    r = requests.get(url)
    return r.text

Однако я не уверен, что ничего не сломается, давно requests не пользовался.


ИМХО можно переименовать переменную perehod на redirect, а переменную a (используется для записи файла) на f. Так будет понятнее, что это именно файл.

12 ...
7

Information

Rating
3,056-th
Location
Москва, Москва и Московская обл., Россия
Registered
Activity