• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Slider nie działa

VPS Starter Arubacloud
0 głosów
312 wizyt
pytanie zadane 8 lipca 2018 w JavaScript przez bicnet Gaduła (4,800 p.)

Cześć,

zrobiłem slider w JS , ale co nie do końca działa tak jak chciałem.

Działa: samo zmienianie się obrazków, jak skończą się obrazki to powraca do pierwszego, kropki na dole działają.

Nie działa: strzałka lewo/prawo, usuwanie backgroundu nieaktualnej kropki, po wczytaniu się strony przez 3s pierwszy obrazek się nie wczytuje w ogóle, a potem jest już normalnie.

<div class="slider">
                 <img>
                 <div class="prev"><span class="fas fa-chevron-circle-left"></span></div>
                 <div class="next"><span class="fas fa-chevron-circle-right"></span></div>
                  <div class="dots">
                        <span></span>
                        <span></span>
                        <span></span>
                  </div>
 </div>
let i = 0;
const img = [ 'img/logo.png', 'img/slider1.jpg', 'img/slider2.jpg' ];

let dots = document.querySelectorAll(".dots span");
dots = [... dots];

const prev = document.querySelector(".prev");
const next = document.querySelector(".next");

const slide = document.querySelector(".slider img");


const changeImg = () =>
{
    slide.src = img[i];

    prev.addEventListener("click", () =>
     {
        i--;
        if(i < 0) { i = 0; slide.src = img[0]; }
        else {slide.src = img[i] };
     });

    next.addEventListener("click", () =>  i++ );

    dots[0].addEventListener("click", () => slide.src = img[0] )
    dots[1].addEventListener("click", () => slide.src = img[1] )
    dots[2].addEventListener("click", () => slide.src = img[2] )

    dots[i].style.background = "gray";


     if( img.length - 1  > i )   i++;
     else i = 0;
}

