2016-07-17 5 views
0
ので、私は本当の基本的な文字列生成器を持って

JavaScriptの文字列ジェネレータ

:)をしてください判断してはいけないが、それはそれは、それはプリントに私が望むように動作していない私はJavaScriptに新たなんだ

1つの文字列(例えば「C」)のではなくあなたがforループの外で変数newsを宣言して初期化する必要があり、マルチ文字列

var alphabet = "abcdefghijklmnopqrstuvwxyz" 

for (var i=0;i<alphabet.length;i++) { 
    var news = ""; 
    news = news + alphabet[Math.floor(Math.random() * alphabet.length)] 
    if (i == alphabet.length - 1) { 
     console.log(news) 
    } 
} 
+1

ここに投稿する際は、コードを正しくインデントしてください。 – jfriend00

答えて

2

varを使って宣言することは問題ありません。なぜなら、それが上に吊り下げられるからです。しかし初期化は重要です。それはforループの外になければなりません。

var alphabet="abcdefghijklmnopqrstuvwxyz"; 
var news=""; 

for (var i=0;i<alphabet.length;i++) { 
    news=news+alphabet[Math.floor(Math.random()*alphabet.length)] 
    if (i==alphabet.length-1) { 
    console.log(news) 
    } 
} 
0

マイナーミス。ループの外側にvar news = '';を定義するだけです。 (また、セミコロンを使用します)。

var alphabet="abcdefghijklmnopqrstuvwxyz"; 
var news=""; 
for (var i=0;i<alphabet.length;i++) { 
    news=news+alphabet[Math.floor(Math.random()*alphabet.length)] 
    if (i==alphabet.length-1) { 
    console.log(news) 
    } 
}