質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.37%

javascriptというか初歩的なプログラミング問題

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 721

Alpha

score 41

javascriptで1000から1000までの整数を昇順に出力しかつ20個表示した後に改行させていくプログラムを作っているのですが、iで10000までとおいているのですが実行結果で10200まで表示されてしまいます。

var i;
  var sum = 0; 
  var j;
  for(i=1000; i <=10000; i+=20){ 
    for(j=0; j <=20; j++){
    document.write(" ",i);
    sum = i+=20
      }
            document.write("<br>");
  }

おそらく変数の置き方が間違っているのだと思うのですが何処か分かりません。
ご指摘、回答の方よろしくお願いします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

+2

参考にされている教材が非常に古そうに見えるので,一度こちらに目を通されることをおすすめします.JavaScriptは最近特に変遷の激しい言語なので,5年以上前の情報は全く役にたたないと考えてもいいぐらいです.

というわけで,現代的な書き方に直します.
(HTML5では文字コードとタイトルの指定は必須だが,<html> <head> <body> の省略は可能)

<!DOCTYPE html>
<meta charset="UTF-8">
<title>Example</title>
<div id="content"></div>
<script>
'use strict';
addEventListener('DOMContentLoaded', function () { // HTML構造を読み込み終わった時に実行
    var content = document.querySelector('#content'); // id="content"のdivを取得
    function write(text) {
        content.insertAdjacentHTML('beforeend', text); // contentの末尾にテキストを追加
    }
    /* ここに上で作ったwrite関数を使って処理を書く */
});
</script>

さて,既存のコードの問題点ですが…

  • そもそも1行の個数が20になっていない(21になっている)
  • 内部ループでiに対して操作を加えており,行の頭の数字に対しては外部ループでの加算処理も含めると実質40が加算されている
  • iが10000になったときにも加算処理が行われるので10000を超えてしまう

など,直すべきところがかなり多いです.一例としては以下のようになるでしょうか.

<!DOCTYPE html>
<meta charset="UTF-8">
<title>Example</title>
<div id="content"></div>
<script>
'use strict';
addEventListener('DOMContentLoaded', function () {
    var content = document.querySelector('#content');
    function write(text) {
        content.insertAdjacentHTML('beforeend', text);
    }
    var from = 1000;
    var to = 10000;
    var span = 20;
    for (var i = 0; i <= (to - from) / span; ++i) {
        write(' ' + (from + i * span));
        if (i % span === span - 1) {
            write('<br>');
        }
    }
});
</script>

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/06/09 13:46

    ループを書くときは

    - カウンタは0番からはじめる
    - カウンタに加算する値は1にする
    - 終了条件の比較には「<」を使う

    が原則です.但し今回は表示したい整数が1000〜10000を20個単位に分割した,

    「10000を含まない450個」

    ではなく

    「10000も含む451個」であり,

    (10000 - 1000) / 20

    の値より1多くなっているので,1回余分にループするために「<=」を採用しています.

    キャンセル

  • 2016/06/09 17:09

    参考にしている教材は一応四年前のものです。
    参考にさせていただいたものがまったく違うものとは言えませんが根本から少し違うもので多少理解に苦しみましたが、実行結果としては求めていたものになったので助かりました。

    キャンセル

+1

iのfor文の条件をよく見てください。
iが10000「以下」という条件なので、10000ちょうどになっても止まらず、
iが10020になって初めてiのループは終わります。
iが10000のとき、中のjのfor文が動いてしまいます。
10000で止めたいのであれば、iは9980で止まってほしいはずです。
iのforの条件文を変えてみると良いでしょう。条件文は考えてみてください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/06/09 17:06

    forの条件文を(i=1000; i <= 10000; )にすると10080になってしまいました。
    やはりほかの条件が思いつきません。

    キャンセル

0

プログラムに無理があるように思えますが、それをベースに修正するならこのような感じになると思います。

var i;
var sum = 0; 
var j;
for(i=1000; i < 10000; ){ 
    for(j=0; j < 20; j++){
        document.write(" ",i);
        sum = i++;
    }
    document.write("<br>");
}

まずはご自分で何が変わったか、何が問題だったかを確認してみてください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/06/09 17:07

    参考にさせていただいたのですがやはりiの条件文が違うのかうまくいきませんでした。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.37%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る