setInterval( () =>
{
   changeImg();
}, 3000)
  .slider
        {
           width: 100%;
            height: 75%;
           display: flex;
           justify-content: center;
           position: relative;

            img:nth-child(1)
            {
               width: 100%;
               height: 25vh;
            }

               .prev
              {
                  width: 15%;
                  height: 30%;
                  position: absolute;
                  top: 40%;
                  left: 0;
                  opacity: 0.5;
                       .fa-chevron-circle-left
                       {
                          width: 95%;
                          height: 95%;
                       }
              }

              .next
              {
                    width: 15%;
                    height: 30%;
                    position: absolute;
                    top: 40%;
                    right: 0;
                    opacity: 0.5;
                        .fa-chevron-circle-right
                        {
                            width: 95%;
                            height: 95%;
                        }
              }

              .fa-chevron-circle-left:hover , .fa-chevron-circle-right:hover
              {
                 opacity: 1;
              }

              .dots
              {
                 width: 100%;
                 height: 15%;
                 display: flex;
                 justify-content: center;
                 align-items: center;
                 position: absolute;
                 top: 80%;
                  span
                  {
                       width: 25px;
                       height: 25px;
                       padding: 0 5px 0 5px;
                       border: 1px solid black;
                       border-radius: 50%;
                  }
              }

 

Pozdrawiam

1 odpowiedź

0 głosów
odpowiedź 8 lipca 2018 przez Tomek Sochacki Ekspert (227,510 p.)
next.addEventListener("click", () =>  i++ );

w tej linijce po zdarzeniu click z next ty w calback zwracasz wartość i, a nie modyfikujesz wartość zmiennej "i". Jest ona powyżej zadeklarowana jako typ prosty:

let i = 0; // czyli i: number

więc przekazujesz ją przez wartość, a nie przez referencję. Pewnie chodziło Ci o to, aby modyfikować jej wartość, więc zrób:

next.addEventListener("click", () => {  i++ } );

albo w pełni jawnie:

next.addEventListener("click", () => {
    i++;
    return undefined;
});

 

komentarz 8 lipca 2018 przez bicnet Gaduła (4,800 p.)
Zrobiłem tak, no ale nie działa dalej, nic się nie zmieniło. dalej jak klikam next to robi się src="undefined" (mam w js return undefined).
komentarz 8 lipca 2018 przez Tomek Sochacki Ekspert (227,510 p.)
a dlaczego obsługę tego zdarzenia click na next masz w ogóle wewnątrz changeImg(), która odpala się cyklicznie co ok. 3000ms?
komentarz 8 lipca 2018 przez bicnet Gaduła (4,800 p.)
A no faktycznie , czyli to trzeba wyrzucić będzie poza changeImg(), zrobiłem tak ale dalej nic.
komentarz 9 lipca 2018 przez bicnet Gaduła (4,800 p.)

Poprawiłem trochę dzisiaj ile złapałem wolnego czasu i kod wygląda już tak:

let i = 0;
const img = [ 'img/logo.png', 'img/slider1.jpg', 'img/slider2.jpg' ];

let dots = document.querySelectorAll(".dots span");
dots = [... dots];

const prev = document.querySelector(".prev");
const next = document.querySelector(".next");

const slide = document.querySelector(".slider img");


const changeImg = () =>
{
     slide.src = img[i];
     if( img.length - 1  > i )   i++;
     else i = 0;
}

const changeDot = () =>
{
   dots.forEach(  (e) => e.style.background = "transparent");
   dots[i].style.background = "gray";
}

    next.addEventListener("click", () =>
    {
    i++;
     if( i > img.length- 1 ) return;
    });

    prev.addEventListener("click", () =>
    {
      i--;
      if(i < 0) { i = 0; slide.src = img[0]; }
      else {slide.src = img[i] };
    });

    setTimeout( () =>
    {
        dots[0].addEventListener("click", () => {slide.src = img[0]; i = 0;} )
        dots[1].addEventListener("click", () => {slide.src = img[1]; i = 1;} )
        dots[2].addEventListener("click", () => {slide.src = img[2]; i = 2;} )
    }, 1000)


changeImg();
dots[0].style.background = "gray";


setInterval( () =>
{
   changeDot();
   changeImg();
}, 1000)

Zmiany: pierwsze zdjęcie po załadowaniu strony już się pojawia, działają kropki + aktualna zdjęcie ma swoją kropkę.

Nie działa dalej: arrow-left, arrow-right

komentarz 9 lipca 2018 przez Tomek Sochacki Ekspert (227,510 p.)
next.addEventListener("click", () => {
    i++;
    if( i > img.length- 1 ) return;
});

W tym miejscu po kliknięciu w next robisz dwie rzeczy - inkrementujesz "i" oraz ZAWSZE zwracasz undefined. Otóż jeśli i > img.length - 1 to zwracasz undefined, co robi "return", natomiast jeśli warunek nie jest spełniony to zwracasz niejawnie "return undefined".

Czyli... jak dobrze się zastanowisz to zobaczysz, że po za tą zmianą wartości "i" generalnie nic się w tym kodzie nie dzieje i można by to zapisać jako:

next.addEventListener("click", () => { i++; });

No i teraz co... otóż nic, zwiększyliśmy "i" i jest super, ale nic z tym "i" nie robimy :) Musiałyś teraz podmienić odpowiednio src obrazka. Spróbuj np. wewnątrz callback obsługującego kliknięcie w "next" po inkrementacji zmiennej "I" wywołać funkcję changeImg(), np:

next.addEventListener("click", () => { 
    i++; 
    changeImg();
});

I daj znać czy działanie się poprawiło, a jeśli tak to jak będziesz chciał zajmiemy się małą refaktoryzacją tego kodu, bo jest on troszkę kiepski w dalszym utrzymaniu, ale to na potem zostawmy, na razie dojdźmy do celu, czyli do pełnego działania :)

komentarz 9 lipca 2018 przez bicnet Gaduła (4,800 p.)
edycja 10 lipca 2018 przez bicnet
Dodałem tak, ale działa to: gdy klikam next z img[0] to wskakuje mi na img[2], a z img[2] na img [1] a z img [1] na nie znaleziono ścieżki, czyli tak jakby dodaje nam po 2 do [i] zamiast 1?

 

A prev: img[0] + prev = nic,  img[1] + prev  = nic, img[2] + prev = nie znaleziono ścieżki

 

@edit

