2015-05-14 22 views
1

我使用jQuery的試圖將文本分配到數獨謎題(用表進行),並且可以使用下面的代碼通過indvidual細胞做到這一點:我試圖讓簡單的jQuery代碼以迭代

$('#popSud').click(function(){ 
$('#1').text('1'); 
}); 

但我想將它應用於整個81細胞的表,我試圖下面的代碼,但沒有成功:

function popSud(sudName) { 
    for (var i = 0; i<81; i++) { 
    $('#i').text(sudName[i-1]); 
    } 
} 
$('#popSud').click(function(){ 
    popSud(easy); 
    }); 

我設法讓我所有的錯誤使用開發工具,所以我擊中牆壁消失。我正在使用81數組作爲代碼。

回答

1

這部分不工作:

$('#i').text(sudName[i-1]); 

jQuery是字面上尋找元素ID爲「我」,而不是用數字,你正在尋找。

試試這個代碼:

$('#' + i).text(sudName[i-1]); 

注意i從0開始計數,所以你的元素ID必須從0開始爲好。看起來像現在你的代碼中會有幾個錯誤的代碼。

除此之外,我建議重新考慮你給你的函數和變量的名稱。現在,您可能知道popSud的含義,但是如果您必須在6個月內再次觸摸代碼?可能不是那麼多。你爲什麼不叫它populateSudoku?考慮到幾乎所有現代開發環境都具有代碼完成功能,額外的可讀性完全免費!

+0

感謝您的回答是最有幫助的。我將採取建議考慮 – justin

+0

不客氣。 – Timo