проверка на деление на ноль в javascript

Question

Я пытаюсь выполнить операцию калькулятора, в зависимости от входных значений и оператора, результат должен отображаться. Я поражен проверкой деления на 0, и я не могу отобразить результат в другой функции. Пожалуйста, кто-то может мне помочь!

function calCaulation(e) { 
var x = document.getElementById("first").value; 
var y = document.getElementById("second").value; 
var z = document.getElementById("oper").value; 
var a = ""; 

if ((isNaN(x) || x == "") || (isNaN(y) || y == "")) { 
a = "Sum:Please enter the valid Number"; 
} 
if (z == "/" && y == 0) { 
a = "Divide By Zero Error"; 
} 
return a; 
} 
else { 
var x = parseFloat(document.getElementById("first").value); 
var y = parseFloat(document.getElementById("second").value); 
switch (z) { 
case ("+"): 
    a = "Sum:" + ((x + y).toFixed(2)); 
    break; 
case ("-"): 
    a = "Sub:" + ((x - y).toFixed(2)); 
    break; 
case ("*"): 
    a = "Mul:" + ((x * y).toFixed(2)); 
    break; 
case ("/"): 
    a = "Div:" + ((x/y).toFixed(2)); 
    break; 
default: 
    a = "Invalid Operator"; 
} 
return a; 
} 
} 

Answer 1

У вас есть скобка и else статья слишком много.

См. Здесь для работы jsfiddle. https://jsfiddle.net/r2aq88tu/4/

редактировать: также, ваше возвращение заявление после проверки ошибок будет оставить функцию до того, как реальный код будет выполнять, я переместил return a внутри каждого if, чтобы заставить его работать.

Ваш новый код:

function calCaulation(e) { 
    var x = document.getElementById("first").value; 
    var y = document.getElementById("second").value; 
    var z = document.getElementById("oper").value; 
    var a = ""; 

    if ((isNaN(x) || x == "") || (isNaN(y) || y == "")) { 
     a = "Sum:Please enter the valid Number"; 
     return a; 
    } 
    if (z == "/" && y == 0) { 
     a = "Divide By Zero Error"; 
     return a; 
    } 


    var x = parseFloat(x); 
    var y = parseFloat(y); 
    switch (z) { 
     case ("+"): 
      a = "Sum:" + ((x + y).toFixed(2)); 
      break; 
     case ("-"): 
      a = "Sub:" + ((x - y).toFixed(2)); 
      break; 
     case ("*"): 
      a = "Mul:" + ((x * y).toFixed(2)); 
      break; 
     case ("/"): 
      a = "Div:" + ((x/y).toFixed(2)); 
      break; 
     default: 
      a = "Invalid Operator"; 
    } 
    return a; 

}