Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> Funkcja nie wyświetla poprawnie wszystkich elementów, po zatwierdzeniu funkcja nie wyświetla godziny
Mastersieciweb
post 28.12.2013, 18:16:17
Post #1





Grupa: Zarejestrowani
Postów: 84
Pomógł: 0
Dołączył: 14.10.2013

Ostrzeżenie: (0%)
-----

Witam mam mały problem, nie wiem jak zwykle w czym tkwi błąd, mam do napisania funkcje która po wybraniu konkretnego miasta i zatwierdzeniu buttonem ma mi pokazac w polu aktualny czas, oczywiscie co do poprawnosci godziny w danym miescie proszę nie zwracać uwagi, problem jest taki że nie wyświetla mi godziny i nie mam zielonego pojęcia gdzie tu jest błąd.. z góry dziękuje za pomoc, oto mój kod:

[HTML] pobierz, plaintext 
  1. <html>
  2.   <head>
  3.  
  4.   <title>zegarr</title>
  5.  
  6.  
  7.   </head>
  8.   <body>
  9.   <script>
  10.   function sik(){
  11.   var polee=window.document.getElementById('pole1');
  12.   var now=new Date();
  13.   var godzina=now.getUTCHours();
  14.   var minuta=now.getUTCMinutes();
  15.   var sekunda=now.getUTCSeconsd();
  16.   var londyn1= window.document.getElementById('londyn');
  17.   if (londyn1.checked == true){
  18.   polee=document.write(godzina+":"+minuta+":"+sekunda);}
  19.   var chicago1= window.document.getElementById('chicago');
  20.   if (chicago1.checked == true){
  21.   godzina=godzina+9;
  22.   polee=document.write(godzina+":"+minuta+":"+sekunda);}
  23.   var newyork1= window.document.getElementById('newyork');
  24.   if (newyork1.checked == true){
  25.   godzina=godzina-5;
  26.   polee=document.write(godzina+":"+minuta+":"+sekunda);}
  27. }
  28.   </script>
  29.   <form name="k">
  30.   <input type="text" name="pole" id="pole1">
  31.   <input type="button" value="nacisnij" onClick="sik();"><br>
  32.   <input type="radio" name="f" id="londyn">Londyn<br>
  33.   <input type="radio" name="f" id="chicago">Chicago<br>
  34.   <input type="radio" name="f" id="newyork">New York<br>
  35.  
  36.   </form>
  37.  
  38.  
  39.   </body>
  40. </html>
[HTML] pobierz, plaintext
Powód edycji: [Daiquiri]: Poprawiam temat
Go to the top of the page
+Quote Post
kamil4u
post 28.12.2013, 18:23:11
Post #2





Grupa: Zarejestrowani
Postów: 2 350
Pomógł: 512
Dołączył: 4.01.2009
Skąd: Wrocław / Świdnica

Ostrzeżenie: (0%)
-----

1. Konsola błędów
2. var sekunda=now.getUTCSeconds();
3. poczytaj innerHTML
4. poczytaj o setInterval

Pozdrawiam


--------------------
>>> iPage.pl - JS, HTML5, CSS3, Android - moja oferta
>>> myPrzepis.pl - blog kulinarny
Go to the top of the page
+Quote Post
wladimirec
post 28.12.2013, 18:46:38
Post #3





Grupa: Zarejestrowani
Postów: 28
Pomógł: 7
Dołączył: 27.10.2004

Ostrzeżenie: (0%)
-----

Mój przedmówca ma racje - powinieneś poznać chociażby podstawy JS
Oto moja propozycja funkcji:

[HTML] pobierz, plaintext 
  1. function sik(){
  2.  
  3.   var now=new Date();
  4.   var godzina=now.getUTCHours();
  5.   var minuta=now.getUTCMinutes();
  6.   var sekunda=now.getUTCSeconds();
  7.  
  8.   var polee=window.document.getElementById('pole1');
  9.   var chicago1= window.document.getElementById('chicago');
  10.   var newyork1= window.document.getElementById('newyork');
  11.  
  12.   var h_diff = 0;
  13.  
  14.   if (chicago1.checked == true){
  15. 	h_diff=9;
  16.   }
  17.   if (newyork1.checked == true){
  18.     h_diff=-5;
  19.   }
  20.  
  21.   polee.value = (godzina+h_diff)%24+":"+minuta+":"+sekunda;
  22.  
  23. }
