javascript - Javascript インデックス変数

Question

インデックス変数を操作して、配列値をインクリメントします。コンセプトは正しいと思いますが、JavaScript の動作を妨げている何かが構文にあると思います。

<html>
<head>
<style type="text/css">

 body {font-family: arial;}
 </style>
<script type="text/javascript">
function ChangeIt()
{
var colors;
colors = new Array("red", "blue", "green", "yellow", "purple");
i=0;
document.body.style.backgroundColor = colors[0];

var t = setInterval(function() {
    i=(i>=color.length) ? 0 : i+1;
    document.body.style.backgroundColor = colors[i];
}, 3000);
}
</script>
</head>
<body>
This page begins with a red background and
changes the body background to four other colors
after three seconds.<br />

The Javascript
function is set in header section and called
from the body.
</body>
<script type="text/javascript">
ChangeIt();
</script>
</body>
</html>

score 2 · Accepted Answer

Its simple , while setting i value you are using color instead of colors use,

 i=(i>=colors.length) ? 0 : i+1;

now it works fine , just checked

score 1 · Accepted Answer

1

Replace color.length with colors.length

于 2013-10-23T13:22:20.057 に答える

score 0 · Accepted Answer

貴重なブラウザコンソールを確認すると、次のように表示されます。

Uncaught ReferenceError: 色が定義されていません

colorの代わりに使用しているためですcolors。

コード：

function ChangeIt() {
    var colors;
    colors = new Array("red", "blue", "green", "yellow", "purple");
    i = 0;
    document.body.style.backgroundColor = colors[0];

    var t = setInterval(function () {
        i = (i >= colors.length) ? 0 : i + 1;
        document.body.style.backgroundColor = colors[i];
    }, 3000);
}

デモ: http://jsfiddle.net/IrvinDominin/wDC9r/

score 0 · Accepted Answer

主な間違いはcolor.length、確かにを意味するときに参照することですcolors.length。

ただし、コードに関する他の多くの点を改善することができます。

colors宣言するときに配列を割り当てます。
の代わりに配列リテラルを使用できますnew Array()。
変数を 1 行で宣言し、iローカルにしChangeIt()ますが、間隔関数のスコープで引き続き使用できるようにします。
「increment i」行を読みやすいものに変更します (たとえば、03 項ステートメントの「else」としてを割り当てます)。

変更点は次のとおりです。

function ChangeIt() {
     var colors = ["red", "blue", "green", "yellow", "purple"], 
         i = 0,  
         fn = function() {
             i = (i < colors.length) ? i + 1 : 0;
             document.body.style.backgroundColor = colors[i];
         },
         t = setInterval(fn, 3000);
     fn(); // To set the background immediately.
};

score 0 · Accepted Answer

あなたは「s」を忘れました

var t = setInterval(function() {
    i=(i>=color.length) ? 0 : i+1;
    document.body.style.backgroundColor = colors[i];
}, 3000);
}

ただ変える

i=(i>=color.length) ? 0 : i+1;

これに

i=(i>=colors.length) ? 0 : i+1;

.

javascript - Javascript インデックス変数

5 に答える 5

Related

Reference