5. - Serggi0/python-project-lvl3 GitHub Wiki

Валентин

https://github.com/Serggi0/python-project-lvl3/blob/fcabef33c70b4c3b5057d671a31a407bd2a1baab/page_loader/normalize_data.py#L81
sistem → system https://github.com/Serggi0/python-project-lvl3/blob/fcabef33c70b4c3b5057d671a31a407bd2a1baab/page_loader/web_data_processing.py#L96
https://github.com/Serggi0/python-project-lvl3/blob/fcabef33c70b4c3b5057d671a31a407bd2a1baab/page_loader/web_data_processing.py#L54
relativ → relative https://github.com/Serggi0/python-project-lvl3/blob/fcabef33c70b4c3b5057d671a31a407bd2a1baab/page_loader/web_data_processing.py#L56
названия для функций-предикатов лучше формулировать как вопрос, в данном случае мы проверяем, является ли ресурс локальным https://github.com/Serggi0/python-project-lvl3/blob/fcabef33c70b4c3b5057d671a31a407bd2a1baab/page_loader/web_data_processing.py#L43
может check_domain_name передавать просто url и там внутри уже получать домен? https://github.com/Serggi0/python-project-lvl3/blob/fcabef33c70b4c3b5057d671a31a407bd2a1baab/page_loader/normalize_data.py#L37
urljoin и так корректно для всех кейсов работает, кажется тут эти if’ы лишние https://github.com/Serggi0/python-project-lvl3/blob/fcabef33c70b4c3b5057d671a31a407bd2a1baab/page_loader/normalize_data.py#L48
тут тоже лучше делать через urlib, а не через регулярки по возможности https://github.com/Serggi0/python-project-lvl3/blob/fcabef33c70b4c3b5057d671a31a407bd2a1baab/page_loader/normalize_data.py#L39
мы сюда же никогда не передаем None, к чему этот if? https://github.com/Serggi0/python-project-lvl3/blob/fcabef33c70b4c3b5057d671a31a407bd2a1baab/page_loader/web_data_processing.py#L78
Тут не может быть пустой response же, посмотрите на реалилзацию get_response_server https://github.com/Serggi0/python-project-lvl3/blob/fcabef33c70b4c3b5057d671a31a407bd2a1baab/page_loader/normalize_data.py#L15
по-моему это можно и не выносить в отдельную функцию