2016-10-31 19 views
0

すべての行は常に同じ番号を持ちます、なぜですか?彼らはそれを設定するために乱数を取るべきです。また、どうすれば修復できますか?一度私はここで答えを見たが、今私はそれを見つけることができません。2次元配列ランダムジェネレータjQuery

var mapSizex = 5; 
 
var mapSizey = 6; 
 
var mapArray = []; 
 

 
$(function() { 
 
    console.log("ready!"); 
 
    $('#map-draw').html(drawMap()); 
 
}); 
 

 
function mapGenerator() { 
 
    for (i = 0; i < mapSizex; i++) { 
 
    for (x = 0; x < mapSizey; x++) { 
 
     mapArray[i, x] = getRandom(1, 5); 
 
    } 
 
    } 
 
} 
 

 
function drawMap() { 
 
    mapGenerator(); 
 
    var map = ''; 
 
    tileID = 0; 
 
    for (i = 0; i < mapSizex; i++) { 
 
    map = map + '<br style="clear: both;">'; 
 
    for (x = 0; x < mapSizey; x++) { 
 
     map = map + '<div class="tile tileID' + tileID + '">' + mapArray[i, x] + '</div>'; 
 
     tileID++; 
 
    } 
 
    } 
 
    return map; 
 
} 
 

 
function getRandom(min, max) { 
 
    var x = Math.floor((Math.random() * max) + min); 
 
    return x; 
 
}
.tile { 
 
    float: left; 
 
    height: 20px; 
 
    width: 20px; 
 
    border: 1px solid black; 
 
    text-align: center; 
 
}
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script> 
 
<div id="main-container"> 
 
    <div id="map-container"> 
 
    <div id="map-draw"></div> 
 
    </div> 
 
</div>

それを投稿するには、私はいくつかのより多くのコンテンツを追加する必要がありますが、すべては情報の私が何を意味するかを見つけるために十分でなければなりません含まれています。ここ

+4

次元配列のドント作業:

はここにあなたの固定コードです。 mapArray [i、x]は無効です。mapArray [i] [x] – juvian

+0

[JavaScriptで2次元配列を作成するにはどうすればいいですか?](http://stackoverflow.com/questions/966225/how – juvian

+3

追加の注意として、[comma operator](https:// developerの有効な使用であるため、構文はエラーをスローしません。 .mozilla.org/ja/docs/Web/JavaScript/Reference/Operators/Comma_Operator)、結果を説明しています。 – ASDFGerte

答えて

2

は、それはmapArray[i][x]あるべきと私は外側のループでmapArray[i]=[];を追加JSFIDDLEウォーキングコード

へのリンクです。そのような

var mapSizex=5; 
var mapSizey=6; 
var mapArray=[]; 

$(function() { 
    console.log("ready!"); 
    $('#map-draw').html(drawMap()); 
}); 

function mapGenerator(){ 
    for(i=0;i<mapSizex;i++){ 
    mapArray[i]=[]; 
     for(x=0;x<mapSizey;x++){ 
      mapArray[i][x]= getRandom(1,5); 
     console.log(i,x,getRandom(1,5)) 
     } 
    } 
} 
function drawMap(){ 
    mapGenerator(); 
    console.log(mapArray) 
    var map=''; 
    tileID=0; 
    for(i=0;i<mapSizex;i++){ 
     map=map+'<br style="clear: both;">'; 
     for(x=0;x<mapSizey;x++){ 
      map=map+'<div class="tile tileID'+tileID+'">'+mapArray[i][x]+'</div>'; 
      tileID++; 
     } 
    }return map; 
} 
function getRandom(min,max) { 
    var x = Math.floor((Math.random() * max) + min); 
    return x; 
} 
+0

すでに修理済みですが、ありがとうございます! ベストフォーラム: – BatOOn

+0

mapArray [i] = []; - >有用な:)知っておいてよかった。 – BatOOn