Added oop task

[ibragimovaamina]

No description provided.

[krglkvrmn]

[krglkvrmn]

Я бы лучше использовал время первого и последних сообщений в качестве дефолтных дат, тем более, что их просто получить

Suggested change

	if start is None:
	start = self.default_start
	if end is None:
	end = datetime.now()
	if start is None:
	start = self._chat_history[-1].datetime
	if end is None:
	end = self._chat_history[0].datetime

[krglkvrmn]

[krglkvrmn]

В таких случаях лучше писать else в котором кидать исключение, так как юзер может получить None, передав не тот тип, и потом будет гадать, что пошло не так

Suggested change

	if isinstance(message, Message):
	message.datetime = datetime.now() # Set attribute datetime when message is recieved by chat
	self._chat_history.insert(0, message)
	if isinstance(message, Message):
	message.datetime = datetime.now() # Set attribute datetime when message is recieved by chat
	self._chat_history.insert(0, message)
	else:
	raise TypeError(f"Method does not support type {type(message)}")

[krglkvrmn]

Обычно функции, которые имеют в названии show, не возвращают, а показывают данные

Suggested change

	def show(self):
	if self.datetime:
	return f'{self.datetime} {self.user}: {self.text}'
	else:
	return f'{self.user}: {self.text}'
	def show(self):
	if self.datetime:
	print(f'{self.datetime} {self.user}: {self.text}')
	else:
	print(f'{self.user}: {self.text}')

[krglkvrmn]

Аналогично предыдущему, но подход через такое взаимодействие send и recieve хороший

[krglkvrmn]

Зачем?