И несколько параметров

function andMultipleExpr(){
  let logicalAnd;
  let i;
  for (i = 0; i < arguments.length; i++){
    logicalAnd =  arguments[i] && arguments[i+1];
  }
  return logicalAnd;
}

console.log(andMultipleExpr(true, true, false, false));

Я ожидаю выполнения этого кода: true && true && false && false, и это должно вернуть false .

Как заставить это работать в JS? Спасибо

Используйте Array.prototype.everyвсе переданные аргументы, чтобы проверить, все ли они истинны;

function andMultipleExpr(...a) {
  if(a.length === 0) return false; // return false when no argument being passed
  return a.every(Boolean);
}

console.log(andMultipleExpr(true, true, false)); // should return false
console.log(andMultipleExpr(true, true, true)); // should return true

Тебе следует

1. Начните с logicalAndнабораtrue

2. Использовать logicalAndпри обновлении, а не использовать две записи изarguments

Минимальное изменение:

function andMultipleExpr(){
    let logicalAnd = true; // ***
    let i;
    for (i = 0; i < arguments.length; i++){
        logicalAnd = logicalAnd && arguments[i]; // ***
    }
    return logicalAnd;
}
console.log(andMultipleExpr(true, true, false, false));

Но у решения mbojko есть преимущество короткого замыкания (остановка цикла, когда он впервые находит ложное значение), что кажется хорошей идеей.

Поскольку вы используете ES2015 +, вам, вероятно, следует использовать параметр rest, а не arguments, и вы можете использовать for-ofцикл:

function andMultipleExpr(...flags) {
    let logicalAnd = true;
    for (const flag of flags) {
        logicalAnd = logicalAnd && flag;
    }
    return logicalAnd;
}
console.log(andMultipleExpr(true, true, false, false));

Вы также можете замкнуть, что в соответствии с подходом Мбойко

function andMultipleExpr(...flags) {
    for (const flag of flags) {
        if (!flag) {
            return false;
        }
    }
    return true;
}
console.log(andMultipleExpr(true, true, false, false));

Некоторые люди могут бросить reduceэто, но решение Арчиevery намного лучше. (Но поскольку ваше сравнение не является строгим, я бы просто сделал это .every(flag => flag).)

Ранний возврат должен сделать код более эффективным и более коротким:

function andMultipleExpr() {
  for (let i = 0; i < arguments.length; i++) {
    if (!arguments[i]) {
      return false;
    }
  }

  return true;
}

Я думаю, что это очень короткий путь с использованием ES6 Array.prototype.reduce

let andMultipleExpr = (...args) => args.reduce((a, b) => a && b);

console.log(andMultipleExpr(true, true, false, false));

Для получения более подробной информации о функции снижения, пожалуйста, прочитайте MDN

Вы можете взять Array#everyи вернуть последнее значение.

Такой подход возвращает реальный результат логического И&& .

Используя этот подход, создается короткое замыкание для первого найденного ложного значения. Затем итерация останавливается.

function andMultipleExpr(...args) {
    var result; // any return value is configurable for empty args
    args.every(v => result = v);
    return result;
}

console.log(andMultipleExpr(true, true, false, false));
console.log(andMultipleExpr(true, true, 1, 2));
console.log(andMultipleExpr(true, 0, 1, 2));

Возможно, вы хотите услышать, что пошло не так с циклом:

for (i = 0; i < arguments.length; i++){
  logicalAnd =  arguments[i] && arguments[i+1];
}

1. этот цикл хранит &&последние два элемента, с которыми он сталкивается. В идеальном случае это были бы &&вместе два последних элемента массива (что уже не то, что вам нужно)
2. Кроме того, в конце цикла i=arguments.length-1он проверяет последний элемент массива и i+1является элементом «после» последнего, то есть undefined. С точки зрения логических отношений это рассматривается false, но в этом случае &&создает само значение, и поэтому функция возвращает undefinedвсе время (это можно было упомянуть в вопросе).

Документы

expr1 && expr2: Если expr1можно преобразовать в true, возвращает expr2; еще, возвращается expr1.

arr=[true];
console.log("your case:",arr[0] && arr[1]);

console.log("1 && 2:", 1 && 2);

function andMultipleExpr(){
    let logicalAnd = arguments[0] || false;
    for (let i = 1; i < arguments.length && logicalAnd; i++){
        logicalAnd = logicalAnd && arguments[i];
    }
    return logicalAnd;
}

console.log("():",andMultipleExpr());
console.log("(false):",andMultipleExpr(false));
console.log("(true):",andMultipleExpr(true));
console.log("(true,true):",andMultipleExpr(true,true));
console.log("(true, true, false, false):",andMultipleExpr(true, true, false, false));

и затем вы можете оптимизировать его в соответствии с ответом Арчи : результат &&-ing элементов - это trueесли все элементы есть true, и вам не нужно выполнять одну &&операцию для вычисления результата:

function andMultipleExpr(){
    if(arguments.length===0){
      return false;
    }
    for (let i = 0; i < arguments.length; i++){
      if(!arguments[i]){
        return false;
      }
    }
    return true;
}

console.log("():",andMultipleExpr());
console.log("(false):",andMultipleExpr(false));
console.log("(true):",andMultipleExpr(true));
console.log("(true,true):",andMultipleExpr(true,true));
console.log("(true, true, false, false):",andMultipleExpr(true, true, false, false));

(В приведенных выше фрагментах я стремился falseсоздать пустой список аргументов.)