Witam, uczę się programowania bardzo krótki okres czasu, postanowiłem zacząć uczyć się JS i jest to mój pierwszy skrypt (obejrzałem odcinek od MZ a później sam kodowałem całość) i prosiłbym was o ocenę kodu.
Na pewno wiele osób napisze mi, że jest on do poprawy, ale mam pytanie, czy to w jaki sposób napisałem go jest jakimś tragicznym błędem, niewybaczalnym? Czy od biedy ujdzie? Co jest ewentualnie do poprawy? (prosiłbym o jakieś dokładne objaśnienie lub zalinkowanie) :) Z góry dziękuję.
var numer = 1;
var timer;
function dodaj()
{
clearTimeout(timer);
if (numer>4) numer = 1;
numer++;
var plik = "<img src=\"images/" + numer + ".png\" />" + "<span class=\"left\" onclick=\"odejmij()\"></span>" + "<span class=\"right\" onclick=\"dodaj()\"></span>";
document.getElementById("slider").innerHTML = plik;
numer--;
zmienslajd();
}
function odejmij()
{
clearTimeout(timer);
numer--;
var plik = "<img src=\"images/" + numer + ".png\" />" + "<span class=\"left\" onclick=\"odejmij()\"></span>" + "<span class=\"right\" onclick=\"dodaj()\"></span>";
document.getElementById("slider").innerHTML = plik;
numer--;
if (numer>4) numer = 1;
if (numer<1) numer = 4;
zmienslajd();
}
function zmienslajd()
{
if (numer>4) numer = 1;
var plik = "<img src=\"images/" + numer + ".png\" class=\"wow fadeIn animated\" data-wow-offset=\"10\" data-wow-duration=\"1s\" data-wow-delay=\"0.5s\" />" + "<span class=\"left\" onclick=\"odejmij()\"></span>" + "<span class=\"right\" onclick=\"dodaj()\"></span>";
document.getElementById("slider").innerHTML = plik;
numer++;
timer = setTimeout(zmienslajd, 5000);
}
<!DOCTYPE html>
<html lang="pl">
<head>
<script src="js/jquery-3.1.1.min.js"></script>
<link rel="stylesheet" href="css/animate.css"/>
<link rel="stylesheet" href="css/main.css"/>
<script src="js/slider.js"></script>
</head>
<body>
<div id="slider"></div>
<script>
$( document ).ready(function() { zmienslajd(); });
</script>
</body>
</html>