getLargestExpressionResult(a,b)

Question

I'm trying to get this function done but something is wrong, it should return 50 for a=5,b=10 or 2 for a=1,b=1. However any other way to write this function would be interesting

function getLargestExpressionResult(a, b) {
  const sum = a + b;
  const prod = a * b;
  const diff = a - b;
  const quot = a \ b;
  
  let res = sum;
  
  if (prod > res) res = prod;
  if (diff > res) res = diff;
  if (quot > res) res = quot;
  
  return res;
}

enter code here

you need to take `/` instead of backslash `\\\`. – Nina Scholz Feb 14 '22 at 10:59 — Nina Scholz, Feb 14 '22 at 10:59

score 1 · Answer 1 · answered Feb 14 '22 at 10:59

1

You can use Math.max(). It can take multiple parameters and gives you max. Also for division it is /

function getLargestExpressionResult(a, b) {
  const sum = a + b;
  const prod = a * b;
  const diff = a - b;
  const quot = a / b;
  
  return Math.max(sum,prod,diff,quot);
}

console.log(getLargestExpressionResult(5, 10))

answered Feb 14 '22 at 10:59

cmgchess

7,996
37
44
62

the purpose of this was to make it working without using Math – Marius Feb 14 '22 at 11:02
1

then just change the \ to /. your one is fine. https://stackoverflow.com/questions/33851810/how-do-javascript-math-max-and-math-min-actually-work – cmgchess Feb 14 '22 at 11:04

score 1 · Answer 2 · answered Feb 14 '22 at 11:02

You don't really need to find maximum element on your own, you can simply use Math.max:

const getLargestExpressionResult = (a, b) => Math.max(a * b, a / b, a + b, a - b);

console.log(getLargestExpressionResult(5, 10));
console.log(getLargestExpressionResult(1, 1));

Also should be noticed division is /, not \.

Or if you need to avoid Math usage:

getLargestExpressionResult = (a, b) => {
    const expressionValues = [ a + b, a - b, a * b, a / b ];
    let max = expressionValues[0];
    for (const value of expressionValues) {
        if (value > max) max = value;
    }
    return max;
}

console.log(getLargestExpressionResult(5, 10));
console.log(getLargestExpressionResult(1, 1));

getLargestExpressionResult(a,b)

2 Answers2