Рекурсивная функция Javascript с отсрочкой не возвращается

Question

Я получил эту рекурсивную функцию. Я вижу, что это циклично, когда возврат данных является нулевым, но он не возвращает обещание, когда данные не являются нулевыми после выполнения рекурсивной задачи. Похоже, что, закончив делать рекурсивную задачу, обещание где-то потеряется. Кто-нибудь укажет, что я сделал здесь неправильно?

var callrq1 = function(globalsystemid, globalgraphid, start, end, lastcheck) { 
    var datetimeformat = "YYYY-MM-DD HH:mm:ss"; 
    var d1 = new $.Deferred(); 
    var request1 = "../system/" + globalsystemid + "/highcharts.xml?type=" + globalgraphid + "&start=" + start + "&end=" + end; 
    var requeststring1 = makejson(request1); //this makejson function is an ajax get and return promise 
    requeststring1.done(function(data) { 
    if (data != null) { 
     d1.resolve(data); 
    } else { 
     var theend = moment(lastcheck).format(datetimeformat); 
     var newstart = moment(end).format(datetimeformat); 
     var newend = moment(end).add(1, 'weeks').format(datetimeformat); 
     if (newend <= theend) { 
     //recursive callrq1 
     callrq1(globalsystemid, globalgraphid, newstart, newend, theend); 
     } else { 
     d1.resolve(null); 
     } 
    } 
    }); 
    return d1.promise(); 
} 

callrq1(globalsystemid, globalgraphid, starttimeobj.start, starttimeobj.end, endtimeobj.start).then(function(data) { 
    console.log(data); 
}); 

Answer 1

Вы пропустили разделяющие ваш отложенные в случае рекурсивного вызова. Тем не менее, вы shouldn't be using a deferred для этого, в первую очередь! Просто подключите обратный вызов then и верните обещание результата от вашей функции. Вы можете даже вернуть обещания от обратного вызова, который мы используем для рекурсивного случая:

function callrq1(globalsystemid, globalgraphid, start, end, lastcheck) { 
    var datetimeformat = "YYYY-MM-DD HH:mm:ss"; 
    var request1 = "../system/" + globalsystemid + "/highcharts.xml?type=" + globalgraphid + "&start=" + start + "&end=" + end; 
    var requeststring1 = makejson(request1); //this makejson function is an ajax get and return promise 
    return requeststring1.then(function(data) { 
//^^^^^^    ^^^^ 
    if (data != null) { 
     return data; 
// ^^^^^^ 
    } else { 
     var theend = moment(lastcheck).format(datetimeformat); 
     var newstart = moment(end).format(datetimeformat); 
     var newend = moment(end).add(1, 'weeks').format(datetimeformat); 
     if (newend <= theend) { 
     return callrq1(globalsystemid, globalgraphid, newstart, newend, theend); 
//  ^^^^^^ 
     } else { 
     return null; 
//  ^^^^^^ 
     } 
    } 
    }); 
} 

Answer 2

Вы не разрешение отсроченных в случае рекурсии

var callrq1 = function (globalsystemid, globalgraphid, start, end, lastcheck) { 
    var datetimeformat = "YYYY-MM-DD HH:mm:ss"; 
    var d1 = new $.Deferred(); 
    var request1 = "../system/" + globalsystemid + "/highcharts.xml?type=" + globalgraphid + "&start=" + start + "&end=" + end; 
    var requeststring1 = makejson(request1); //this makejson function is an ajax get and return promise 
    requeststring1.done(function (data) { 
     if (data != null) { 
      d1.resolve(data); 
     } else { 
      var theend = moment(lastcheck).format(datetimeformat); 
      var newstart = moment(end).format(datetimeformat); 
      var newend = moment(end).add(1, 'weeks').format(datetimeformat); 
      if (newend <= theend) { 
       //recursive callrq1 
       callrq1(globalsystemid, globalgraphid, newstart, newend, theend).done(function(data){ 
        d1.resolve(data);//pass any data required 
       }); 
      } else { 
       d1.resolve(null); 
      } 
     } 
    }); 
    return d1.promise(); 
} 

callrq1(globalsystemid, globalgraphid, starttimeobj.start, starttimeobj.end, endtimeobj.start).then(function (data) { 
    console.log(data); 
});