Python-сообщество

Доброго дня.

Я 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.

Спасибо.

Отредактировано likin (Июль 12, 2014 19:31:03)

likin

return list([url[0] for url in urls])

likin
This is a script that builds a sitemap of a site whith a defined depth.

likin

    with open(filename, 'wb') as csvfile:

likin

urls = re.findall(r'<a\s{1,3}href=[\'"]?([^\'" >]+)[\'"][^>]*>?([^<]+)', html)

<!--
 
 <a href=hello>text</a>
 
 -->

Отредактировано py.user.next (Июль 12, 2014 12:09:53)

Спасибо. Разбираюсь.

Ещё добавлю.
optparse устарел. Используй argparse.
Старайся писать код, более похожий на третий питон, а именно, используй print, как функцию и ‘format’ для форматирования строк.
print лучше вообще не использовать, а даже в таких маленьких проектах использовать logger. Это приучает к аккуратности.

Даже глянуть не могу. 404…

На урл внимательно посмотри…

Затупил. Просто гитхабом не пользуюсь, ты же знаешь Джитом тоже. Поэтому прохожу мимо

Отредактировано 4kpt_II (Июль 12, 2014 15:12:53)

Только SVN, только ХАРДКОР!!! :-D

Отредактировано ZZZ (Июль 12, 2014 17:04:33)

Неа. Только hg, только python

ZZZ
Ещё добавлю.optparse устарел. Используй argparse.Старайся писать код, более похожий на третий питон, а именно, используй print, как функцию и ‘format’ для форматирования строк.print лучше вообще не использовать, а даже в таких маленьких проектах использовать logger. Это приучает к аккуратности.