这样使用forEach不会做任何事情-使用map代替。

let numbersArr = n.split("").map(num => {
  if (Number(num) > 5) {
    num = "0";
  } else if (Number(num) <= 5) {
    num = "1";
  }
  return num;
});

return numbersArr.join("");

请注意，要生成所需的输出，您需要稍微更改条件：

if (Number(num) >= 5) {
  num = "1";
} else {
  num = "0";
}

forEach不会带来元素对原始值的引用，而是带来值的副本。您可以轻松地通过索引手动访问它：

function fakeBinary(n) {
    let numbersArr = n.split('');
    numbersArr.forEach((num, i) => {
//                          ^--- note that `i` is brought and used below to access the element at index [i].
        if(Number(num) < 5) {
            numbersArr[i] = '0';
        } else if(Number(num) >= 5) {
            numbersArr[i] = '1';
        }
    });
    return numbersArr.join('');
}


console.log(fakeBinary('3457'));

请注意，您也可以使用其他原型，我只是尽可能地接近您的解决方案，您可能还希望使用map，甚至（尽管不合适）reduce，甚至常规的for循环。

要使用forEach来实现这一点，您需要使用额外的参数来引用数组和索引。

function fakeBinary(n) {
    let numbersArr = n.split('');
    numbersArr.forEach((num, index, arr) => {
        if(Number(num) < 5) {
            arr[index] = '0';
        } else if(Number(num) >= 5) {
            arr[index] = '1';
        }
    });
    return numbersArr.join('');
}

console.log(fakeBinary('3457'));

但是forEach不是返回一个新数组的正确方法。

function fakeBinary(n) {
    return n
      .split('')
      .map(num => (Number(num) < 5) ? '0' : '1')
      .join('');
}

console.log(fakeBinary('3457'));

使用Array.from()的较短版本

const fakeBinary = (str) => Array.from(str, n => +(+n >= 5)).join('');

console.log(fakeBinary('3457'));