2016-10-13 8 views
0

私はcsv内の各項目の出現回数をカウントしようとしています。私のCSVは次のようなものです:else javascriptループが機能しない場合

date    item 
8/10/2015  mobile 
1/25/2015  mobile 
10/28/2015  mobile 
11/9/2015  mobile 
8/2/2015  mobile 
9/24/2015 0:00 laptop 
9/30/2015 0:00 laptop 
5/15/2015 0:00 laptop 
7/7/2015 0:00 laptop 

私は出現を比較するために他に使用しています。しかし、それは文字列を比較していないだけで、カウンタをインクリメントしています。

私のコードは次のとおりです。

<div> 
    <select id="mySelect" onchange="onChange()"> 
     <option value="2016">2016</option> 
     <option value="2015">2015</option> 
     <option value="2014">2014</option> 
    </select> 
    </div> 
<script> 
    function onChange(){ 
     var selectobject = document.getElementById("mySelect"); 
     var MBct = 0; 
     var LPct = 0; 
     d3.csv("item2.csv", function(error,data) { 
      data.forEach(function(d) { 
       var date = d.date; 
       var item = d.item; 
       a = d.date.split(" ",1); 
       b=a[0].split("/"); 
       d.year = b[2]*1; 
       console.log("MBct:"+MBct); 
       if(selectobject.value == d.year){ 
        console.log("Entered if loop:selectobject.value="+selectobject.value+" ,d.year="+d.year); 
        console.log("item:"+item); 


    if(d.item === "mobile"){ 
        MBct = MBct + 1; 
        console.log("MBct:"+MBct); 
       } 
       else if(d.item === "laptop"){ 
        LPct = LPct + 1; 
        console.log("LPct:"+LPct); 
       } 
       else console.log("not present"); 
    })})} 
</script> 

私の出力は次のようにする必要があります:

MBct:5 , LPct:4 

それは次のように来ている:MBct:9、LPct:0

誰でも任意のアイデア? ? '

+2

Urファンクション閉じ括弧がありません –

+1

あなたはクロムインスペクタまたは同様のツールでデバッグしようとしましたか? – snit80

+1

あなたのコードを適切にインデントしていれば、たくさんの閉じ括弧が見当たらないでしょう。私はそれにも驚いています。 – vlaz

答えて

0

あなたはd3.csv(function(error) {、最も重要なdata.forEach(function(d) {括弧が閉じていません。

EDIT: jsfiddleでyoureのジャバスクリプトを参照してください:https://jsfiddle.net/HollyPony/my5rcqkr/1/

Youreのブラケットが...

Protipsに一致していません:あなたは1を開いたときに自動近いブラケットにIDEを使用してください。または古い学校の方法は、手動であなたのブラケットを閉じ始めてから書き込みます。

+0

これは既に私のコードで閉じています.. – JGS

+0

jsTesterでテスト済み:www.webtoolkitonline.com/javascript-tester.html 無効です。 jsfiddle:https://jsfiddle.net/HollyPony/my5rcqkr/1/でテストされています。バッチ閉鎖ブラケット。 私の手によってテストされた...同じ結果: – HollyPony

関連する問題