[HTML] pobierz, plaintext 
<script> 
 
function liczg() 
{
if (document.getElementById('godz').value > 23 ) {
window.alert("błąd godziny");
document.getElementById('godz').value=""; }
 
} 
 
function liczm() 
{
if (document.getElementById('min').value > 59 ) {
window.alert("błąd minut");
document.getElementById('min').value=""; }
}
</script>
 
 
<input name="godz" type="text" class="INPUT" maxlength="2"  onkeypress="if (event.keyCode < 48 || event.keyCode > 57) event.returnValue = false;" onChange="liczg();" />&nbsp;&nbsp;<strong>:</strong>&nbsp;
    <input name="min" type="text" class="INPUT" maxlength="2" onkeypress="if (event.keyCode < 48 || event.keyCode > 57) event.returnValue = false;" onChange="liczm()"  />
[HTML] pobierz, plaintext 

jestem zielony z js i to moj pierwszy skrypt prosze o wyrozumialosc i pomoc
- poprawnosc wprowadzonych danych jest sprawdzana tylko raz
- jak zrobic by po wyswietleniu komunikatu o bledzie kursor ustawil sie
w polu gdzie byl blad
pozdrawiam

http://developer.mozilla.org/pl/docs/DOM:element.focus

Czyli w Twoim kodzie:

- pierwsza funkcja:
po wywołaniu metody alert() napisz:
document.getElementById('godz').focus();

- w drugiej funkcji tak samo
// alert
document.getElementById('min').focus();

dzieki za zainteresowanie moim problemem
w tresci input dopisalem id="godz" (id="min")
i nadal jest tak jak bylo
czyli dane sa sprawdzane tylko za pierwszym razem
a "document.getElementById('godz').focus();" nie ustawia kursora
w wybranym polu
pozdrawiam

po pierwsze pamiętaj, że getElementById można używać gdy strona jest załadowana w całości... po drugie onChange zadziała gdy zatwierdzisz wartość klikając poza pole input! Lepiej zrobic coś takiego:

[HTML] pobierz, plaintext 
<script>
 
 
  liczg = function(oForm) {
      if (oForm.godz.value > 23 ) {
          window.alert("błąd godziny");
          oForm.godz.value = ""; 
      }
  }
 
  liczm = function(oForm) {
      if (oForm.min.value > 59 ) {
          window.alert("błąd minut");
          oForm.min.value=""; 
      }
  }
 
  </script>
 
  <form>
  <input name="godz" type="text" class="INPUT" maxlength="2" onkeypress="if (event.keyCode < 48 || event.keyCode > 57) event.returnValue = false;" onkeyup="liczg(this.form);" />&nbsp;&nbsp;<strong>:</strong>&nbsp;
  <input name="min" type="text" class="INPUT" maxlength="2" onkeypress="if (event.keyCode < 48 || event.keyCode > 57) event.returnValue = false;" onkeyup="liczm(this.form);" />
  </form>
[HTML] pobierz, plaintext 

dzieki
dziala
musze to przetrawic