因为你改的是参数值啊，而且你可以把参数值给重写覆盖掉了。为什么test数据没有改变，请参阅这个：js 中参数是按值传递的，但声明对象时又是引用指向某个对象的，该怎么理解？

代码修正，很简单，把你的forEach换成map就行了，而且注意你的原代码里item既是参数，又被你var了一次，小小的细节请注意，coding坏习惯要改：

function solution(input, markers){
    var test= input.split("\n");
    var len=markers.length ,strlen=test.length;
    for(var i=0;i<len;i++){
        test = test.map(function(item){
            return item.split(markers[i])[0];
        });
    }
    return test;
}
console.log(solution("apples, pears # and bananas\ngrapes\nbananas !apples",["#","!"]))//["apples, pears ", "grapes", "bananas "]