[js]czas Opcje

[rychu123]

witam, co jest w tym skrypcie źle ze nie odświeża

[PHP] pobierz, plaintext 
<script language="javascript" >
function czas() {
 
var nazwy_mies = [ 'Styczeń', 'Luty', 'Marzec', 'Kwiecień', 'Maj', 
    'Czerwiec', 'Lipiec', 'Sierpień', 'Wrzesień', 'Październik', 
    'Listopad', 'Grudzień' ];
 
// Odczytanie bieżącej daty i czasu, i rozbicie ich na składowe
var data = new Date();
var rok = data.getFullYear();
var mies = data.getMonth();
var dzien = data.getDate();
var godz = data.getHours();
var min = data.getMinutes();
var sec = data.getSeconds();
 
// Dodanie zera na początku minut i sekund jeżeli trzeba
if (min < 10)
    min = '0' + min;
if (sec < 10)
    sec = '0' + sec;
 
// Utworzenie odpowiednio sformatowanej daty i czasu
var data_i_czas = dzien + ' ' + nazwy_mies[mies] + ' ' + rok 
    + ', ' + godz + ':' + min + ':' + sec;
 
 
return data_i_czas;
 
}
setTimeout('czas()' ,500);
document.write(czas());
 
 
</script>
[PHP] pobierz, plaintext 

Ten post edytował rychu123 26.01.2010, 22:04:55

[nospor]

na pierwszy rzut oka rzuca sie to:
setTimeout(czas() ,500);
powinno byc
setTimeout('czas()' ,500);

Drugiego rzutu oka juz nie robilem

[rychu123]

nie działa dalej proszę o pomoc

[nospor]

mowiac nie odswieza, masz na mysli ze pierwszy czas sie wyswietla a potem juz nie? No bo tylko raz wywolujesz te funkcje

[piotrooo89]

a mam taki gotowy zegarek, zmień na swoje i będzie hulać:

[HTML] pobierz, plaintext 
<form name="info" action="#">
<input type="text" id="zegar" value="" size="8" class="zegar" readonly="readonly">
 
</form>
<script>
function zegarek()
{
var zegar = new Date();
 
var czas;
 
var sekunda, minuta, godzina;
sekunda = zegar.getSeconds();
minuta = zegar.getMinutes();
godzina = zegar.getHours();
 
 
czas = godzina +":"+ minuta +":"+ sekunda;
 
window.document.forms.info.zegar.value=czas;
setTimeout("zegarek();",1000);
}
 
zegarek();
</script>
[HTML] pobierz, plaintext 

[rychu123]

chce żeby zegar cały czas liczył czas

[nospor]

no to ci napisales: odpalasz wyswietlanie czasu tylko raz. naprawdę pisze nie zrozumiale?

ALbo uzyj setInterval() albo setTimeout() wywoluj tez w funkcji.

[rychu123]

no to ci napisales: odpalasz wyswietlanie czasu tylko raz. naprawdę pisze nie zrozumiale?

ALbo uzyj setInterval() albo setTimeout() wywoluj tez w funkcji.

ale to nie działa

[nospor]

wszystkim dziala,tylko nie tobie. Na dodatek nie pokazales jak zastosowales moje rady wiec nadal mozemy zgadywac co masz źle...

Kod ktory podał piotrooo89 zawiera wlasciwe uzycie setTimeout() i jakos jemu dziala.

[rychu123]

[PHP] pobierz, plaintext 
<script language="javascript" >
function czas() {
 
var nazwy_mies = [ 'Styczeń', 'Luty', 'Marzec', 'Kwiecień', 'Maj', 
    'Czerwiec', 'Lipiec', 'Sierpień', 'Wrzesień', 'Październik', 
    'Listopad', 'Grudzień' ];
 
// Odczytanie bieżącej daty i czasu, i rozbicie ich na składowe
var data = new Date();
var rok = data.getFullYear();
var mies = data.getMonth();
var dzien = data.getDate();
var godz = data.getHours();
var min = data.getMinutes();
var sec = data.getSeconds();
 
// Dodanie zera na początku minut i sekund jeżeli trzeba
if (min < 10)
    min = '0' + min;
if (sec < 10)
    sec = '0' + sec;
 
// Utworzenie odpowiednio sformatowanej daty i czasu
var data_i_czas = dzien + ' ' + nazwy_mies[mies] + ' ' + rok 
    + ', ' + godz + ':' + min + ':' + sec;
 
 
return data_i_czas;
setTimeout('czas();' ,500);
}
 
