正如@VLAZ在评论中提到的，你的string.length大于字符串的实际长度，这就是为什么它会跳出数组。
有很多方法可以消除字符串中的undefined。一个是VLAZ所说的，它只是长度。另一个是在串联上使用空合并运算符。以下样品

const reverseString1 = function(string) {
    let newWord = "";
    for (i = string.length - 1; i > -1; i--) {
        newWord = newWord.concat(string[i]);
    }
    return newWord
};

const reverseString2 = function(string) {
    let newWord = "";
    for (i = string.length; i > -1; i--) {
        newWord = newWord.concat(string[i] ?? "");
    }
    return newWord
};

console.log(reverseString1("Hello1"));
console.log(reverseString2("Hello2"));

索引应该从（n-1）开始，而不是从n开始

const reverseString = function(string) {
    let newWord = "";
    for (i = string.length - 1; i > -1; i--) {
        newWord = newWord.concat(string[i]);
    }
    return newWord
};