Task 145

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

Task 145

Deirra

Условие:
Обрезание c учетом слов

Реализуй функцию shortenByWords(s, n).

Она должна принять строку и число.

Вернуть нужно новую строку, состоящую максимум из n символов оригинальной строки.

Если тебе нужно обрезать несколько символов, добавь ... в конце.

Нельзя обрезать строку посередине слова, поэтому включай в результат только полные слова

Так, shortenByWords('this string is too long to fit our ad', 21) вернет this string is too...
Мой код;
export const shortenByWords = (s, n) => { let s1 = s.split("") let g =[] let z = [] if (s1.length <= n) { return s } else{ for (let i = 0; i <= n-1; i++) { g.push(s1[i]) } for (let r = g.length-1; r > 0; r--) { if (g[r]===" ") { for (let r1 = 0; r1 < r; r1++) { z.push(g[r1]) } z.push("...") return z.join("") } else{ g[r] = "" } } } }

Ошибка:

НЕТ 1. Функция shortenByWords должна возвращать строку

ДА2. Строку нельзя изменять, если n больше или равно ее длине.

НЕТ3. Результат должен включать только полные слова

НЕТ4. Результат должен заканчиваться на ..., если исходная строка должна быть сокращена

Coderslang_Master

В целом код работает, но иногда возвращает undefined

Попробуй проверить на строке some;11xx string

Deirra

Coderslang_Master
У меня работает

Или я не так понял?

Coderslang_Master

Попробуй вторым параметром передать другое число. Например 5.

Deirra

Coderslang_Master
Спасибо помогли!!!

Sasha

Добрый день!
исправьте пожалуйста ошибку:

export const shortenByWords = (s, n) => {
  if (s.length > n) {
    if (s[n+1] === ' ') {
        return s.slice(0, n) + '...';
    } else {
      return s.slice(0, n) + '...';
    }
} else {
  return s;
}
}

обрезало середину слова, программа пропустила

Coderslang_Master

@Sasha чтобы проверить возможную ошибку, нужны входящие параметры для функции. Чему были равны s и n?

Sasha

const longString = 'this string is quiteeeeeeeeeeeeeee long, especially for its age.'
const shortString = 'a short one'

console.log(shortenByWords(longString, 20));
console.log(shortenByWords(shortString, 20));

Grimshock

Код с комментами:

export const shortenByWords = (s, n) => {
  let a;                                                            // обьявляем переменную
  if (s.length <= n) {                                              // проверяем строку, 
//чтобы она была меньше/равно чем n (т.е. 20-и символам), если условие выполнено - возвращаем
//строку без изменений 
    return s;
  } else {                                                          // если строка всё же
// больше чем n (более 20-и символов) - режем её до длины, которая равна нашему n (20) и 
// запихиваем в переменную 'a'
    a = s.slice(0, n);
    return a.slice(0, a.lastIndexOf(' ')) + '...';                  // и далее уже 
// обрезанную строку возвращаем с нашим вторым условием (3. Результат должен включать только 
// полные слова), т.е. с помощью lastIndexOf мы находим тот самый последний пробел и с помощью 
// slice обрезаем до этого самого пробела. В конце добавляем троеточие 
  }
};

Код без комментов:

export const shortenByWords = (s, n) => {
  let a;
  if (s.length <= n) {
    return s;
  } else {
    a = s.slice(0, n);
    return a.slice(0, a.lastIndexOf(' ')) + '...';
  }
};

В итоге получаем:

Задание выполнено. Просидел над ним около двух часов, гуглив варианты (варианты не подходили), но на самом деле всё необходимое было тут:
https://learn.coderslang.com/ru/0004-built-in-string-functions-in-js/, главное хорошенько подумать 😃

Artis

Вставлю свои 5 копеек)
export const shortenByWords = (s, n) => {
let newS = '';
if(s.length > n){
newS = s.substr(0, n);
return newS += '...';
} else {
return s;
}
}