Best way to prevent/handle divide by 0 in javascri

2019-01-17 16:05发布

What is the best way to prevent divide by 0 in javascript that is accepting user inputs. If there is no particular way to achieve this what would be the best way to handle such a situation so as to not prevent other scripts from executing?

Any insights are much appreciated.

8条回答
混吃等死
2楼-- · 2019-01-17 16:30

There is no way to do that with the normal / and /= operators.

The best way to do what you want is with guards:

function notZero(n) {
  n = +n;  // Coerce to number.
  if (!n) {  // Matches +0, -0, NaN
    throw new Error('Invalid dividend ' + n);
  }
  return n;
}

and then do division like

numerator / notZero(denominator)

Alternatively you can always guard the output

function dividend(numerator, denominator) {
  var quotient = numerator / denominator;
  if (quotient !== quotient) { throw new Error(numerator + " / " + denominator); }
  return quotient;
}

but that loses the readability and expressiveness of /=.

查看更多
欢心
3楼-- · 2019-01-17 16:30

To prevent (unwanted) execution

  1. Always verify critical user input and/or results
  2. Use logic and/or callbacks you can prevent to execute
  3. On HTML forms etc. you can use i.e. return false; as value to stop submission.
查看更多
Juvenile、少年°
4楼-- · 2019-01-17 16:34

The best way is contextual. But here's the easiest:

function myFunction( input ){
    input = 0 ? 0.0001 : input; // same as if( input == 0 ){ input = 0.0001; }
    return 1 / input;
}

Basically if the input is zero, turn it into a very small number before using as a denominator. Works great for integers, since after your division you can round them back down.

A couple caveats prevent this from being universal:

  • It could cause false positives if your input accepts really small numbers
  • It won't trigger any error-handling code, if you need to do something special if zero is entered

So it's best for general-purpose, non-critical cases. For example, if you need to return the result of a complex calculation and don't care if the answer is accurate to N digits (determined by 0.0001 vs. 0.00000001, etc.); you just don't want it to break on a divide-by-zero.

As another answer suggested, you could also create a reusable global function.

function divisor( n ){ return ( n = 0 ? 0.0001 : n ); }
function myFunction( input ){ return 1 / divisor( input ); }

Possible improvements:

function divisor( n, orError ){
    if( typeof n == 'undefined' || isNaN( n ) || !n ){
        if( orError ){ throw new Error( 'Divide by zero.' ); }
        return 0.000000000000001;
    }else{ return 0 + n; }
}

This would take any value (null, number, string, object) and if invalid or zero, return the failsafe zero-like value. It would also coerce the output to a number just in case it was a string and you were doing something odd. All this would ensure that your divisor function always worked. Finally, for cases where you wanted to handle such errors yourself, you could set the second parameter to true and use a try/catch.

查看更多
倾城 Initia
5楼-- · 2019-01-17 16:44

Off the top of my head you could:

  1. Check the user input to see if the denominator is zero (or evaluates to zero, depending on what your script actually does).
  2. Check if the result of the action isFinite() and if not then handle appropriately.
查看更多
Bombasti
6楼-- · 2019-01-17 16:45

what would be the best way to handle such a situation so as to not prevent other scripts from executing

Division by zero doesn't seem to prevent other scripts from execution in JavaScript:

var a = 20;
var b = 0;
var result = a/b;
console.log(result); // returns Infinity

If you want something different to happen in case of division by zero, you could use

function divideIfNotZero(numerator, denominator) {
  if (denominator === 0 || isNaN(denominator)) {
        return null;
  }
  else {
        return numerator / denominator;
  }
}
查看更多
疯言疯语
7楼-- · 2019-01-17 16:45

A bit different than stopping execution, but the ternary operator is a pretty slick way to customize variable assignment.

var one = 1,
    zero = 0,
    customValue = 1;


var quotient = zero===0 ? customValue : one / zero;

This way, by setting the customVariable to the integer of your choice, you can expect a predictable result when division by zero occurs.

查看更多
登录 后发表回答