W next dałem zamiast i++ , to i = i , jestem zaskoczony, ale to zadziałało, jednak nie wiem jak to działa więc mnie nie satysfakcjonuje, a do tego prev nie działa na takiej zasadzie ( no nie wpisze sobie i = i bo to będzie takie samo jak next, a i-- nie działa).

Za to testowałem z kropkami to każda kropka ma zdjęcie +1 (czyli do 1 zdj. jest kropka do 2, a kropka jest jak licznik , to coś jest nie tak z tym licznikiem)
komentarz 11 lipca 2018 przez bicnet Gaduła (4,800 p.)

@Tomek Sochacki,

da rady to jakoś uratować, żeby działało, czy muszę przerobić cały kod prawie od zera?

komentarz 12 lipca 2018 przez Tomek Sochacki Ekspert (227,510 p.)

A próbowałeś w ogóle debugować swój kod...? Chyba nie :)

Zobaczmy co masz w kodzie:

const prev = document.querySelector(".prev");

a teraz piszesz, że potem gdzieś dodajesz img[0] + prev. Pytanie więc co Ci siedzi w tych zmiennych, bo z kodu ja że w prev siedzi obiekt (referencja DOM), a w zmiennej img[0] siedzi ciąg znakowy stanowiący ścieżkę do pliku.

Nie wiem więc w końcu jaki jest Twój ostateczny kod, bo z tego który ostatni dałeś to jak piszę, w prev jest obiekt a nie żaden ciąg znakowy więc ich konkatenacja nie ma żadnego sensu...

Najlepiej takie przykłady dawaj jako codepen tylko pamiętaj, że jeśli pobierasz jakieś zdjęcia lokalnie u siebie to na codepen daj jakieś ścieżki z http... albo wprowadź testowo grafiki jako base64 itp. Tak, żeby można było je zobaczyć :)

komentarz 12 lipca 2018 przez bicnet Gaduła (4,800 p.)
Tak chyba będzie najszybciej:

https://codepen.io/anon/pen/djoMPM
komentarz 12 lipca 2018 przez Tomek Sochacki Ekspert (227,510 p.)
Straszny bałagan jest w tym kodzie i szczerze to nie chce mi się go całego analizować. Tu masz przykład na szybko prostego slidera prev-next:

https://codepen.io/anon/pen/gjpgGE?editors=1010

Kod daleki od ideału, nawet bardzo daleki, ale chodziło mi o pokazanie jakiegoś jednego z przykładów na analizę tego indeksu obrazka.

Spróbuj pokombinować.
komentarz 12 lipca 2018 przez bicnet Gaduła (4,800 p.)
Okej, coś porobię żeby działało. Dzięki za pomoc.
komentarz 15 lipca 2018 przez bicnet Gaduła (4,800 p.)

@Tomek Sochacki,

śmieszna historia, bo zacząłem szukać różnych poradników na Youtubie, co by mi odpowiadało, zazwyczaj albo były kropki, albo strzałki, albo dość stara składnia, więc po dwóch godzinach irytacji (bo zaczynałem coś z jednego poradnika, później, że jednak tych strzałek nie zgram z tym drugim poradnikiem z kropkami, zacząłem myśleć co faktycznie jest nie tak z moim sliderem, odpowiedź okazała się bardzo prosta: licznik był nie w tej linijce co trzeba było, dlatego za każdym razem dodawało mi po 2 zamiast po 1. Wrzucę Ci kod, fajnie jakbyś napisał czy można takim sliderem pochwalić się na githuba czy jest beznadziejny i lepiej sobie odpuścić.

 

https://codepen.io/anon/pen/qybOvV

 

Podobne pytania

0 głosów
0 odpowiedzi 125 wizyt
pytanie zadane 10 marca 2019 w JavaScript przez Sobol3k Użytkownik (690 p.)
0 głosów
1 odpowiedź 384 wizyt
pytanie zadane 2 lipca 2017 w JavaScript przez sempiordlem Nowicjusz (210 p.)
0 głosów
1 odpowiedź 263 wizyt
pytanie zadane 26 września 2016 w JavaScript przez agnieshax Nowicjusz (120 p.)

92,452 zapytań

141,262 odpowiedzi

319,078 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...