web-dev-qa-db-ja.com

JavaScriptで文字列を返す

呼び出されたときに正しい文字列を返す単純な関数を作成しようとしています：

function getState(abbr){
   if (abbr=="WY")
   {
   return "Wyoming";
   }
}

そして、呼び出しは次のようになります：

var stateName = getState("WY");

ただし、返されるのはすべて0です。

明らかなものがない場合は申し訳ありません。

更新-私の元の確率は「＆」が原因でしたここに私が扱っている実際のコードがあります：

function getState(abbr){
    var url = "states.asp"
    var state = ""; 
    $.get(url, function(data) {
        var i = 0;
        $.each($('state',data),function(index, el) {            
            if (abbr == ($(this).attr("abbr"))){
                //alert($(this).attr("abbr"));
                state = $(this).text();
            }//if (abbr == $(this).attr("abbr")){
        });//$.each($('state',data),function(index, el) {
    }).success(function() { 
        alert("x" + state);
        return state;
    }); //.success(function() { 
    //$.get(url, function(data) {
    alert("y" + state); 
    return state;
}

私の呼び出しの結果として「未定義」になります：

alert(getState("WY"));

Alert（ "x" + state）は機能します。

更新＃2 ---以下は、states.aspが生成するものすべてです（現時点では）...後で会社などを返します。

<?xml version="1.0" encoding="utf-8"?>
<STATELIST>
<STATE abbr="AL">Alabama</STATE>
<STATE abbr="AK">Alaska</STATE>
<STATE abbr="AZ">Arizona</STATE>
<STATE abbr="AR">Arkansas</STATE>
<STATE abbr="CA">California</STATE>
<STATE abbr="CO">Colorado</STATE>
<STATE abbr="CT">Connecticut</STATE>
<STATE abbr="DE">Delaware</STATE>
<STATE abbr="FL">Florida</STATE>
<STATE abbr="GA">Georgia</STATE>
<STATE abbr="HI">Hawaii</STATE>
<STATE abbr="ID">Idaho</STATE>
<STATE abbr="IL">Illinois</STATE>
<STATE abbr="IN">Indiana</STATE>
<STATE abbr="IA">Iowa</STATE>
<STATE abbr="KS">Kansas</STATE>
<STATE abbr="KY">Kentucky</STATE>
<STATE abbr="LA">Louisiana</STATE>
<STATE abbr="ME">Maine</STATE>
<STATE abbr="MD">Maryland</STATE>
<STATE abbr="MA">Massachusetts</STATE>
<STATE abbr="MI">Michigan</STATE>
<STATE abbr="MN">Minnesota</STATE>
<STATE abbr="MS">Mississippi</STATE>
<STATE abbr="MO">Missouri</STATE>
<STATE abbr="MT">Montana</STATE>
<STATE abbr="NE">Nebraska</STATE>
<STATE abbr="NV">Nevada</STATE>
<STATE abbr="NH">New Hampshire</STATE>
<STATE abbr="NJ">New Jersey</STATE>
<STATE abbr="NM">New Mexico</STATE>
<STATE abbr="NY">New York</STATE>
<STATE abbr="NC">North Carolina</STATE>
<STATE abbr="ND">North Dakota</STATE>
<STATE abbr="OH">Ohio</STATE>
<STATE abbr="OK">Oklahoma</STATE>
<STATE abbr="OR">Oregon</STATE>
<STATE abbr="PA">Pennsylvania</STATE>
<STATE abbr="RI">Rhode Island</STATE>
<STATE abbr="SC">South Carolina</STATE>
<STATE abbr="SD">South Dakota</STATE>
<STATE abbr="TN">Tennessee</STATE>
<STATE abbr="TX">Texas</STATE>
<STATE abbr="UT">Utah</STATE>
<STATE abbr="VT">Vermont</STATE>
<STATE abbr="VA">Virginia</STATE>
<STATE abbr="WA">Washington</STATE>
<STATE abbr="WV">West Virginia</STATE>
<STATE abbr="WI">Wisconsin</STATE>
<STATE abbr="WY">Wyoming</STATE>
</STATELIST>
javascriptstringasp-classicreturn
8
tree

最も簡単な方法はハッシュを作成することです-関数は必要ありません。

_var states = {
    'AL': 'Alabama',
    'AK': 'Alaska',
    'AZ': 'Arizona',
    'AR': 'Arkansas',
    'CA': 'California',
    'CO': 'Colorado',
    'CT': 'Connecticut',
    ...
    'WY': 'Wyoming'
};