document.write(czas());
 
 
</script>
[PHP] pobierz, plaintext 

to powiedz mi co jest źle

[nospor]

Kod

return data_i_czas;
setTimeout('czas();' ,500);

W kazdym jezyku programowania to co jest po RETURN nie wykonuje sie. Czy juz wiesz co jest źle?
A na przyszlosc nie kaz sie prosic o każdą rzecz tylko sam od razu podaj jak najwiecej informacji...

[rychu123]

czyli ma być tak ? To nie działa

[PHP] pobierz, plaintext 
setTimeout('czas();' ,500);
return data_i_czas;
[PHP] pobierz, plaintext 

[nospor]

ok, to kolejna podpowiedź: w swojej funkcji nic z tym czasem nie robisz. Wyliczasz go i tyle.

Przykladowo kod piotrooo89 coś z tym czasem robi, Twoj nic.

[zegarek84]

czyli ma być tak ? To nie działa

[PHP] pobierz, plaintext 
setTimeout('czas();' ,500);
return data_i_czas;
[PHP] pobierz, plaintext 

jeśli już używasz setTimeout to nie w takiej konstrukcji - raz, że jest ciutkę wolniejsza a dwa to robisz coś w stylu eval... przekaż funkcję jako pierwszy parametr czyli setTimeout(czas ,500); - poza tym, skoro czas wyświetlasz z dokładnością do 1s to po co liczysz to 2 razy?? - wystarczy setTimeout(czas ,1000);... a nawet jeśli chcesz przekazać parametry do tej funkcji to "normalnie" podaje się je jako kolejne argumenty funkcji setTimeout - tylko w ie tak się nie da - ale wszystko można obejść - wystarczy oczytać o zasięgu zmiennych - eval is evil

skoro uparłeś się na return to to powinno być jeszcze ciutkę inaczej - gdyż funkcję powinieneś brać jako argument - ale ogólnie mi się nie chciało za dużo pisać - przerobiłem Twój skrypt deczko i usunąłem też return... - ogólnie na multum sposobów to można zrobić...

a i jeśli masz konwersję do stringu to szybsze jest .concat do stringu oraz połączenie tablicy przez .join() od znaku +:

[HTML] pobierz, plaintext 
<HTML>
<HEAD>
</HEAD>
<BODY>
<div id='zegar'></div>
 
<script>
var nazwy_mies = [ 'Styczeń', 'Luty', 'Marzec', 'Kwiecień', 'Maj', 
    'Czerwiec', 'Lipiec', 'Sierpień', 'Wrzesień', 'Październik', 
    'Listopad', 'Grudzień' ],
	$timerID = null , $pole=null;;
function czas() {
// Odczytanie bieżącej daty i czasu, i rozbicie ich na składowe
var data = new Date(),
rok = data.getFullYear(),
mies = data.getMonth(),
dzien = data.getDate(),
godz = data.getHours(),
min = data.getMinutes(),
sec = data.getSeconds(),
data_i_czas=null;
 
// Dodanie zera na początku minut i sekund jeżeli trzeba
if (min < 10)
    min = '0'.concat(min);
if (sec < 10)
    sec = '0'.concat(sec);
 
// Utworzenie odpowiednio sformatowanej daty i czasu
data_i_czas = [dzien, ' ', nazwy_mies[mies], ' ', rok, 
    ', ', godz, ':', min, ':', sec].join('');
$pole.innerHTML = data_i_czas;
$timerID=setTimeout(czas ,1000);
}
 
(function(){
  $pole=document.getElementById('zegar');
  czas();
})()
</script>
</BODY>
</HTML>
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<script language="javascript" >
function czas() {
....
return data_i_czas;
setTimeout('czas();' ,500);
}
document.write(czas());
</script>
[PHP] pobierz, plaintext 

to powiedz mi co jest źle

przede wszystkim używając document.write wyświetliłeś funkcję pierwszy raz... potem popatrz na to co zostawiłem jeszcze w pętli... hmmm... pomijając jeszcze kilka faktów to drugi raz funkcji nie wykonasz gdyż funkcja przestaje działać po zwróceniu wartości w return ... hmmm... nawet jakbyś zamienił kolejność tych ostatnich linijek w funkcji to o tyle poszedł byś do przodu, że funkcja ciągle by się wykonywała, jednak nigdzie nie wyświetlała gdyż... hmm... do niczego nie przypisałeś wartości tej funkcji

Ten post edytował zegarek84 26.01.2010, 23:56:15