[HTML] pobierz, plaintext


Ale że to forum ma służyć do nauki, to:
  1. Masz literówkę w linii var sekunda=now.getUTCSeconsd();
  2. document.write nie służy do ustawiania wartości pola
  3. logikę można nieco uprościć
  4. pamiętaj o uwzględnieniu 24-godzinnego zakresu po operacji dodawania godzin %24
Go to the top of the page
+Quote Post
kamil4u
post 28.12.2013, 19:30:24
Post #4





Grupa: Zarejestrowani
Postów: 2 350
Pomógł: 512
Dołączył: 4.01.2009
Skąd: Wrocław / Świdnica

Ostrzeżenie: (0%)
-----

Jeśli już mówimy jak to powinniśmy zrobić porządnie to nie:
Cytat
pamiętaj o uwzględnieniu 24-godzinnego zakresu po operacji dodawania godzin %24

a raczej dodanie X godzin jeszcze przed wyciągnięciem godzin, minut, sekund itd. np. :
Kod
var now = new Date();
var new = new Date(now.getTime() + (2*1000*60*60));


Metoda przedmówcy też jest dobra, ale do czasu gdy kiedyś zajdzie potrzeba np. pokazania też, który jest aktualnie dzień to nie trzeba będzie, aż tak kombinować - lepiej od razu myśleć o takich rzeczach smile.gif

Pozdrawiam


--------------------
>>> iPage.pl - JS, HTML5, CSS3, Android - moja oferta
>>> myPrzepis.pl - blog kulinarny
Go to the top of the page
+Quote Post
wladimirec
post 28.12.2013, 20:01:38
Post #5





Grupa: Zarejestrowani
Postów: 28
Pomógł: 7
Dołączył: 27.10.2004

Ostrzeżenie: (0%)
-----

Porządnie to nie zawsze znaczy na zapas. KISS
Zakładanie dodatkowych hipotetycznych wymagań jest bezcelowe.
Jednakowoż znowu muszę Ci przyznać rację, kamil4u wink.gif Istnieją strefy czasowe różniące się od czasu uniwersalnego o niecałkowitą liczbę godzin.
I wtedy:
[HTML] pobierz, plaintext 
  1. function sik(){
  2.   var now=new Date();
  3.  
  4.   var polee=window.document.getElementById('pole1');
  5.   var chicago1= window.document.getElementById('chicago');
  6.   var newyork1= window.document.getElementById('newyork');
  7.  
  8.   var h_diff = 0;
  9.  
  10.   if (chicago1.checked == true){
  11. 	h_diff=9;
  12.   }
  13.   if (newyork1.checked == true){
  14.     h_diff=-5;
  15.   }
  16.  
  17.   var ndate = new Date(now.getTime() + (h_diff*1000*60*60));
  18.  
  19.   polee.value = ndate.getUTCHours()+":"+ndate.getUTCMinutes()+":"+ndate.getUTCSeconds();
  20.  
  21. }
[HTML] pobierz, plaintext
Go to the top of the page
+Quote Post
Mastersieciweb
post 28.12.2013, 21:13:42
Post #6





Grupa: Zarejestrowani
Postów: 84
Pomógł: 0
Dołączył: 14.10.2013

Ostrzeżenie: (0%)
-----

generalnie zrobiłem to bardziej amatorsko, faktycznie nie wpadłem na sposób w jaki umieścić godzine w polu;)
wydawało mi się że znam podstawy JS, jestem w trakcie nauki tego języka ale cóż, nie jest łatwo.

