Установка глобальной переменной в обратном вызове

Question

Пробовал упростить мой код, чтобы показать свою проблему.

var rp = require('request-promise'); 
var ids = []; 

runmyFunction(); 
runmyFunction(); 

function runmyFunction() { 
    var id = 5; 
    console.log("runmyFunc: "+ids); 
    if (ids.indexOf(id)==-1){ 
     myFunction(id); 
    } 
} 

function myFunction(id) { 
    var options = { 
     uri: 'someURI' 
     , headers: { 
      'User-Agent': 'Request-Promise' 
     } 
     , json: true 
    }; 
    rp(options) 
     .then(function (response) { 
      ids.push(5); 
      console.log("myFunc: "+ids); 
     }) 
     .catch(function (err) { 
      console.log(err); 
     }); 
} 

В принципе у меня есть эта функция runmyFunction, которая должна выполнять только myFunction если не идентификатор 12345 в нем. Поэтому, если я запустил его 10 раз, а 11-й возвращает id 12345, он должен прекратить выполнение функции. Проблема заключается в том, что идентификатор никогда не попадает в массив, даже если запрос преуспел.

Аналогичные проблемы связаны с тем, что запрос является асинхронным. Является ли это причиной и в моем коде?

// edit Я играл с ним и редактировал код. Текущий код записывает это в консоль:

runmyFunc: 
runmyFunc: 
myFunc:5 
myFunc:5,5 

Это потому, что это асинхронный. Любые идеи, чтобы избежать этого?

Answer 1

Благодаря @Shilly я нашел решение самостоятельно, добавив еще один массив.

var rp = require('request-promise'); 

var ids = []; 
var idstmp = []; 

runmyFunction(); 
runmyFunction(); 

function runmyFunction() { 
    var id = 5; 
    if (ids.indexOf(id) == -1 && idstmp.indexOf(id) == -1) { 
     console.log("I was here."); 
     idstmp.push(id); 
     myFunction(id); 
    } 
} 

function myFunction(id) { 
    var options = { 
     uri: 'someURI' 
     , headers: { 
      'User-Agent': 'Request-Promise' 
     } 
     , json: true 
    }; 
    rp(options) 
     .then(function (response) { 
      idstmp.splice(idstmp.indexOf(id), 1); 
      ids.push(id); 
      console.log("myFunc: " + ids); 
     }) 
     .catch(function (err) { 
     idstmp.splice(idstmp.indexOf(id), 1); 
     }); 
} 

Что это делает только добавляет идентификаторы, которые curently «используется» по запросу в другой массив поэтому он получает блок в то время как неоспоримым работал. Это довольно обходное решение, но работает для меня.

Answer 2

не уверен, но я думаю, ваша проблема будет решена с помощью .bind() возможно поможет:

var rp = require('request-promise'); 
var ids = []; 


function runmyFunction() { 
    if (ids.indexOf(12345)==-1){ 
    myFunction(); 
    } 
} 

function myFunction() { 
    var options = { 
    uri: 'someURI' 
    , headers: { 
     'User-Agent': 'Request-Promise' 
    } 
    , json: true 
}; 
rp(options) 
    .then(function (response,mids) { 
     mids.push(response.data.id); 
    }.bind(ids)) 
    .catch(function (err) { 
     console.log(err); 
    }); 
}