Zegarek nie da się załadować JS

pytanie zadane 20 października 2016 w JavaScript przez Nowicjusz89 Obywatel (1,050 p.)
zmienione kategorie 20 października 2016 przez Arkadiusz Waluk

Witam,

Problem istnieje w załadowanie zegarka z JS.

Widzę ,żę jest problem z załadowaniem ale nie wiem jak zrobić.Może to pomoże,ale składnia nie jest podświetlona zarówno w JS jak i w HTML odnośnie JS.

HTML:

<!DOCTYPE HTML>
<html lang="pl">
<head>
	<meta charset="utf-8" />
	<meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1" />

	<title>Jan Kowalski - Porfolio</title>

	<meta name="description" content="Stworzę dla Ciebie wyjątkowo stronę.Zatrudnij programistę webowego." />
	<meta name="keywords" content="zamów stronę,tworzenie www,programista,portfolio" />
	
	<link rel="stylesheet" href="style.css" type="text/css"/>
	<link rel="stylesheet" href="css/fontello.css" type="text/css"/>
	<link href="https://fonts.googleapis.com/css?family=Josefin+Sans|Lato&amp;subset=latin-ext" rel="stylesheet">

	<script src="timer.js"></script>
	
</head>

<body onload="odliczanie();">

	<div id="container">

	<div class="rectangle">
		<div id="logo"><a href="zatrudnij-programiste-webowego"class="tilelinkhtml5">Jan Kowalski</a> </div>
		<div id="zegar">12:00:00 </div>
		<div style="clear:both;"></div>
	</div>

	<div class="square">
		  <div class="tile1">
		    <a href="kim-jestem" class="tilelink"><i class="icon-user"></i><br />Kim jestem?</a>
		  </div>
		  <a href="oferta-tworzenie-stron-www"class="tilelinkhtml5">
		  	<div class="tile1">
		    	<i class="icon-laptop"></i><br />Co oferuję?
		  	</div>
		  </a>
		  <div style="clear:both;"></div>

		  <div class="tile2">
		  	<a href="cv-programista-php-javascript cv" class="tilelink"><i class="icon-graduation-cap"></i><br />Curriculum vitae</a>
		 </div>
		  <div class="tile3">
		  	<a href="kontakt-programista-php" class="tilelink"><i class="icon-mail"></i><br />Kontakt ze mną </a>
		  </div>
		  <div style="clear:both;"></div>

		  <div class="tile4">
		  	<i>Talk is cheap.Show me the code!</i><br />- Linus Torvalds,twórca Linuxa
		  </div>
		</div>

	<div class="square">
		<div class="tile5">
		Tytuł podstrony <br /> <br />
		Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed tempus augue ac malesuada sagittis. Nullam lacinia dolor vitae faucibus varius. Integer a velit ultricies, efficitur sapien eu, finibus lacus. Suspendisse potenti. Donec venenatis quis tellus eget vehicula. Vivamus in dignissim tellus. Nullam non enim ut mi fermentum viverra id nec sapien. 
		</div>
		<div class="yt">
			<a href="http://youtube.com" target ="_blank"class="sociallink"><i class="icon-youtube"></i></a>
		</div>
		<div class="fb">
			<a href="http://facebook.com" target ="_blank"class="sociallink"><i class="icon-facebook"></i></a>
		</div>
		<div class="gplus">
		   <a href="http://plus.google.com" target ="_blank"class="sociallink"><i class="icon-gplus"></i></a>
		</div>
		<div class="tw">
		   <a href="http://twitter.com" target ="_blank"class="sociallink"><i class="icon-twitter"></i></a>
		</div>
		<div style="clear:both;"></div>
	</div>
		<div style="clear:both;"></div>
		<div style="clear:both;"></div>

	<div class="rectangle">2015 &copy; Jan Kowalski - Porfolio.Programista webowy zaprasza 
		do współpracy! <i class="icon-mail-alt"></i> jan.kowalski@adres.com </div>
	</div>	

</body>
</html>

JS:

function odliczanie()
	{
		var dzisiaj = new Date();
		
		
		var godzina = dzisiaj.getHours();
		if (godzina<10) godzina = "0"+godzina;
		
		var minuta = dzisiaj.getMinutes();
		if (minuta<10) minuta = "0"+minuta;
		
		var sekunda = dzisiaj.getSeconds();
		if (sekunda<10) sekunda = "0"+sekunda;
		
		document.getElementById("zegar").innerHTML =  "+godzina+":"+minuta+":"+sekunda;
		 
		 setTimeout("odliczanie()",1000);
	}

1 odpowiedź

odpowiedź 20 października 2016 przez Arkadiusz Waluk Ekspert (287,950 p.)
wybrane 21 października 2016 przez ScriptyChris

Najlepsza

document.getElementById("zegar").innerHTML =  "+godzina+":"+minuta+":"+sekunda;

To na pewno nie jest poprawnie zapisane.

komentarz 20 października 2016 przez Nowicjusz89 Obywatel (1,050 p.)

document.getElementById("zegar").innerHTML = godzina+":"+minuta+":"+sekunda;

Poprawione.Również nie działa.

komentarz 20 października 2016 przez Arkadiusz Waluk Ekspert (287,950 p.)

Z tego co widzę to po poprawie tej linii działa: http://jsfiddle.net/87uo9h4o/
(kliknąłem tylko przycisk jsfiddle nad Twoim postem i poprawiłem podaną wyżej linijkę)

komentarz 20 października 2016 przez Nowicjusz89 Obywatel (1,050 p.)

Widzę właśnie ,że na jsfiddle działa ,a w przeglądarce jak i na lokalnym serwerze również bez zmian.Hmm.Jakieś pomysły?

komentarz 20 października 2016 przez Arkadiusz Waluk Ekspert (287,950 p.)

Nie za bardzo, powinno działać - zarówno kod na jsfiddle jak i na Twojej stronie wykonuje przeglądarka. Na pewno dobrze zapisałeś plik, pod dobrą nazwą? Może CTRL+F5, aby pobrać od nowa skrypt?

komentarz 20 października 2016 przez ScriptyChris Mędrzec (190,190 p.)

Jakieś pomysły?

W konsoli jakieś błędy?

Porada poboczna: wywal atrybut onload z tagu <body>, a skrypt z <head> przenieś na koniec tagu <body>.

Nieprawidłowo korzystasz z setTimeout(): https://developer.mozilla.org/en-US/docs/Web/API/WindowTimers/setTimeout

var timeoutID = window.setTimeout(code[, delay]);
///
code

An optional syntax allows you to include a string instead of a function, which is compiled and executed when the timer expires. This syntax is not recommended for the same reasons that make using eval() a security risk.

zamiast kodu w miejsce code przekaż po prostu nazwę (referencję) do funkcji.