javascriptのfor文を使用して思ったような処理がされません

Question

javascriptのfor文についてお願いします。
for文を使用して１００回１を足して、３の倍数ときだけ１を足さないという処理をしようとしています。
あくまでもfor文のみを使用して計算結果をだそうとしていますが、うまくいきません。よろしくお願いします。

<p onclick = "suji100();", id = "suji100">100回1加算、３の倍数のときだけ１を足さない</p>

function suji100(){
var suji = document.getElementById("suji100");
var a = 0;

for(var num = 0; num < 100; num += 1){
	a += 1;
for(var num = 0; num < 100; num += 3){
	a -= 1;
	}
	suji.addEventListener("click", function(){
	suji.innerHTML = "<p>100回１を足すが３の倍数の場合は１を足さない</p><p>答え： " + a + " </p>";
	}, false);
	}
	}

答え： -33

Comments

[kei344]

コードはコードブロックで囲んでいただけませんか？ ```（バッククオート3つ）で囲み、前後に改行をいれるか、コードを選択して「<code>」ボタンを押すとコードブロックになります。

Answer 1

for文を使用して１００回１を足して、３の倍数ときだけ１を足さないという処理をしようとしています。

よく考えて見てくださいな。

ループ1回目 → 1
ループ2回目 → 2
ループ3回目 → 3

足していないので、次も3
以降ずっと3...

ループが終わらない。

書いてある命題と、コードに書かれている内容と矛盾しているので、一体何を求めようとしているのか、読み取れません。

Comments

[busshi77]

ご回答ありがとうございます。まだまだ勉強不足なところが多いのでもう一度みなおさせていただきます。

Answer 2

<!--                     ↓ 「,」は不要（文法に無い） -->
<p onclick = "suji100();", id = "suji100">100回1加算、３の倍数のときだけ１を足さない</p>
<!-- ↑addEventListener使うなら不要 -->

function suji100() {
    var suji = document.getElementById("suji100");
    var a = 0;

    for (var num = 0; num < 100; num += 1) {
        a += 1;
        //   ↓ var は function がスコープになるのでこれは上で定義したものを上書きしている
        for (var num = 0; num < 100; num += 3) { // 3の倍数だけマイナスする（）
            a -= 1;
        }
        // for の中でaddEventListenerをすると for が回るだけイベントを追加する。今回は全部上書きなので「見えない」が、もう少し書き方を考えるべき。
        suji.addEventListener("click", function() {
            suji.innerHTML = "<p>100回１を足すが３の倍数の場合は１を足さない</p><p>答え： " + a + " </p>";
        }, false);
    }
}

例えばこうする↓

var suji = document.getElementById( 'suji100' );

function suji100() {
    var a = 0;
    var num;
    for ( num = 0; num < 100 ; num++ ) {
        if ( num % 3 === 0 ) { continue; }
        a += 1;
    }
    return a;
}

suji.addEventListener( 'click', function() {
    suji.innerHTML = '<p>100回１を足すが３の倍数の場合は１を足さない</p><p>答え： ' + suji100() + ' </p>';
}, false );
```**動くサンプル：**[https://jsfiddle.net/67sq5a2r/](https://jsfiddle.net/67sq5a2r/)

Comments

[busshi77]

ありがとうございます！参考にさせていただきます！

[kei344]

情報の追加・修正の依頼も出しているので、確認しておいてください。

Answer 3

同じnumについて二重ループになっているので、意味不明のプログラムになっています。

末尾の}を、a+=1;の直後に移動させて、ループを独立させてください。

Comments

[busshi77]

ご回答ありがとうございます。試してみます。

[otn]

「試してみます」じゃなくて、試してからコメントしてください。