co jest źle - prosty zegar w js

pytanie zadane 1 czerwca 2020 w JavaScript przez Damian12345 Bywalec (2,940 p.)

Uczę się teraz łączyć js z html i nie działa mi i a chciałbym wiedzieć co tu jest źle?

w sekcji head napisałem coś takiego(to jest poprawna nazwa pliku js);

<script src="strona główna.js" type="text/javascript"/>

a to jest plik js;

function odliczanie
{
var dzisiaj = new Date();

var dzien = dzisiaj.getDate();
if(dzien < 10) dzien = "0"+dzien;

var miesiac = dzisiaj.getMonth()+1;
if(miesiac < 10) miesiac = "0"+miesiac;

var rok = dzisiaj.getFullYear();
if(rok < 10) rok = "0"+rok;


var godzina = dzisiaj.getHours();
if(godzina < 10) godzina = "0"+godzina;

var minuta = dzisiaj.getMinutes();
if(minuta < 10) minutaa = "0"+minuta;

var sekunda = dzisiaj.getSeconds();
if(sekunda < 10) sekunda = "0"+sekunda;

document.getElementById("zegar").innerHTML=dzien;

setTimeout("odliczanie()" , 1000)
}

edit: id diva też się zgadza

komentarz 1 czerwca 2020 przez Paweł Nąckiewicz Nałogowiec (48,990 p.)

src="strona główna.js"

tutaj ma być nazwa pliku zawierającego JavaScript (wraz z ewentualną ścieżką)

komentarz 1 czerwca 2020 przez Damian12345 Bywalec (2,940 p.)

no to jest

komentarz 1 czerwca 2020 przez Paweł Nąckiewicz Nałogowiec (48,990 p.)

a masz podpiętą funkcje onload w body?

<body onload="alert('Przykładowy tekst');">

komentarz 1 czerwca 2020 przez Jackoza Początkujący (410 p.)

W jednym miejscu brakuje Ci średnika w JS.

2	komentarz 1 czerwca 2020 przez tkz Nałogowiec (42,000 p.) Średnik nie jest takim problemem jak brak () w funkcji...

komentarz 1 czerwca 2020 przez Jackoza Początkujący (410 p.)

Rzeczywiście to też jest problem. Oprócz tego zdążyłem jeszcze znaleźć literówkę przy zmiennej minuta w której zamiast napisać minuta napisałeś minutaa. Ogólnie bez tych błędów wszystko się zgadza.

1 odpowiedź

odpowiedź 1 czerwca 2020 przez zoya Bywalec (2,840 p.)

W nazwach plików nie powinieneś używać spacji, polskie znaki też nie są wskazane. Czyli twój skrypt powinien mieć nazwę np. stronaGlowna.js. Jak wspomniano powyżej masz źle zadeklarowaną funkcję odliczanie () -> bez nawiasów. Tu: setTimeout("odliczanie()" , 1000) nie powinieneś wstawiać odliczanie() jako łańcucha znaków, tylko wywołać funkcję. Poza tym, setTimeout wywołujesz w funkcji odliczanie, powinieneś to zrobić poza tą funkcją.