Skip to content

Comments

Добавление важного условия в текст задачи (Хранение отметок "не прочитано")#1912

Closed
h0ttab wants to merge 1 commit intojavascript-tutorial:masterfrom
h0ttab:patch-1
Closed

Добавление важного условия в текст задачи (Хранение отметок "не прочитано")#1912
h0ttab wants to merge 1 commit intojavascript-tutorial:masterfrom
h0ttab:patch-1

Conversation

@h0ttab
Copy link
Contributor

@h0ttab h0ttab commented Nov 13, 2023

Описание

Я считаю, что в задаче пропущено важное условие, которое должно чётко определить, существует ли вероятность того, что пользователь отправит два отдельных сообщения с одинаковым текстом, или же все сообщения от конкретного пользователя будут строго уникальными.

Сейчас я объясню, почему это важно:
Если пользователь может отправить два идентичных сообщения, то решение задачи из учебника не будет работать.

Если следовать решению из учебника, то получается, что если пользователь John написал мне "Hello", и я прочитал это сообщение, а потом John снова напишет мне "Hello" через полчаса - новое сообщение автоматически отметится прочитанным, т.к. сообщение с таким текстом и от такого пользователя уже есть в списке прочитанных.

Кроме того, в задаче чёрным по белому написано, что мы НЕ знаем, когда именно будут удалены старые сообщения. Поэтому мы можем предположить, что вероятность появления идентичных сообщений вполне реальна, т.к. мы не можем полагаться на то, что старое сообщение уже будет удалено к тому моменту, когда придёт новое.

Варианта два: либо добавить в задачу условие, что все сообщения конкретного пользователя будут строго уникальными (и тогда решение автора будет верным), либо надо придумывать новое решение задачи, т.к. в нём не учтено влияние очень существенного фактора, в виде появления одинаковых сообщений от одного и того же пользователя.

Мы все пользуемся соцсетями или мессенджерами, и прекрасно знаем, что некоторые сообщение очень часто повторяются, и если кроме имени и текста сообщения никаких идентификаторов нет (как в условии задачи), то отличить их друг от друга не получится, хотя, технически, это два разных сообщения.

Ссылки

Связанные Issue

@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label Nov 13, 2023
@javascript-translate-bot javascript-translate-bot requested a review from a team November 13, 2023 21:11
@h0ttab h0ttab closed this by deleting the head repository Nov 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review needed Review needed, please approve or request changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants