0

Есть код:

function checkUser(ctx) {
  let fileData = [1, 2, 3];
  fileData.forEach(element => {
    if (element == userId) {
      console.log("true");
      return element;
    }
  });
  console.log("false");
  return false;
}

Я ожидал увидить в консоли false или true. Но вместо true вижу true false вместе. Почему?

2
  • 1
    первый return у Вас для коллбэка метода forEach, а не для функции checkUser. Если что, то цикл forEach прервать нельзя. Либо пользуйтесь обычным for, либо специальными методами массивов some и every
    – Dmytro
    7 июн 2019 в 14:42
  • Воспользуйтесь методом find вместо forEach. Ну или some, да, в этом случае должно быть достаточно. 7 июн 2019 в 14:45

3 ответа 3

0

Используйте for

let fileData = [1, 2, 3],
    userId = 3;
function checkUser(fileData, userId) {
   
  for (let element of fileData) {
    if (element == userId) {
      console.log("true");
      return element;
    }
  }
  console.log("false");
  return false;
};


checkUser(fileData, userId);
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>

0

Найдено даже два ответа.

1) Так как первый return относится к foreach, то можно использовать some или every. @Дмытрык

2) Для моего конкретного случая идеально подходит find вместо б. @smellyshovel

0
function checkUser(ctx) {
let fileData = [1, 2, 3];
  for(let element of fileData){
    if (element == userId) {
      console.log("true");
      return element;
    }
  }
  console.log("false");
  return false;
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.