Почему код моей следующей кнопки зацикливает слайд-шоу, а предыдущая - нет?

Question

Я успешно закодировал кнопки для автоматического продвижения слайд-шоу.Обе кнопки работают, но только следующая кнопка способна перебрать все изображения.Когда я начинаю с предыдущей кнопки, вся функция ломается, и ошибки говорят, что она не может найти неактивные или активные элементы.

Чего мне не хватает?

document.getElementById('next1').onclick = function() {	

	var lon = items.length;
	items[ptr].className= "inactive";

	if(ptr==lon-1){
		ptr=0;
	}
	ptr++;

	items[ptr].className = "active";
}

document.getElementById('previous1').onclick = function() {

	var lon = items.length;
	var p = ptr-1;

	items[ptr].className= "inactive";
		if(ptr==lon){
		ptr=items.length;
	}
	ptr--;
	items[ptr].className = "active";
}

Answer 1

Я нашел это.Это:

document.getElementById('next1').onclick = function() {	
	var lon = items.length;
	//alert(lon);
	items[ptr].className= "inactive";
	if(ptr==lon-1){
		ptr=0;
	}
	ptr++;
	items[ptr].className = "active";
}
document.getElementById('previous1').onclick = function() {	
	var lon = items.length;
	var p = (ptr-1);
	if(p<0){
		p += items.length;
	}
	
	items[ptr].className= "inactive";
	if(ptr==p){
		ptr=items.length;
	}
	ptr = p;
	
	items[ptr].className = "active";
}