Не будет ли утечки памяти?

Question

[motomac]

Пишу небольшой gateway для API на Express. Сервер принимает HTTP-запрос, читает токен и делает несколько запросов на основной REST API, снабжая каждый этим токеном.

Чтобы не вызывать каждый метод из модуля api с токеном, использую конструктор и классы ES6. Но возникли сомнения в правильности моего подхода. Не пахнет ли здесь бесконтрольным созданием экземпляров класса Api при множественных запросах от разных клиентов и последующими утечками памяти?

Ну и в целом может есть более элегантный способ организовать код?

// index.js
const Api = require('./api')

app.get('/forward_to_api', function (req, res) {
  let api = new Api(res.locals.token)

  api.getSomething('something')
    .then(response => ...)
    .catch(error => console.log(`Error: ${error.message}`))

  api.getSomethingNew('somethingNew')
    .then(response => ...)
    .catch(error => console.log(`Error: ${error.message}`))
})

// api.js
module.exports = class Api {
  constructor(token) {
    this.config = {
      headers: { 'Authorization': `Bearer ${token}` }
    }
  }

  getSomething(name) {
    return axios.get(`/api/something/${name}`, this.config)
  }

  getSomethingNew(name) {
    return axios.get(`/api/something_new/${name}`, this.config)
  }
}

Answer 1

[Vit]

Во-первых, утечек памяти не будет, обо всём позаболится сборщик мусора.
Но, во-вторых, у вас в целом код очень странный.
Вы получаете один запрос на /forward_to_api, внутри делаете два параллельных запроса. Как только один из них отрабатывает - вы шлёте клиенту ответ и закрываете с ним соединение. Вы понимаете, что когда отработает второй запрос, ему уже некуда будет отправить свой status 204? В чём смысл такого кода?

Comments

[motomac]

Да-да, на это не смотрите. Это результат невнимательности при подготовке к публикации на Тостере. Там более сложные операции на самом деле.

Спасибо за ответ! Единственное, что меня еще смущает - это то, что у меня несколько точек входа и при текущей организации кода в каждой создается экземпляр Api:

app.get('/forward_to_api1', function (req, res) {
  let api = new Api(res.locals.token)
  ...
}

app.get('/forward_to_api2', function (req, res) {
  let api = new Api(res.locals.token)
  ...
}

[Vit]

motomac, В языках с автоматической сборкой мусора можете не бояться плодить короткоживущие объекты хоть миллионами.

Answer 2

[Coder321]

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

Comments

[motomac]

Там несколько параллельных асинхронных запросов в ответ на событие от клиента, потому подход с одним экземпляром, очевидно, приведет к путанице токенов.

[Coder321]

Так вам не нужно сохранять токен в экземпляре, передавайте его на вход в функцию

getSomething(name, token) {
    return axios.get(`/api/something/${name}`, this.createConfig(token))
  }

 createConfig(token){
     return {
      headers: { 'Authorization': `Bearer ${token}` }
    }
 }