2016-07-13 8 views
0

アップロードされたテキストファイルを読み込んで別のオプションとして並べ替えるコードがあります。<select>私はforループを使用しているので、file.lengthよりも小さく設定しています。テキストファイルのすべてのオプションは正しく読み込まれますが、テキストが完成すると残りのオプションはundefinedと表示されます。私はこれらを取り除く方法を知りたかったのですundefined?混乱している場合は、以下のコメントを追加してください。選択肢の中で定義されていないものを取り除くこと

+0

にあなたの3つのループを組み合わせますか?内部を1つ増やしてください。 – epascarello

+0

あなたのオプションが正しくない、valの代わりにvalueを使用するべきです。値は各オプションごとに一意でなければなりません:http://www.w3schools.com/tags/tag_option.asp – SPlatten

+0

@SPlatten値は一意である必要はありません... – epascarello

答えて

1

から値を取得する必要があります。これをチェックしてくださいFiddle。これは、文の場合、私はループのために追加されたコンテンツを持つ.txtファイルArif, John, Sonya, SuperGirl, The Flash, Batman, Suparman

のために私のために働いた、と3つのループを行うのはなぜ1

for (var i = 0; i < results.length; i=i+3) { 
     $('#name').append('<option val="1">' + results[i] + '</option>'); 
     $('#name2').append('<option val="1">' + results[i] + '</option>'); 
     if(typeof results[i+1] !== 'undefined') { 
     $('#address').append('<option val="1">' + results[i+1] + '</option>'); 
     $('#address2').append('<option val="1">' + results[i+1] + '</option>'); 
     } 
     if(typeof results[i+2] !== 'undefined') { 
     $('#size').append('<option val="1">' + results[i+2] + '</option>'); 
     $('#size2').append('<option val="1">' + results[i+2] + '</option>'); 
     } 
    } 
+0

ありがとう、これはうまくいきました。 – Baner

0

あなたは

for (var i = 0; i < results.length; i=i+3) { 

を行う必要があります。また、私はあなたのコードを更新したファイル

$('#name').append('<option value="1">' + file[i] + '</option>'); 
関連する問題