Почему функция возвращает неверный результат?

Question

[atsin4469]

Задание
Напишите функцию makeItFunny(), которая принимает на вход строку и возвращает её копию, у которой каждый n-ный элемент переведен в верхний регистр. n – задается на входе в функцию.

Мое решение

const makeItFunny = (text, num) => {
  let i = 0;
  let result = '';
  while (i < text.length) {
    if ((i + 1) % num === 0) {
      text[i].toUpperCase();
    }
    result += text[i];
    ++i;
  }
  return result;
}

Результат теста

Expected: "I NevEr LooK bAck". 
Received: "I never look back"

Answer 1

[Pavel Shvedov]

Внимательно читаем секцию "Описание"
https://developer.mozilla.org/ru/docs/Web/JavaScri...

Comments

[atsin4469]

Спасибо, а как можно лаконично исправить функцию?
Есть вот такое решение, но смущает наличие else и дублирование в нем result

const makeItFunny = (str, n) => {
  let i = 0;
  let result = '';
  while (i < str.length) {
    const current = str[i];
    if ((i + 1) % n === 0) {
      result += current.toUpperCase();
    } else {
      result += current;
    }
    i += 1;
  }

  return result;
};

Answer 2

[Сергей Соколов]

Сделать из строки массив букв, каждый n-й элемент заменить на его UpperCase версию, склеить снова в строку.

const makeItFunny = (str, n) => str.split('').map((el, i) => ?????).join(''); // Profit!

Comments

[Дмитрий]

Прошу прощения за тупой вопрос возможно... а 5? Это магия?) Или "здесь должно быть условие"?)

[Сергей Соколов]

Дмитрий, там вам нужно всё понять и написать свой код, типа ) Чтобы получить профит.
Ведь не ждёте спойлером готового решения? Учиться-то кто будет

[Дмитрий]

Сергей Соколов, да я то понял) просто очень смутило наличие Profit в конце.
Я уж грешным делом и засомневался, вдруг знаки вопроса волшебные и сами n могут вычислить

[G F]

Вместо foreach должен быть map

[WbICHA]

Можно сделать немного красивее, чтобы не сплитить и не джойнить.)
[].reduce.call(str, (acc, l, i) => ?????, '')

[Сергей Соколов]

WbICHA,

const makeItFoo = (str, n) => {
  const re = new RegExp(`(${'.'.repeat(n-1)})(.)`, 'g');
  return str.replace(re, (a,b,c) => b + c.toUpperCase());
}
makeItFoo('abcdefgh', 3) // "abCdeFgh"

[Дмитрий]

Сергей Соколов,

const nUpper = (str, n) => str.split('').map((el, i) => (i+1)%n == 0 ? el.toUpperCase() : el ).join('');

у меня так получилось... все же перед split должна быть строка? иначе split не сработает.
console.log(nUpper('happy 2021 new year',3))
haPpy 2021 New yeAr
п.с. я последнее время постепенно познаю js, периодически тренируюсь "на кошках"

меня вот что смущает в задаче - "возвращает копию". а Map же не возвращает копию, а создает новый. Насколько это может быть критично? и еще момент. Если стоит задача - каждый n-ый элемент и n=3 - n считать по строке или индексу элемента? Тогда могут быть разные результаты. Ведь первый индекс - ноль и соответственно всегда вернет правду.
Извиняюсь, что отвлек не являясь по сути автором вопроса.

[Сергей Соколов]

Дмитрий, map() это не про строки, а про массивы. Да тут и не важно. Строка разумеется будет другой, чем исходная. Другим объектом.

Как именно считать N-й элемент, с нуля или как можно позже – дело вкуса, в условии не уточняют, но в примере результата явно второе. Значит вы правильно сделали с (i + 1) % n

[WbICHA]

Сергей Соколов, во-первых, не `(${'.'.repeat(n-1)})(.)`, а `(.{${n - 1}})(.)`.
Во-вторых, регексп медленный.)

[Дмитрий]

Сергей Соколов, спасибо за ответ. Сейчас как-то незаметно для себя углубился в js. Когда-то все на jquery делал, потом перешел на VUE, а сейчас появилось немного времени переписать старые проекты и пришлось углубляться в работу с данными на клиенте.