В чем причина ошибки: Uncaught TypeError: Cannot read property '24' of undefined?

Question

[Искандер Мамедов]

Имеется двумерный массив field размерностью 25х25 и следующая функция:

function bug_start(){
	j=0;
	i=0;
	bugX=0;
	bugY=0;
	check:
	if(field[0][1]==1 && field[1][0]==1){
		alert('Жук заблокирован! Перестройте лабиринт!');
		break check;
	}
	else{
		while(bugX!=480 && bugY!=480){
			while((field[j][i+1]==0) && i<24){
				ctx.drawImage(bug, (i+1)*20, j*20);
				i+=1;
				bugX = bugX + i*20;
				score+=1;
			}
			while((field[j+1][i]==0) && j<24){
				ctx.drawImage(bug, i*20, (j+1)*20);
				j+=1;
				bugY = bugY + j*20;
				score+=1;
			}
			while((field[j][i-1]==0) && ((i-1)>0)){
				ctx.drawImage(bug, (i+1)*20, j*20);
				i-=1;
				bugX = bugX + i*20;
				score+=1;
			}
			while((field[j-1][i]==0) && ((j-1)>0)){
				ctx.drawImage(bug, (i+1)*20, j*20);
				j-=1;
				bugY = bugY + j*20;
				score+=1;
			}
		}
	}
	alert(score);
}

При вызове функция в браузере выходит ошибка: Uncaught TypeError: Cannot read property '24' of undefined на строке while((field[j+1][i]==0) && j<24){
Как ее исправить? Где ошибка в функции?

Answer 1

[Filipp Shcherbanich]

while(field[j+1]&&(field[j+1][i]==0) && j<24){

Comments

[Filipp Shcherbanich]

проблема в том, что в какой-то момент field[j+1] у тебя становился undefined. Вообще лучше бы по-другому всё организовать у тебя, ну да ладно. И так сойдёт)

[Filipp Shcherbanich]

совет - используй один цикл )

[Искандер Мамедов]

В общем этот код управляет жучком, который бегает по карте. Если в массиве число равно 0, то здесь жук может пройти, а если 1, то не может.
field[j+1]&&(field[j+1][i]==0 если изменить как ты сказал, то код работать не будет, массив то двумерный field[j+1]. Я вставил твой код, и программа повисла. Бесконечный цикл получился.

[Искандер Мамедов]

@scherbanich как именно?

[Filipp Shcherbanich]

@Gugr, я просто исправил твою ошибку, которую ты указал, логику не трогал)

Answer 2

[overmes]

все правильно, функция делает именно, то что должна - баг