タイプエラー、未定義のプロパティ 'match'を読み取ることができません

Question

私は以下のJavascriptコードでスタックしていますが、何が問題なのかわかりません。 JavaScriptで

function keepletteronly(str) { 
    str=str.toLowerCase();//Make the string to lower case 
    arr=str.split(""); //This way I make an array out of the string 
    arr1=[]; //make an new array 
for (i=1; i<=arr.length;i++){ 
    if (arr[i].match(/[a-z]/)!=null) { //This is where the problem is 
     arr1.push(arr[i]); //retain only the letters and append to the arr1 
    } 
} 
newstring=arr1.join; 
return newstring; 


} 



keepletteronly("1eye"); 

Answer 1

配列は、最初の項目はインデックス0である意味、ベース0です： コードが常に表示される「タイプエラー、未定義のプロパティ 『一致』を読み取ることができない」、 はここのコードです。配列内に3つのアイテムがあるとしたら、プロパティは3と表示され、3つのアイテムはインデックス0,1および2を占めます。あなたのforループでは

は、あなたが（インデックス0で）最初の項目をスキップするループを意味し、インデックス1（i=1;）で開始し、iが下または配列の長さに等しい間、あなたは繰り返します。配列の長さは3ですが、基数0のため配列の最後の項目はインデックス2になります。だからあなたのforループの最後の反復では、undefinedという配列のインデックス3にアクセスしようとします。あなたが適切0、1と2の代わり1、2と3

を反復します

(i = 0; i < arr.length; i++) 

この方法は：

だから、何をする必要があると、このように見えるためにあなたのforループ式を変更することです

Answer 2

JS配列のインデックスはゼロから始まります。

newstring=arr1.join; 

それは

newstring=arr1.join(''); 

または単にARR1せずに結果の文字列を生成する必要があり、同様に： あなたはまた、別の文も間違ってい

for (i = 0; i < arr.length; i += 1) { 

に文のためにあなたを変更する必要があります：

newstring=''; 
... 
newstring+=arr[i];