Уведомления

Группа в Telegram: @pythonsu

#1 Июль 24, 2014 02:03:21

simple_user
Зарегистрирован: 2014-07-06
Сообщения: 60
Репутация: +  2  -
Профиль   Отправить e-mail  

оценка кода \ парсер для freelancehunt.com

https://github.com/Oleg-MBO/notify-new-work-freelancehunt
оцените \ подскажите что не так

Офлайн

#2 Июль 25, 2014 08:17:34

Shaman
Зарегистрирован: 2013-03-15
Сообщения: 1369
Репутация: +  88  -
Профиль   Отправить e-mail  

оценка кода \ парсер для freelancehunt.com

Офлайн

#3 Июль 25, 2014 16:58:27

simple_user
Зарегистрирован: 2014-07-06
Сообщения: 60
Репутация: +  2  -
Профиль   Отправить e-mail  

оценка кода \ парсер для freelancehunt.com

читал, там наводится пример, когда у класа всего два метода, включая init
как бы вы сделали проще?

Офлайн

#4 Июль 25, 2014 19:56:18

Shaman
Зарегистрирован: 2013-03-15
Сообщения: 1369
Репутация: +  88  -
Профиль   Отправить e-mail  

оценка кода \ парсер для freelancehunt.com

Во-первых, статью я бы применил к вот этому:

class data_pars:
	def __init__(self,title,link,description=""):
		self.__title = title
		assert link.startswith('http://')
		self.__link = link
		self.__description = description
	@property
	def title(self):
		return self.__title
	@property
	def link(self):
		return self.__link
	@property
	def description(self):
		return self.__description
	def __repr__(self):
		return "Data_Pars('{0}' '{1}')".format(self.__title,self.__link)
Тут как раз два метода, при чем потребность в __repr__ не кажется очень насущной, так что я использовал бы collections.namedtuple.

И ещё вопрос полуличного характера: вы на Паскале/Дельфи много писали?

Офлайн

#5 Июль 26, 2014 14:26:25

simple_user
Зарегистрирован: 2014-07-06
Сообщения: 60
Репутация: +  2  -
Профиль   Отправить e-mail  

оценка кода \ парсер для freelancehunt.com

и в правду, как то не прийшло в голову использовать collections.namedtuple
на паскале чуточку писал, 1-й семест 1-го курса в универе)
но не из-за этого я использовал клас если вы об етом

Офлайн

Board footer

Модераторировать

Powered by DjangoBB

Lo-Fi Version