Почему не формируется новая строка из цикла?

Question

[Илья]

Всем привет!
Задача перевернуть каждое слово в строке задом наперед. Почему-то не формируется новая строка newStr, подскажите, что я делаю не так.

const ex = "My favourite game";
    
function wordReverse(str) {
    var newStr = "",
        strLength = str.length;
    for (var i=0; i<strLength; i++) {
        newStr[i] = str[strLength - 1 - i];
    }
    return newStr;
}

function wordsReverse(str) {
    str += " ";
    var wordsReverseArray = [],
        wordStr = "";
    for(var i=0; i<str.length; i++) {
        if(str[i] !== " ") {
            wordStr += str[i];
        } else {
            wordsReverseArray.push(wordReverse(wordStr));
            wordStr = "";
        }
    }
    return wordsReverseArray;
}

console.log(wordsReverse(ex)); // Array(3) [ "", "", "" ]

Спасибо.

Comments

[Ihor Bratukh]

newStr = "", newStr[i]
Нельзя так делать, строка не массив. Попробуйте в консоли это:
'hello'[0] = 'R'
Строка не изменится.

[Илья]

BRAGA96, хмм, а вроде в функции wordsReverse работал со строкой, как с массивом. То есть в функции wordReverse нужно работать с массивом, а потом как-то вернуть строку?

[Ihor Bratukh]

Илья, Можно получить символ строки как индекс у массива, но изменить нельзя, методы массива не действуют. Попробуйте например 'hello'.reverse() - ничего не будет, строка не массив.

Answer 1

[Алексей П]

Здесь вы правильно конкатенируете (вторая функция wordsReverse):
wordStr += str[i];
А здесь (первая функция wordReverse):
newStr[i] = str[strLength - 1 - i];
пытаетесь присвоить пустой строке символ не выделив для нее место. В C вы получили бы ошибку, так как попытались записать за границы выделенной памяти. Т.е. вам здесь так же нужно слепливать строки:
newStr += str[strLength - 1 - i];

Comments

[Илья]

Точно. Спасибо!

Answer 2

[alex]

const ex = "My favourite game";
    
function wordReverse(str) {
    var newStr = '',
        strLength = str.length;
      for (var i = 0; i < strLength; i++) {
        newStr += str[strLength - 1 - i];
    }  
    return newStr;
}

function wordsReverse(str) {
    str += " ";
    var wordsReverseArray = [],
        wordStr = "";
    for(var i=0; i<str.length; i++) {
        if(str[i] !== " ") {
            wordStr += str[i];
        } else {
            wordsReverseArray.push(wordReverse(wordStr));
            wordStr = "";
        }
    }
    return wordsReverseArray.join(' ');
}

console.log(wordsReverse(ex)); // "yM etiruovaf emag"

Answer 3

[Ihor Bratukh]

reverseString('hello, world'); //> "dlrow ,olleh"

function reverseString(string) {
	return string.split('').reverse().join('');
}

Comments

[Илья]

Это вы всю строку перевернули, а мне надо каждое слово из строки)

Answer 4

[Vlad]

const ex = "My favourite game";
console.log(ex.split(' ').map((item) => { return item.split('').reverse().join(''); }).join(' '));

или

var ex = "My favourite game";
console.log(ex.split(' ').map(function(item) { return item.split('').reverse().join(''); }).join(' '));

Comments

[Илья]

А как мне решить это в моем стиле?

[Vlad]

Илья, а чем этот код не устраивает? Две ваши функции или одна строчка.

[Илья]

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