Brauchen Sie Hilfe mit Extjs-Funktion die Rückgabe undefiniert Ergebnisse

Ich will im folgenden mit meinem javascript-codeblock.

Verarbeiten alle aktuellen und neuen device-requests, dh. erkennen,verschlüsseln,entschlüsseln,usw.
Das Ergebnis an die aufrufende Methode

Fragen

Wie kann ich den vorhandenen code und loszuwerden des javascript strict-Warnung:anonym-Funktion nicht immer einen Wert zurück.
Was ist der richtige Weg, der Aufruf meiner Methode?

Jede Hilfe wird sehr geschätzt

Dank!

Hiermit den code:

Dies ist, wie rufe ich die Methode, die derzeit

//Contents of SmEditor.js
var response = Ext.decode(Prometheus.DeviceRequestHelper.detect(request_id));  


//contents of Sm.js
Ext.ns('myApp') 
myApp.DeviceRequestHelper = {  
    detect:function(request_id){  
        var task = function(){  
        Ext.Ajax.request({  
            url: 'device_requests.php',  
            params:{  
                action:'get_device', //in php  
                'request_id':request_id  
                },  
            timeout:30000, //30 seconds  
            success:function(response){//serverside response  
                var result = Ext.decode(response.responseText); //convert to js objects  
                if(result.success == true){//device was detected  
                    cons.log('success,device was detected');  
                    cons.log(result);  
                    Ext.TaskMgr.stop(runTask);  
                    return Ext.encode(result); //javascript strict warning  
                }else{  
                    if(runTask.taskRunCount >= 10){  
                        //retry limit exceeded  
                        Ext.Msg.show({  
                            title:'Server Failure',  
                            msg:"Detection Failed,Unable to detect device",  
                            icon: Ext.MessageBox.ERROR,  
                            buttons: Ext.Msg.OK  
                        });  
                        Ext.MessageBox.getDialog().getEl().setStyle('z-index','80000');  
                        Ext.TaskMgr.stop(runTask);  
                    }  
                }  
            },  
            failure:function(response){  
                Ext.TaskMgr.stop(runTask);  
                Ext.Msg.show({  
                    title:'Server Failure',  
                    msg:"Failed, server communication error",  
                    icon: Ext.MessageBox.ERROR,  
                    buttons: Ext.Msg.OK  
                });  
                Ext.MessageBox.getDialog().getEl().setStyle('z-index','80000');  
            }  
        })  
        }
        var runTask = {  
            run: task,  
            interval:2000,  
            repeat:10  
            };  
        Ext.TaskMgr.start(runTask);  
    }  
}

InformationsquelleAutor QCar | 2010-08-05

0

Um zu verhindern, dass diese Art von Warnung, die Funktion return einen Wert in allen Fällen oder nicht Fällen. Im moment sind Sie nur die Rückgabe eines Wertes in eine if Fall; die anderen Fälle wird nicht wieder alles. Sie können sogar return undefined um die Warnung Weg. Jedoch, was es sagt, ist richtig: dass eine Funktion, die manchmal einen Rückgabewert und manchmal nicht das ist ein wenig seltsam aus und schlägt dann machst du etwas falsch.

Was Sie scheinen zu wollen, zu tun haben, ist die innere return im success Methode einen Wert zurückgeben von der detect() Methode. Das ist absolut nicht möglich. Die innere Funktion kann nur einen Wert an den Aufrufer des success, die Prototypen selbst. Durch die Zeit dies geschieht, ist die detect() Methode hat sich längst zurückgegeben.

Was Sie hier haben, ist der asynchrone code. Die detect() Methode können einen AJAX-request, aber es muss dann wieder sofort an den Aufrufer zurückgeben wird, die Kontrolle an den browser. Einige Zeit später, die HTTP-Anforderung, die hinter der AJAX-Aufruf wird abgeschlossen, und dann die success Funktion ausgelöst wird. JavaScript nicht aufrufen asynchroner code synchron, oder Umgekehrt.

Was Sie tun müssen, ist, übergeben Sie eine callback-Funktion in Ihrer Methode, und rufen Sie dann zurück auf Fertigstellung:
```
Prometheus.DeviceRequestHelper.detect(request_id, function(response) {
    //do something with `response`
});

myApp.DeviceRequestHelper= {  
    detect: function(request_id, callback) {
        ...
        Ext.Ajax.request({
            ...
            success: function(xhr) {
                var result= Ext.decode(xhr.responseText);
                if (result.success)
                    callback(result);
                ...
            },
            ...
        });
    },
    ...
 };
```
(Ich entfernte den extra Ext.encode->Ext.decode paar, das nur scheint wie eine Verschwendung von Zeit.)
- Super!!! Danke für deine Zeit um meine Frage zu beantworten.Ich werde die änderungen implementieren und testen Sie den code, bevor ich markiere die Frage als beantwortet. Wieder einmal, vielen Dank!
- Nach dem code, den Sie verwenden, um call detect()... sind Sie sicher, dass die function() {...} ist innerhalb der eckigen Klammern als zweites argument?
- Danke!!! GELÖST ich habe vergessen, die Funktion() in die Klammern. Ich bekomme die Ausgabe, die ich wollte 🙂
InformationsquelleAutor bobince

Zuerst, Ihre Erkennung wird die Methode keinen Wert zurück und wird sofort wieder(sogar vor dem ajax-Aufruf abgeschlossen ist), weil der ajax-Aufruf ist asynchron

Zweiten, gibt es keinen Punkt gibt einen Wert zurück in Ihre Erfolgs-handler. Stattdessen müssen Sie eine callback-Funktion, um Ihre Methode erkennen, zum Beispiel so:

Ext.decode(Prometheus.DeviceRequestHelper.detect(request_id, function(response) {
      //do something with your response
}));

//detect function takes a callback function as a parameter
myApp.DeviceRequestHelper = {  
    detect:function(request_id, funCallback){  //pass in a callback function that is
                                               //called when result was a success
        var task = function(){  
           Ext.Ajax.request({  
               url: 'device_requests.php',  
               params:{  
                   action:'get_device', //in php  
                   'request_id':request_id  
                   },  
               timeout:30000, //30 seconds  
               success:function(response){//serverside response  
                   var result = Ext.decode(response.responseText); //convert to js objects  
                   if(result.success == true){//device was detected  
                       cons.log('success,device was detected');  
                       cons.log(result);  
                       Ext.TaskMgr.stop(runTask);  
                       //return Ext.encode(result); //javascript strict warning
                       funCallback(Ext.encode(result)); //===========> callback function called.
                   }else{  
                       if(runTask.taskRunCount >= 10){  
                           //retry limit exceeded  
                           Ext.Msg.show({  
                               title:'Server Failure',  
                               msg:"Detection Failed,Unable to detect device",  
                               icon: Ext.MessageBox.ERROR,  
                               buttons: Ext.Msg.OK  
                           });  
                           Ext.MessageBox.getDialog().getEl().setStyle('z-index','80000');  
                           Ext.TaskMgr.stop(runTask);  
                       }  
                   }  
               },  
               failure:function(response){  
                   //... failure handing code  
               }  
           });  
        }
        var runTask = {  
            run: task,  
            interval:2000,  
            repeat:10  
            };  
        Ext.TaskMgr.start(runTask);  
    }  
}

Vielen Dank für deine Antwort, es ist klar, dass du und bobince sind in der Vereinbarung über die Nutzung einer callback-Funktion.Bobince gab mir eine ausführliche Erklärung, warum mein code nicht funktionieren.

InformationsquelleAutor naikus

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.