Задача 132

Этот сайт лучше всего просматривать в современном браузере с включённым JavaScript.

Задача 132

Archy_k

Задание:
Почини функцию getLocale.
Она должна вернуть userConfig.locale только если значение этого поля найдено в knownLocales.
Иначе - en.

Моё решение:


export const getLocale = (userConfig, knownLocales) => {
    
    for (let i = 0; i < knownLocales.length; i++) {
    
      if (userConfig.locale == knownLocales[i]) {
      return userConfig.locale;
      } else {
    return 'en';
    }
  }
};

Результат - не верно!
Перебрал кучу варианов, но не понимаю что не так делаю.
Не выдают мои варианты правильного ответа.

Сдаюсь, подскажите что не так?

Immuno

Archy_k При сравнении лучше использовать ===. Использование обычного сравнения == может вызывать проблемы. Например, оно не отличает 0 от false.
У вас return в for, следовательно каждый проход цикла возвращает 'en', а так не задумано.

Archy_k

уже и лекцию перечитал несколько раз...
дайте просто ответ, я не понимаю и не пойму, а так хоть сверюсь!
заранее спасибо.

Immuno

Archy_k Если я дам Вам ответ, вы ничему не научитесь. Внимательно посмотрите на ответы, которые выдаёт ваша программа и поймите, почему так получается, проблема в неправильном положении return 'en'. Он у Вас в цикле и каждый раз цикл возвращает 'eu'. Не смотря на сравнение. Подумайте еще. Если не получается, попробуйте решить другие задачи, а потом вернуться к этой.

Coderslang_Master

@Archy_k проблема в том, что ты выходишь из цикла после первой итерации, а нужно пройтись по всем элементам knownLocales.

Grimshock

Тоже ломал голову, никак не выходило реализовать 🙂
Получилось сделать с помощью двух циклов:

export const getLocale = (userConfig, knownLocales) => {
  for (let i = 0; i < knownLocales.length; i++) {
    if (userConfig.locale === knownLocales[i]) {
      return userConfig.locale;
    }
  }
  for (let i = 0; i < knownLocales.length; i++) {
    if (userConfig.locale !== knownLocales[i]) {
      return 'en';
    }
  } 
};

Но что-то мне подсказывает что есть более лаконичный вариант

media_kot

Grimshock
Просто вместо второго for оставьте return 'en'. Логика такова: если вы нашли язык в массиве, то цикл прервется выходом из функции и возвратом найденного значения. А если не найдет в цикле значение, то возвращается уже 'en'

Murano

media_kot вообще не очевидно. То есть прошел цикл и если не одно из заданных условий не выполнено, то программа просто пойдет читать дальше код и прочтет return ('en');
Так получается?

Крутая задача, мозги закипели

Sergey_P

Классная задача, понял как работает этот цикл )
Оставлю здесь, может кому-то будет полезным.

Моё решение такое:
export const getLocale = (userConfig, knownLocales) => {
for (let i = 0; i < knownLocales.length; i++) {
if (userConfig.locale === knownLocales)
{
return userConfig.locale;
}
}
return 'en';
}

Тут главный вопрос "Почему return 'en' не в цикле? Почему неправильно будет добавлять его в else?"
Для себя ответил, когда просто пошагово прошелся по тому как работает цикл. Сразу на примере userConfig2:

1) Берем значение 'ru' и сравниваем его с первым элементом массива knownLocales 'en'
2) Результат этого значения - false. И уже на этом этапе цикл переходит сразу к else и выполняет return 'en'. То есть просто не идет дальше.

Если же return 'en' вынесен из цикла, то сравнение проходит по всем элементам массива и только после того, как результат всех сравнений 'false' - переходит к return 'en'.

Поправьте, если не так.

Hlopshak

export const getLocale = (userConfig, knownLocales) => {
for (let i = 0; i < knownLocales.length; i++) {
if (userConfig.locale === knownLocales)
return userConfig.locale;
}
return 'en';
}

teeter

export const getLocale = (userConfig, knownLocales) => { for (let i = 0; i < knownLocales.length; i++) { while (userConfig.locale === knownLocales[i]) { return userConfig.locale } } return 'en'; }
вариант с упрощенной системой