Используйте возвращаемое значение delete() вместо лишней проверки!

У Map.delete() и Set.delete() есть полезное свойство: метод возвращает true, только если элемент действительно существовал и был удалён.

Поэтому вместо двойного поиска:
if (cache.has(key)) {
cache.delete(key);
onRemoved(key);
}

можно сразу использовать результат удаления:
if (cache.delete(key)) {
onRemoved(key);
}

То же самое работает с Set:
if (activeUsers.delete(userId)) {
updateCounter();
}

Вы не выполняете сначала has(), а затем отдельный delete(). Сама операция удаления одновременно отвечает на вопрос, был ли элемент.

🔥 Не проверяйте коллекцию перед удалением, если проверка нужна только ради условия — delete() уже возвращает нужный результат.

📣 JS Ready | #совет