- Реализуйте класс Mailbox
-
Если считаете, что Mailbox не хватает еще каких-то технических функций для более готового к использованию API, добавьте их или напишите, что бы вы добавили. Если считаете, что данное API содержит какие-то проблемы с точки зрения проектирования, опишите их и предложите рефакторинг.
-
Какие части кода можно было бы концептуально абстрагировать из реализации Mailbox и сделать их переиспользуемыми? Попробуйте максимально разбить реализацию на переиспользуемые части.
- Добавил возможность указывать, какую часть аргумента конструктора Mailbox нужно использовать для проверки — создан такой mailbox или ещё нет:
const {createMailboxFactory} = require('./mailbox');
const Mailbox = createMailboxFactory((user) => user.email);
const firstMailbox = new Mailbox({name: 'Bob', email: '[email protected]'});
const secondMailbox = new Mailbox({name: 'Alice', email: '[email protected]'});
console.log(firstMailbox === secondMailbox); // => true- Добавил возможность использовать объект, используемый при создании Mailbox в pre-хуках:
// Не показывать Бобу сообщения о его дне рождения
bobMailbox.pre(function(mail, user, send) {
if (user.name !== 'Bob' || mail !== 'Bob\'s birthday') {
send(mail);
}
});На мой взгляд, такая реализация Mailbox даёт больше возможностей пользователям этого класса.
- Вынес в отдельный файл механизм подписок
const subscriptions = createSubscriptions(); // Так создаётся список подписок
subscriptions.add(function (message) {...}); // Так добавляется новая подписка
subscriptions.run(message); // А вот так они все запускаются- Вынес в отдельный файл механизм предусловий
const preconditions = createPreconditions(
identity, // Передаю объект, привязанный к мэйлбоксу
(message) => ... // Указываю, что делать с сообщением, если все предусловия прошли
);
preconditions.add(function (message, user, send) { ... }); // Добавляю новое предусловие
preconditions.run(message); // Запускаю проверку по всем предусловиямБольше, вроде, выносить нечего :)