var stateName = states["WY"];
_

編集

これで、getState()がサーバーから状態名を取得する必要があることをよりよく理解できます。これは、通常の同期コーディングとはかなり異なる非同期コーディングの世界にあなたを導きます。

実現する最も重要なことは、getState()は、指定された州の省略形の州名を単に返すことはできないということです。どうして？サーバーへのajax呼び出しはasynchronousです。つまり、getState()はサーバーの応答を待たずに戻ります。

非同期性を処理するには、基本的に2つの方法があります。

  • コールバック関数をgetState()に渡して、応答を受け取ったときに何をすべきかを伝えます
  • サーバーが応答したときに応答するように、getState()が呼び出されたときに処理できる「promise」と呼ばれる特別なタイプのオブジェクトを返すようにgetState()を調整します。

以下のコードは2番目のアプローチを採用しています。

_var states = {};//cache of state names, with state abbreviations as keys
function getState(abbr) {
    var dfrd = $.Deferred();//A deferred object, whose promise will be returned.
    if(!states[abbr]) {
        $.ajax({
            url: "states.asp",
            dataType: 'XML',
            success: function(data) {
                //Load up the cache
                $.each($('state', data), function(i, el) {
                    states[el.attr('abbr')] = $(el).text();
                });
                //Now resolve or reject the deferred object depending in whether states[abbr] has been cached
                if(states[abbr]) {
                    dfrd.resolve(abbr, states[abbr]);//Success! Let's resolve the deferred object (and its promise).
                }
                else {
                    dfrd.reject(abbr, 'States successfully downloaded but ' + abbr + ' was not included');
                }
            },
            error: function() {
                dfrd.reject(abbr, 'Download of states failed');
            }
        });
    }
    else {
        //The state name is already cached
        //The deferred object (and its promise) can be resolved without needing to make another ajax call.
        dfrd.resolve(abbr, states[abbr]);
    }
    return dfrd.promise();
}
_

未テスト

これで、知っておく必要があるのは、getState()を呼び出す方法だけです。

_getState("WY").done(function(abbr, state) {
    alert(abbr + ': ' + state);
    //other stuff here
}).fail(function(abbr, message) {
    alert(abbr + ': ' + message);
    //other stuff here
});
_

ご覧のように、getState()が返す値が.done()関数の2番目の引数として表示されます。目安として、省略形（ "WY"）が最初の引数として表示されます。

エラー状態を処理したい場合（常に良いアイデア）、.fail()コールバックで処理します。

すべてがどのように機能するかについてのより多くの手がかりについては、コード内のコメントを参照してください。

8
Beetroot-Beetroot

あなたのコードには絶対に何も壊れていません。これを見てください 実用的なデモ 。

function getState(abbr){
   if (abbr=="WY")
   {
   return "Wyoming";
   }
}
var stateName = getState("WY");
alert(stateName);
​
4
Paul Fleming

HTML：

<div id="msg">...</div>

JS：

function getState(abbr){
   if (abbr=="WY")
   {
   return "Wyoming";
   }
}
var stateName = getState("WY");
​document.getElementById('msg').innerHTML=stateName;
3
Ali Mohammadi

これは、評価する多くのifステートメントになります。関数は、ワイオミングに到達するために、すべての50のifステートメントを通過する必要がある可能性があります。 switchステートメントを使用して、次のように書き直します。

function getState(abbr){

   switch case(abbr) {

      case "WY":
         return "Wyoming";
      ...

   }
}

これはもっと速いはずだと思います。入力が正しくない場合に備えて、defaultケースも追加します。

2
frenchie

これは私の解決策です。私はまだ非同期の振る舞いを理解しようとしているストローをつかんでいますが、これがうまくいきました：

の代わりに：

function searchServing(which,choice){
   var state = getState("WY");
   $("#searchResults").html("<tr><td>" + state + "</td></tr>");
   .get(function() {

   } //for reference call this getSearchServing
}

私はコールを逆にする必要がありました：

function getState(abbr){
  .get(function(){
      $("#searchResults").html("<tr><td>" + state + "</td></tr>");
  })
  .success(function() { 
    searchServing('serving',abbr);
  })
}

まだgetSearchServingがgetStateの前に実行される理由がわからない、なぜgetStateから値を返せなかったのかわからない???

0
tree