0

Пытаюсь передать в переменную request

function custFunc() {}
custFunc.prototype.one = function(options) {
    var this.requestCustom = request.defaults({
        timeout: options.timeout || defaultTimeout
    });
});

но при попытке вызвать функцию из другой выходит ошибка

custFunc.prototype.two = function(options) {
    this.requestCustom.get(/****/);
});

выдает: TypeError: Cannot read property 'get' of undefined

При этом вывов через request нормально работает, что я делаю не так?

2
  • Не понятно, откуда берётся request. И почему должно появиться свойство requestWrapper
    – vp_arth
    7 фев 2017 в 13:45
  • лишний var перед инициализацией свойства requestCustom
    – vp_arth
    7 фев 2017 в 13:46

1 ответ 1

0

У вас лишний var перед инициализацией свойства requestCustom

function custFunc() {}
custFunc.prototype.one = function(options) {
    this.requestCustom = request.defaults({
        timeout: options.timeout || defaultTimeout
    });
});
custFunc.prototype.two = function(options) {
    this.requestCustom.get(/****/);
});

К тому же нужно понимать, что до использования метода two нужно непременно вызвать one;

   var f = new custFunc();
   f.one({});
   f.two({});

Теперь, если request.defaults возвращает объект с методом get, всё должно работать.


Возможно, можно воспользоваться паттерном LazyLoad:

custFunc.prototype.two = function(options) {
    if (!this.requestCustom) {
      this.one(options);
    }
    this.requestCustom.get(/****/);
});

@PavelMayorov suggestion:

function custFunc(options) {
  this.options = options;
}
custFunc.prototype.one = function() {
    this.requestCustom = request.defaults({
        timeout: this.options.timeout || defaultTimeout
    });
});
custFunc.prototype.two = function(options) {
    if (!this.requestCustom) {
      this.one();
    }
    this.requestCustom.get(/****/);
});
///

var f = new custFunc({timeout: 1e3});
f.two({id: 5});
2
  • Для ленивого выполнения параметр options надо передавать в конструктор, а не в каждый метод. Иначе будут побочные эффекты. 7 фев 2017 в 14:02
  • Добавил этот вариант, но подозреваю тут будут серьёзные расхождения с изначальной задачей автора(которую мы, естественно, не видели)
    – vp_arth
    7 фев 2017 в 14:09

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.