Slider JS problem z nawgacją

pytanie zadane 2 maja 2015 w JavaScript przez makoso Mądrala (7,380 p.)
edycja 2 maja 2015 przez makoso

Mam mały problem po obejrzeniu kursu JS odc 2 chcę zrobić sobie slider wydziergałem prawie wszystko tylko przyciski dalej i wcześniej mi nie działają jak ktoś załapie mieszaninę w tym kodzie niech podpowie co jest nie tak :)

chodzi o funkcje dalej i wcześniej reszta działa prawidłowo podejrzeć można tutaj:
http://mebleeee.esy.es

<script src="http://code.jquery.com/jquery-1.11.2.min.js"></script>


<script type="text/javascript">


function _zmien()
{
     var kolekcja = document.getElementsByTagName('span');
     for(i=0;i<kolekcja.length;i++)
     {
           if( kolekcja[i].getAttribute('rel') == 'abc' )
          {
                  kolekcja[i].style.display = 'block';
           }
     }
}
function _zmien1()
{
     var kolekcja = document.getElementsByTagName('span');
     for(i=0;i<kolekcja.length;i++)
     {
           if( kolekcja[i].getAttribute('rel') == 'abc' )
          {
                  kolekcja[i].style.display = 'none';
           }
     }
}

var numer = Math.floor(Math.random()*5)+1;
var czas1 = 0;
var czas2 = 0;

function dalej()
{
	clearTimeOut(czas1);
	clearTimeOut(czas2);
	numer=numer +1;
	schowaj();
	setTimeout("zmienslajd", 500);	
}
function wczesniej()
{
	clearTimeOut(czas1);
	clearTimeOut(czas2);
	numer=numer -1;
	schowaj();
	setTimeout("zmienslajd", 500);	
}

function schowaj(){	
	$("#slider").fadeOut(500);
}
function zmienslajd()
{
	numer++;if (numer>5) numer=1;	
	var plik="<span class=\"n\" style=\"top:0;left:-22px;\" rel=\"abc\" onmouseover=\"_zmien()\" onmouseout=\"_zmien1()\" onclick=\"wczesniej()\"><</span><img src=\" " + numer +".jpg\" rel=\"abc\" onmouseover=\"_zmien()\"  onmouseout=\"_zmien1()\"/><span class=\"n\" style=\"top:0;right:30px;\" rel=\"abc\" onmouseover=\"_zmien()\"  onmouseout=\"_zmien1()\" onclick=\"dalej()\">></span>";	
	document.getElementById("slider").innerHTML = plik;
	$("#slider").fadeIn(500);
	czas1 = setTimeout("zmienslajd()", 5000);	
	czas2 = setTimeout("schowaj()", 4500);	
}
</script>
</head>
<body onload="zmienslajd()">



<div id="blok">
<div id="slider" >



</div>
</div>

2 odpowiedzi

odpowiedź 2 maja 2015 przez makoso Mądrala (7,380 p.)

Najlepsza

Prawidłowe rozwiąznie dzięki SyntaxError

function dalej()
{
    clearTimeout(czas1);
    clearTimeout(czas2);
    numer=numer ;
    schowaj();
    setTimeout(zmienslajd, 500);
}
function wczesniej()
{
    clearTimeout(czas1);
    clearTimeout(czas2);
    numer=numer--;
    schowaj();
    setTimeout(zmienslajd, 500);
}