Uaktualnianie zegara (nie ten z kursu) - problem

pytanie zadane 9 stycznia 2016 w JavaScript przez demotywatorking Obywatel (1,210 p.)

Chciałbym zrobić skrypt odświeżający zegar co sekundę. Niestety moja wiedza z JS i jQuery nieco kuleje, więc zwracam się o pomoc.

Napisałem sobie taki skrypt:

$(document).ready(function() {

  function aktualizacja()
  {
    var godziny = $('#godziny').val();
    var minuty = $('#minuty').val();
    var sekundy = $('#sekundy').val();
    sekundy++;
    if(sekundy >= 60)
    {
      minuty++;
      sekundy = sekundy - 60;
    }
    if(minuty >= 60)
    {
      godziny++;
      minuty = minuty - 60;
    }
    if(godziny < 10) godziny = "0" + godziny;
    if(minuty < 10) minuty = "0" + minuty;
    if(sekundy < 10) sekundy = "0" + sekundy;
    $('#godziny').text(godziny);
    $('#minuty').text(minuty);
    $('#sekundy').text(sekundy);
    setTimeout("aktualizacja()", 1000);
  }

};)

W dokumencie na serwerze te dane wyglądają tak:

<span id="godziny">0</span>:<span id="minuty">24</span>:<span id="sekundy">17</span>

Niestety zegar się nie uaktualnia. Próbowałem już tam różne rzeczy zmieniać w kodzie, ale nadal nie mogę sobie z tym poradzić. Z góry dzięki za pomoc.

1 odpowiedź

odpowiedź 9 stycznia 2016 przez Shelvi96 Obywatel (1,440 p.)

Czy w tagu <body> dorzuciłeś onload="nazwa_funkcji_wywoływanej_przy_ładowaniu_strony"? Jeśli nie, to funkcja ta nie jest po prostu wywoływana, stąd brak aktualizacji.

komentarz 9 stycznia 2016 przez Shelvi96 Obywatel (1,440 p.)

https://youtu.be/OcwON22ctYc?t=30m42s - tutaj mówi o tym Pan Mirosław

komentarz 9 stycznia 2016 przez demotywatorking Obywatel (1,210 p.)

Byłem pewny, że jak wpiszę to w $(document).ready to się sama wykona, bo jak było o menu przyklejanym, to chyba nie trzeba było wywołań nigdzie pisać i mnie to zmyliło.

komentarz 9 stycznia 2016 przez Comandeer Guru (601,450 p.)

Nie, NIE DOSTAWIA SIĘ TEGO TYPU ATRYBUTÓW DO BODY. To jest naruszanie zasady rozdzielności warstw aplikacji.

Wystarczy wywołać funkcję aktualizacja od razu po jej zadeklarowaniu – tyle.

komentarz 9 stycznia 2016 przez ScriptyChris Mędrzec (190,190 p.)

NIE DOSTAWIA SIĘ TEGO TYPU ATRYBUTÓW DO BODY.

Masz na myśli, że skrypt dodany na końcu body nie powinien zawierać $(document).ready (albo odpowiednika w czystym JavaScript)?

komentarz 9 stycznia 2016 przez Comandeer Guru (601,450 p.)

Mam na myśli atrybuty HTML, nie konstrukcje w JS. Nie podpina się zdarzeń poprzez HTML, bo to mieszanie warstw aplikacji: https://kornel.ski/onclick

Inna rzecz, że umieszczenie skryptu na końcu body spełnia taką samą funkcję, co $(document).ready – odpala skrypt JS, gdy drzewko DOM jest (w miarę) kompletne.