Понимание событийной архитектуры Node.js

प्रथम दूसरा

Для работы с EventEmitter нужно создать расширяющий его класс.

И далее по тексту

class WithLog extends EventEmitter {
  execute(taskFunc) {
    console.log('Before executing');
    this.emit('begin');
    taskFunc();
    this.emit('end');
    console.log('After executing');
  }
}

Этот пример нарушает "S" и "I" из SOLID и создаёт наивысшую связанность.

Для достижения большей гибкости кода лучше избегать расширения стандартных классов.

Во-первых всегда можно сделать импиративно:

const emitter = new EventEmitter();
emitter.emit('begin');
taskFunc();
emitter.emit('end');

Такой код легко читать и легко менять. Минус такого кода в том что он выглядет "не круто".

Если уж не терпится создать класс для повторного использования кода, то лучше помнить мантру делигирование — лучшая альтернатива наследованию:

class WithLog {
  constructor(emitter) {
    this.emitter = emitter;
  }

  execute(taskFunc) {
    console.log('Before executing');
    this.emitter.emit('begin');
    taskFunc();
    this.emitter.emit('end');
    console.log('After executing');
  }
}

const withLog = new WithLog(new EventEmitter());
withLog.execute();

Оператор new несёт гораздо меньше потенциального вреда и рисков полного рефакторинга чем оператор extend.

Такие дела.

Во-первых вказано “Создавайте такие хост-функции, которые принимают коллбэки либо всегда синхронно, либо всегда асинхронно” а про dezalgo не сказано. http://blog.izs.me/post/59142742143/designing-apis-for-asynchrony
Ну а во-вторых, автор допустил ошибку в своей “универсальной” функции

readFileAsArray = function(file, cb = () => {})

Сразу бросилось в глаза значение cb по-умолчанию. И не зря. Потому что в случае когда
1) readFileAsArray принимает колбэк и не использует возвращаемый промис и
2) fs.readFile возвращает ошибку
то срабатывает reject(err), не ловится и вызывает unhandledRejection, что в дальнейшем в статье и происходит

UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: ENOENT: no such file or directory, open

а автор похоже не понимает почему. Уверен, что значение по-умолчанию надо от cb убрать, а тело должно проверять ее наличие

    fs.readFile(file, function(err, data) {
      if (err) {
        if (!cb) return reject(err);
        return cb(err);
      }
      const lines = data.toString().trim().split('\n');
      if (!cb) return resolve(lines);
      cb(null, lines);
    });