配列内の文字列を検索する最良の方法

Question

URLから取得したデータを配列の作物に追加します。この後、私は新しい配列の助けを借りて重複作物を確認していますcropsSoFar。

 var crops=[]; 

    for(int i=0; i<results.length;i++) 
    { 
     var terrains= results.features[i].attributes.Terrain_Id; 
     var rawDataOfTerrain=getjsondata("http://or.org"); 

     var trsetdata= jQuery.parseJSON(tersetjson).d.results; 

     crops.push(trsetdata); 
    } 

     var cropsSoFar= []; 

     for(int i=0; i<crops.length;i++) 
     { 
      var crop= crops[i]; 
      if(crop in cropsSoFar) 
      { 
       crops.pop(crop); 
      } 
      cropsSoFar.push(crops[i]); 

     } 

     if(crops.length!=1) 
      alert("Please select terrains with the same crop"); 

これは配列に文字列を検索するための正しい方法ですか？もしそうなら、より簡単な方法がありますか？そうでない場合は、どこが間違っていますか？

Answer 1

cropsの最初の作成時に重複を確認するには、indexOfを使用します。

var crops = []; 

for(int i = 0; i < results.length; i++) { 
    var terrains = results.features[i].attributes.Terrain_Id; 
    var rawDataOfTerrain =getjsondata("http://or.org"); 
    var trsetdata = $.parseJSON(tersetjson).d.results; 

    if(crops.indexOf(trsetdata) === -1) 
     crops.push(trsetdata); 
} 

if(crops.length != 1) 
     alert("Please select terrains with the same crop"); 

Answer 2

docに表示されるようArray.prototype.indexOfを使用するよりも、あなたの配列は、単純な文字列配列は、ある場合。

function getMyStrings(value) { 
    return value.match(/m/); 
} 

var filtered = ["my", "simple", "strings", "array"].filter(getMyStrings); 

の場合：

var list = ["my", "simple", "strings", "array"]; 
var founds = []; 
var i = list.indexOf("simple"); 

if (i != -1) 
    founds.push(list[i]); 

あなたが何をしたいのか、代わりに、リストをフィルタリングしている場合、あなたはArray.prototype.filterを使用する必要があり、それはあなたにパラメータと戻り、新しい配列をコールバック関数を取得します1つの結果しかない場合は、findメソッドを使用できます。コールバックを受け取りますが、値は1つだけです。

コールバック関数を渡す利点は、単純な文字列ではなく複雑なオブジェクトを簡単に処理できることです。