Java script, как сделать так чтобы срабатывали все условия if else?

Question

[nikiGal]

Решаю задачу по функциям, условия задачи написать функцию для расчета количества корма для животных. Где numAnimals количество животных, avgFood количество еды в кг на одного животного. Необходимо чтобы функция возвращала количество еды на всех животных, либо -1 если numAnimals или avgFood меньше нуля или не являются числом. Я написал эту функцию, но у меня срабатывают все условия кроме последнего: else {return numAnimals*avgFood;] Это условие возвращает так же значение -1. понимаю что проблема в if или else if но что не пойму. спасибо за подсказки!
а вот сам код.

function calculateFoodOrder(numAnimals, avgFood) {
var numAnimals = 10;
var avgFood = 3;
if (numAnimals < 0) { return -1; }
else if (numAnimals = String) { return -1; }
else if (avgFood < 0) { return -1; }
else if (avgFood = String) { return -1; }
else
{return numAnimals * avgFood; }
}
console.log(calculateFoodOrder());

Comments

[WbICHA]

numAnimals = String
Это что?

[Ярослав Иванов]

WbICHA, скопировал конструктор)))

Answer 1

[Tim]

https://developer.mozilla.org/ru/docs/Web/JavaScri...

function calculateFoodOrder(numAnimals = 0, avgFood = 0) {
    if(Number.isInteger(numAnimals) && Number.isInteger(avgFood) && numAnimals > 0 && avgFood > 0 ) {
        return numAnimals * avgFood;
    }
    return -1;
}

Короткая запись

const calculateFoodOrder = (n = 0, f = 0) => (Number.isInteger(n) && Number.isInteger(f) && n > 0 && f > 0 ? n * f : -1);




console.log(calculateFoodOrder()); // -1
console.log(calculateFoodOrder(-1,4)); // -1
console.log(calculateFoodOrder(1,3));//3
console.log(calculateFoodOrder([], 200)); // -1
console.log(calculateFoodOrder(Infinity, 200)); // -1

Answer 2

[Александр]

все грехи данного кода:

function calculateFoodOrder(numAnimals, avgFood) {
  var numAnimals = 10;
  var avgFood = 3;
  if (numAnimals < 0) {
    return -1;
  } else if (numAnimals = String) {
    return -1;
  } else if (avgFood < 0) {
    return -1;
  } else if (avgFood = String) {
    return -1;
  } else {
    return numAnimals * avgFood;
  }
}

console.log(calculateFoodOrder());

• = - оператор присваивания, не сравнения.
  чтобы сравнить A с B, вы должны использовать один из операторов сравнения:
  ==, ===, <=, >=, <, > . . .
  
  
• если вы хотите проверить, является ли передаваемое параметру значение строкой, вы должны:
  1. убрать вышеобъявленные переменные с помощью var
  2. проверить, является ли значение параметра numAnimals строкой - это делается с помощью оператора typeof:
  

  if (typeof numAnimals === "string") { return . . . }

  
  
  
• чтобы правильно вызвать функцию с таким кодом и получить правильный ответ - вам нужно передать в круглые скобки значения, которые присвоятся параметрам функции (numAnimals и avgFood):
  

  console.log(  calculateFoodOrder(значение1, значение2)  ) 
  
  console.log(  calculateFoodOrder(550, 241)  ) // пример

  
  не забывайте, что, если вы передаёте параметру функции строку - вы должны обрамить её в один из доступных видов кавычек (``, '', "").
  
  

Comments

[WbICHA]

не забывайте, что, если вы передаёте параметру функции строку - вы должны обрамить её в один из доступных видов кавычек (``, '', "").

Вот так ты взял и запорол следующий вопрос автора.)

[WbICHA]

все грехи данного кода:

Забыл упомянуть, что вместо 10 ифов, всё можно было в одном сделать через ||.

[WbICHA]

И что проверку на строку стоит делать ДО того, как происходит преобразование в число (в ифах с <).
И проверку делать не на строку, а на то, что числом не является.

Answer 3

[Сергей delphinpro]

Зачем проверять на строку? А если будет передан массив или буль?
Если вы ожидаете число, то и проверять нужно на число.

Для улучшения читаемости кода, можно написать отдельную функцию валидации аргумента.

function validArgument(n) {
  return typeof n === 'number' && n >= 0;
}

function calculateFoodOrder(numAnimals, avgFood) {
  if (validArgument(numAnimals) && validArgument(avgFood)) {
    return numAnimals * avgFood;
  }

  return -1;
}