Błąd w liczniku JavaScript

Question

     var numerD = 1;         //globalne zmienne ich wartości są zmieniane w funkcjach
     var back = numerD;
     var next = numerD;

     window.onload = podmianaDiva;

     function podmianaDiva()
       {
         alert(numerD);                                                                     //sprawdzenie zmiennej numerD
         var numerDiva = '<div id="div' + numerD + '"></div>';
         document.getElementById("content").innerHTML = numerDiva;
         if (numerD = 1) witaj();                                                           //wywołanie slidera (nie istotne)
         var buttonB = document.getElementById("arrow1"); 
//po kliknięciu w przycisk wywołanie funkcji
         buttonB.onclick = zmianaBack;
         var buttonN = document.getElementById("arrow2");
         buttonN.onclick = zmianaNext;

     }

     function zmianaBack()
     {
         back--;                    // odjęcie 1 od wartości numerD
         if (back <= 0) back = 5;   // jeżeli wartość jest za mała zacznij od końca
         numerD = back;             // zmienienie wartości numerD
         podmianaDiva();            //wywołanie funkcji startowej 
     }

     function zmianaNext() 
     {
         next++;                   //analogicznie jak wyżej
         if (next >= 6) next = 1;
         numerD = next;
         podmianaDiva();
     }

 

Witam tak wygląda fragment kodu źródłowego który napisałem (dalsza część to slider ale on działa dobrze) problem jest w tym że jak na przykład naciskam prawy przycisk (id="arrow1" zmianaNext)  to wartość zmienia się o +1 a gdy naciskam już lewy to wartość zamiast zmienić się o -1 czyli wrócić do początkowej zmienia się na losową (np 5,4,3) nigdy na tą co powinna analogicznie jest kiedy najpierw nacisnę lewy przycisk a później prawy. Ktoś widzi błąd, lub wie jak to naprawić? Pomocy, pierwszy raz pisze w js

Answer 1

Okropnie czyta się twój kod. Bezsensowne nazwy zmiennych i id. Rozumiem, że chodzi ci o zwiększanie i zmniejszanie tej zmiennej numerD. Raz nazywasz coś po polsku raz po angielsku. Bardzo ciężko zrozumieć co ten kod ma robić. Jeszcze ten okropny blok na kod wszystko zamazuje. Zerknij na mała podpowiedź na dole:

https://jsfiddle.net/q2m5z2cp/2/

Więc teraz wystarczy napisać funkcję podmień diva i wywoływać.

I podpowiedź na przyszłość: Używaj nawiasów klamrowych, ZAWSZE. Deklaruj zmienne na początku funkcji.

Answer 2

if (numerD = 1)
to na pewno jest źle : - )

Comments

usunąłem ten fragment kodu, nadal alert(numerD) zwraca złe liczby

---

dzieje się tak, ponieważ zmienne back i next nie mają nic ze sobą wspólnego. Strzelam, że jak dwa razy naciśniesz next to wyskoczy Ci 3 i gdy naciśniesz back to wyskoczy Ci 5 :)