Task 151

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

Task 151

Vlad

на мой взгляд, практические задачи не соответствуют изложенным материалам,
с задачей 151 вообще проблема на проблеме, даже если скопировать код с лекции - он не работает...
перепробовал несколько вариантов: ни один не работает.
export const isShortWord = () => {
return (s) => {
return s.length < 10;

}
}

Coderslang_Master

Vlad давай разделим проблему на две.

У всех учеников разный уровень. Некоторые задачи ДОЛЖНЫ быть трудными. Их нужно пропустить и вернуться позже. Пришли, пожалуйста, список задач которые ты не смог решить.
Присылай пример кода из лекции, который не работает. isShortWord в лекции 15 я найти не смог.

salton

При решении задачи получаю следующий ответ:

Изменил предложенное условие задачи:

import { isShortWord } from './functions.js';

const words = [ 'good', 'things', 'happen', 'if', 'you', 'do', 'something good', 'for_a_long_time' ];
const shortWords = words.filter(isShortWord(10));

console.log(shortWords);    // [ 'good', 'things', 'happen', 'if', 'you', 'do' ]

Сама функция выглядит:

export const isShortWord = (s) => {
  return (n) => {
    return n.length < s;
  }
}

Coderslang_Master

salton
Первая проблема - isShortWord(s) возвращает функцию, а не boolean, как требует условие.
Вторую проблему думаю заметишь чуть позже сам 🙂

salton

Coderslang_Master как только исправил первую проблему - сразу же открылись глаза на все остальное!
PS. задача слишком легкая, ищешь какой-то подвох, нюанс - а все лежит на поверхности!

Ilia

Задача прошла при не выполненном 2 условии

2) Строка состоит из одного слова (в ней нет пробелов)

export const isShortWord = (s) => {
return s.length < 10;
}

условие(стандартное)

import { isShortWord } from './functions.js';
const words = [ 'good', 'things', 'happen', 'if', 'you', 'do', 'something good', 'for_a_long_time' ];
const shortWords = words.filter(isShortWord);
console.log(shortWords); // [ 'good', 'things', 'happen', 'if', 'you', 'do' ]

Вообще не знаю как соответствовать 2 условию, единственное что нашел это во так:

export const isShortWord = (s) => {
return s.length < 10 || s.includes(' ');
}

но мне надо чтоб s.includes(' '); возвращал противоположное буливское значение,чтоб если слово имеет пробел вернуло false, кароч капец пол дня сижу, нужна подсказка очень!)

Coderslang_Master

Ilia инвертировать результат includes ты можешь так:

!s.includes(' ');

вернет true, если в строке НЕТ пробела и false если пробел есть в строке.

Ilia

Coderslang_Master
Так и знал, что какая-нить мелочь, пыхтел пыхтел и вот оно)
! поставил его, понял, что || не подходит, почитал еще раз про &&
и со спокойной душой можно отдохнуть 😀

Olesya

@Coderslang_Master
Чувствую себя совершенно бестолковой, но от этой задачи у меня больше вопросов чем идей.

Задача: прописать функцию, что принимает строку и возвращает true, если строка удовлетворяет требованиям: одно слово и меньше 10 символов.

в Решении:

const shortWords = words.filter(isShortWord(10));

Функция:

  return (n) =>  {
    for (let i = 0; i < n.length; i++) {
      if (n[i].length < s && !n[i].includes(' ')) {
        return true
        }
      }
    }
  }

Так вот, мне не понятен конструкт функции с функцией.
Несколько раз перечитала лекцию, но не могу понять что за чем происходит? Как программа понимает что из заданного в функции s, а что n? У меня оно в принципе не срабатывает, так что я не могу понять, равнозначны ли функции:

export const isShortWord = (n) => {
  return (s) => { return s.length > n}}

