Какие вы можете привести антипаттерны (или примеры плохого стиля) программирования для node.js?nodejs-61

// Плохой пример
fs.readFile('file1.txt', 'utf8', (err, data1) => {
  if (err) throw err;
  fs.readFile('file2.txt', 'utf8', (err, data2) => {
    if (err) throw err;
    fs.writeFile('result.txt', data1 + data2, (err) => {
      if (err) throw err;
      console.log('Done!');
    });
  });
});

// Хороший пример
async function processFiles() {
  try {
    const [data1, data2] = await Promise.all([
      fs.promises.readFile('file1.txt', 'utf8'),
      fs.promises.readFile('file2.txt', 'utf8')
    ]);
    await fs.promises.writeFile('result.txt', data1 + data2);
    console.log('Done!');
  } catch (err) {
    console.error(err);
  }
}

// Плохой пример - синхронные операции в обработчике запроса
app.get('/data', (req, res) => {
  const data = fs.readFileSync('huge-file.json'); // Блокирует event loop
  res.send(data);
});

// Плохой пример 1 - игнорирование ошибок
fs.readFile('file.txt', (err, data) => {
  console.log(data); // Что если err?
});

// Плохой пример 2 - неправильное использование rejections
new Promise((resolve, reject) => {
  someAsyncOperation((err) => {
    if (err) throw err; // Не попадет в catch!
  });
});

// Плохой пример - использование глобальных переменных
let cache = {};

app.get('/data', (req, res) => {
  if (!cache.data) {
    cache.data = fetchData(); // Проблемы при кластеризации
  }
  res.send(cache.data);
});

// Плохой пример - отсутствие обработки backpressure
readable.pipe(writable); // Без обработки переполнения

readable.on('data', (chunk) => {
  if (!writable.write(chunk)) {
    readable.pause();
    writable.once('drain', () => readable.resume());
  }
});

// Плохой пример - middleware с множеством ответственностей
app.use((req, res, next) => {
  authenticate(req);
  logRequest(req);
  validateParams(req);
  checkPermissions(req);
  transformData(req);
  next(); // Слишком сложно для тестирования
});

// Плохой пример - загрузка всего файла в память
app.post('/upload', (req, res) => {
  let data = '';
  req.on('data', chunk => data += chunk); // Проблема с большими файлами
  req.on('end', () => processData(data));
});

app.post('/upload', (req, res) => {
  const transform = new TransformStream({ /* ... */ });
  req.pipe(transform).pipe(res);
});

// Плохой пример - жесткие зависимости от окружения
if (process.env.NODE_ENV === 'production') {
  db.connect('prod:password@localhost'); // Креды в коде
}

// Плохой пример
setTimeout(() => {}, 86400000); // Что это за число?
if (status === 4) { ... } // Что означает 4?

// Плохой пример - require внутри функций
function getConfig() {
  return require('./config.json'); // Кэшируется только при первом вызове
}

// Плохой пример - циклические зависимости
// a.js
require('./b');
// b.js
require('./a');