Доброго дня.
Я PHP разработчик, собираюсь начинать работать, как Python разработчик. У меня есть тестовое задание, которое надо отшлифовать. Проведите, пожалуйста codereview, проект маленький - https://github.com/vlikin/sitemap_builder_test .
Проект строит sitemap. В данном случае, он парсит мой сайт http://shelepen.com.ua . Очень краткая информация находится сдесь - https://github.com/vlikin/sitemap_builder_test/blob/master/README.txt.
Спасибо.
Полная версия: Нуждаюсь в codereview.
likinУлыбнуло.return list([url[0] for url in urls])
В README.txt “with” написано с “h”, хотя в коде правильно написано with, а это одно и то же слово.
likin
This is a script that builds a sitemap of a site whith a defined depth.
likinwith open(filename, 'wb') as csvfile:
Прочитай pep8, это важно. Там ещё кое-что есть.
likinВот это, наверное, самое ужасное там.urls = re.findall(r'<a\s{1,3}href=[\'"]?([^\'" >]+)[\'"][^>]*>?([^<]+)', html)
<!-- <a href=hello>text</a> -->
Спасибо. Разбираюсь.
Ещё добавлю.
optparse устарел. Используй argparse.
Старайся писать код, более похожий на третий питон, а именно, используй print, как функцию и ‘format’ для форматирования строк.
print лучше вообще не использовать, а даже в таких маленьких проектах использовать logger. Это приучает к аккуратности.
optparse устарел. Используй argparse.
Старайся писать код, более похожий на третий питон, а именно, используй print, как функцию и ‘format’ для форматирования строк.
print лучше вообще не использовать, а даже в таких маленьких проектах использовать logger. Это приучает к аккуратности.
Даже глянуть не могу. 404…
На урл внимательно посмотри…
Затупил. Просто гитхабом не пользуюсь, ты же знаешь 
Джитом тоже. Поэтому прохожу мимо
Джитом тоже. Поэтому прохожу мимо
Только SVN, только ХАРДКОР!!! :-D
Неа. Только hg, только python
ZZZ
Ещё добавлю.optparse устарел. Используй argparse.Старайся писать код, более похожий на третий питон, а именно, используй print, как функцию и ‘format’ для форматирования строк.print лучше вообще не использовать, а даже в таких маленьких проектах использовать logger. Это приучает к аккуратности.
Добавил logging. Оставил optparse, так как сдесь написано, что argparse основана на optparse, то есть не является устаревшей - https://docs.python.org/2/howto/argparse.html
Спасибо.