Dzielnie przez zero JS zabezpieczenie formularza

Question

Cześć, potrzebuje pomocy w programie JavaScript a mianowicie w zabezpieczeniu formularza przed dzieleniem przez zero oraz pozostawieniem formularza pustego. 

Program działa kiedy:
Oba formularze są puste
Wartość 'a' jest uzupełniona a wartość 'b' równa zero

Problem tkwi w momencie kiedy b==0 a wartość 'a' jest pusta, wtedy jest komunikat, że nie można dzielić przez zero, chodzi aby w tym wypadku pokazywało aby uzupełnić 

function dziel()   
        {
        var a = (document.getElementById("a").value);
        var b = (document.getElementById("b").value); 
        if (isNaN(a) || !b)
          {
            document.getElementById("wynik").innerHTML = "uzupelnij";
        }
            else if(b==0)
            {
                document.getElementById("wynik").innerHTML = "nie wolno dzielić przez zero";
            }
            else{
             var wynik = a / b;
             document.getElementById("wynik").innerHTML = "Wynik działania wynosi: "+ wynik; 
            }
          }

 

Comments

"Cholero, nie dziel przez zero" ~~ :)

Answer 1

W liniach 3 i 4 przepuść dane z formularza przez funkcję parseFloat lub parseInt (jak pobierasz tylko liczby całkowite  z pola typu number). W warunku w linii 5 również dla zmiennej b użyj funkcji isNaN.

Answer 2

Osobiście zrobiłbym to tak:

function dziel() {
   var a = parseFloat((document.getElementById("a").value));
   var b = parseFloat((document.getElementById("b").value)); 
   if (!isNaN(a) && !isNaN(b)) {
      if(b != 0) {
         var wynik = a / b;
         document.getElementById("wynik").innerHTML = "Wynik działania wynosi: " + wynik;
      } else {
         document.getElementById("wynik").innerHTML = "nie wolno dzielić przez zero";
      }
   } else {
      document.getElementById("wynik").innerHTML = "uzupelnij";
   }
}

Do pobierania liczb z formularza lepiej użyć inputów typu number.

Answer 3

Możesz taką walidację zrobić na poziomie HTML. Ustaw atrybuty required oraz min="1" (tutaj warto ustawić typ inputa na number) lub jakiś ułamek, typu 0.1 i wtedy wysłanie formularza zostanie zablokowane przez przeglądarkę; a możesz też odpalić taką walidację z poziomu JS-a, żeby otrzymać boolean oznaczający (nie)poprawność kontrolki/formularza i na tej podstawie pokazać użytkownikowi własne komunikaty o błędach.