[HTML] pobierz, plaintext 
  1. <html>
  2.   <head>
  3.  
  4.   <title>zegarr</title>
  5.  
  6.  
  7.   </head>
  8.   <body>
  9.   <script>
  10.   function sik(){
  11.   var polee=window.document.getElementById('pole1');
  12.   var now=new Date();
  13.   var godzina=now.getUTCHours();
  14.   var minuta=now.getUTCMinutes();
  15.   var sekunda=now.getUTCSeconds();
  16.  
  17.  
  18.   var londyn1= window.document.getElementById('londyn');
  19.   if (londyn1.checked == true){
  20.   polee.value=(godzina+":"+minuta+":"+sekunda);}
  21.   var chicago1= window.document.getElementById('chicago');
  22.   if (chicago1.checked == true){
  23.   godzina=godzina+9;
  24.   if (godzina>24){
  25.   godzina=godzina-24;}
  26.   if (godzina<0){
  27.   godzina=godzina+24}
  28.   if (godzina<10){
  29.   godzina="0"+godzina}
  30.   polee.value=(godzina+":"+minuta+":"+sekunda); }
  31.   var newyork1= window.document.getElementById('newyork');
  32.   if (newyork1.checked == true){
  33.   godzina=godzina-5;
  34.   if (godzina>24){
  35.   godzina=godzina-24;}
  36.   if (godzina<0){
  37.   godzina=godzina+24}
  38.   if (godzina<10){
  39.   godzina="0"+godzina}
  40.   polee.value=(godzina+":"+minuta+":"+sekunda);} 
  41. }
  42.   </script>
  43.   <form name="k">
  44.   <input type="text" name="pole" id="pole1">
  45.   <input type="button" value="nacisnij" onClick="sik();"><br>
  46.   <input type="radio" name="f" id="londyn">Londyn<br>
  47.   <input type="radio" name="f" id="chicago">Chicago<br>
  48.   <input type="radio" name="f" id="newyork">New York<br>
  49.  
  50.   </form>
  51.  
  52.  
  53.   </body>
  54. </html>
[HTML] pobierz, plaintext


jeszcze pytanko, wiadomo że to nie ten temat na te pytanie ale znacie jakieś ćwiczenia w JS od podstaw do wyższego poziomu zebym mógł rozwiązywać je i się szkolić?smile.gif
Go to the top of the page
+Quote Post
wladimirec
post 28.12.2013, 21:33:25
Post #7





Grupa: Zarejestrowani
Postów: 28
Pomógł: 7
Dołączył: 27.10.2004

Ostrzeżenie: (0%)
-----

Cytat(Mastersieciweb @ 28.12.2013, 23:13:42 ) *
jeszcze pytanko, wiadomo że to nie ten temat na te pytanie ale znacie jakieś ćwiczenia w JS od podstaw do wyższego poziomu zebym mógł rozwiązywać je i się szkolić?smile.gif


Znalezione w moich zakładach, może Ci się przyda:
Go to the top of the page
+Quote Post
kamil4u
post 28.12.2013, 22:05:46
Post #8





Grupa: Zarejestrowani
Postów: 2 350
Pomógł: 512
Dołączył: 4.01.2009
Skąd: Wrocław / Świdnica

Ostrzeżenie: (0%)
-----

Cytat
Porządnie to nie zawsze znaczy na zapas. KISS
Zakładanie dodatkowych hipotetycznych wymagań jest bezcelowe.

Racja, ale tym sposobem jest szybciej( brak zbędnych operacji matematycznych ), kod jest krótszy, kod stwarza nowe możliwości - na pewno więcej zalet, niż wad. O tym co Ty podałeś nie pomyślałem - ale masz tutaj rację.
Zasada jest zachowana smile.gif

A co do samego stwierdzenia - "Zakładanie dodatkowych hipotetycznych wymagań jest bezcelowe" to masz rację, ale jeśli możesz zrobić coś co później pomoże Ci rozwiązać dany problem i nie zajmie całość zbyt dużo czasu to warto napisać tak kod, aby był uniwersalny i o to mi głównie chodziło smile.gif

Pozdrawiam


--------------------
>>> iPage.pl - JS, HTML5, CSS3, Android - moja oferta
>>> myPrzepis.pl - blog kulinarny
Go to the top of the page
+Quote Post
« Następny starszy · JavaScript · Następny nowszy »
 

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Standardowy · Przełącz na: Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Wersja Lo-Fi Aktualny czas: 13.06.2025 - 22:56
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn