私のxhr.onreadystatechange関数はifコードを実行する前にelseコードを2回実行しています

Question

JavaSciptで簡単な天気アプリを作っています。私の目標は、ユーザーが場所を入力し、天気提供者にその場所がない場合、入力ボックスが揺れます（これはloadWeatherError（）と同じです）。以下のコードは、elseステートメントを2回実行してから、ifコードを実行しています。これは、loadWeatherError（）関数が2回実行されており、場所が有効であっても入力ボックスが揺れることを意味します。だから、私がそれを実行すると、エラー2アラートが2回、エラー1アラートが1回表示されます。 loadWeatherError（）関数は一度しか実行せず、天気プロバイダが正しいデータを返さなかった場合にのみ実行する方法がありますか？

xhr.onreadystatechange = function() { 
 
    var DONE = 4; // readyState 4 means the request is done. 
 
    var OK = 200; // status 200 is a successful return. 
 
    if (xhr.readyState === DONE) { 
 
    if (xhr.status === OK) 
 
     alert("Error 1"); 
 
    var data = JSON.parse(xhr.responseText); 
 
    if (data.response) { //deal with wunderground api 
 

 
    } else { //deal with Yahoo api 
 

 
    } 
 
    } else { 
 
    alert("Error 2"); 
 
    loadWeatherError(); 
 
    options.error("There is a problem receiving the latest weather. Try again."); 
 
    } 
 

 
};

Answer 1

何が起こっているあなたの状態が変化しているが、それはあなたがDONE値に到達するために準備状態のそれぞれを通過する必要が4に等しいされていません。状態が変わるたびにコードが実行されているため、エラーが発生します。状態が正しくない場合、エラーを出力するコードを削除します。

xhr.onreadystatechange = function() { 
    var DONE = 4; // readyState 4 means the request is done. 
    var OK = 200; // status 200 is a successful return. 
    if (xhr.readyState === DONE) { 
    if (xhr.status === OK) { 
     alert("Error 1"); 
    } else { 
     alert("Error 2"); 
     loadWeatherError(); 
     options.error("There is a problem receiving the latest weather. Try again."); 
    } 
    var data = JSON.parse(xhr.responseText); 
    if (data.response) { //deal with wunderground api 

    } else { //deal with Yahoo api 
     alert("Error 2"); 
     loadWeatherError(); 
     options.error("There is a problem receiving the latest weather. Try again."); 
    } 
    } 
};