и
export const isShortWord = (s) => { return (n) => { return s.length > n }}
?
Помогите, пожалуйста!(

Coderslang_Master

Olesya в этом примере, функция isShortWord возвращает новую функцию:

export const isShortWord = (n) => {
  return (s) => s.length < n;
}

Первый параметр - это n.

const shorterThan5 = isShortWord(5);

Теперь shorterThan5 - функция. Ее можно было бы написать отдельно, без isShortWord и тогда это выглядело бы так:

const shorterThan5 = (s) => {
  return s.length < 5;
}

Тут уже нет n, т.к. число 5 "сохранено" внутри функции и не изменится в будущем.

Функции shorterThan5 из двух примеров выше - идентичны. Для внешнего наблюдателя, который не знаком с реализацией, не будет никакой разницы какую из них использовать.

Преимущество первого варианта в том, что можно создать сколько угодно функций с помощью isShortWord(n) и избежать дубликации кода.

Olesya

@Coderslang_Master
Написала функцию для задачи "прописать функцию, что принимает строку и возвращает true, если строка удовлетворяет требованиям: одно слово и меньше 10 символов" :

export const isShortWord = (s) => {
  return (n) =>  {
    for (let i = 0; i < n.length; i++) {
      if (n.length < s && !n.includes(' ')) {
        return true
      }
    }
  }
}

В solusion подправила третью строку:

import { isShortWord } from './functions.js';
const words = [ 'good', 'things', 'happen', 'if', 'you', 'do', 'something good', 'for_a_long_time' ];
const shortWords = words.filter(isShortWord(10));
console.log(shortWords);    // [ 'good', 'things', 'happen', 'if', 'you', 'do' ]

И в итоге получила нужный ответ:
[ 'good', 'things', 'happen', 'if', 'you', 'do' ]

Но при проверке задачи выдаёт, что всё неправильно.
Не могу понять, что его не устраивать? 🙁

salton

Olesya не устраивает то, что функция isShortWord у вас возвращает не булево, а другую функцию

Coderslang_Master

Olesya давай начнем с того, что переименуем параметры.

s - строка, n - число.
Зачем тебе цикл?

Olesya

salton Почему? у меня же в конце цепочки всё заканчивается return true .

salton

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

salton

хотите проверить -

export const isShortWord = (s) => {

        return true
      }

вставьте это и проверьте задачу - первый пункт должен выполниться

Olesya

Coderslang_Master

параметры переименовала.
Цикл, как я считала, нужен, так как мы имеем дело с массивом и надо перебрать все строки в нём.
При этом я не могла понять почему при команде s[i] выдавало только одну букву)) Но если мы каким-то образом сразу попадаем в строку, то понятно, почему так выходило 🙂

В итоге вышло:

export const isShortWord = (n) => {
  return (s) =>  {
    if (s.length < n && !s.includes(' ')) {
      return true
    }
  }
}

Чувствую себя полной двоечницей, но можно ещё подсказку?(

Coderslang_Master

Olesya отлично. Теперь читаем условие задачи и обращаем внимание на предыдущий комментарий от @salton 🙂

В этой задаче не нужны замыкания.

Olesya

Coderslang_Master
Спасибо за подсказку: программа ответ приняла, но при этом запуск самого кода выдаёт ошибку и внезапно начал ругаться на длину строки.

При этом в solution я вернула первоначальную запись:
const shortWords = words.filter(isShortWord());
В function всё сократилось до двух строчек:

export const isShortWord = (s) => {
  return (s.length < 10 && !s.includes(' '))
  }

А в итоге при запуске програма выдаёт ошибку:

 return (s.length < 10 && !s.includes(' '))
            ^
TypeError: Cannot read properties of undefined (reading 'length')

Как такое может быть? И что ему не нравиться (до этого же всё работало!) 🙁

salton

вся проблема в этой строке: const shortWords = words.filter(isShortWord());
вы правильно сделали, что решили убрать аргументы из функции...
но в условиях задачи на сколько мне помнится написано что-то вроде: "обратите внимание как передается функция isShortWord". и не зря. верните как все было: const shortWords = words.filter(isShortWord); и будет вам счастье! заодно прочитайте как работают методы массивов - фильтр, мап и другие, это реально интересная и полезная информация, которая может понадобится до конца обучения!

Coderslang_Master

salton отличный ответ. Все правильно. Добавлю только что по filter, map и reduce будет отдельная лекция. Это самые важные методы для работы с массивами в JS.

Следующая страница »