2015-12-30 40 views
0

我试图重新创建reduce方法,但是作为原始函数。重新创建JavaScript的缩小函数

这帮助我了解了很多,但我有一个小问题: How to re-create Underscore.js _.reduce method? 当我插上阵列它的作品的权利,但对象不工作。我弄不明白。对于我应该改变的内容,我将不胜感激。我评论了我认为应该改变的地方,但我很困惑我需要改变它。我也创建了我自己的每个功能。

你可以在这里找到我的代码:http://goo.gl/6RU9Bc

任何意见将是超级有用。谢谢!!

这是我到目前为止有:

var myArray=[1,2,3,4,5]; 

var myObject={ 
    num1:1, 
    num2:2, 
    num3:3 
}; 

function each(collection, callback){ 
    if(Array.isArray(collection)) { 
    for(var i=0, l=collection.length; i<l; i++){ 
     callback(collection[i]); 
    }; 
    }else if(collection === "object") { 
    for(var prop in collection){ 
     callback(collection[prop]); 
    }; 
    } 
} 

function multiply(num, num2){ 
    return num*num2; 
} 

function reduce(collection, callback, accumulator){ 
    each(collection, function(element){ 
    if(accumulator === undefined) { 
     return accumulator = element; // is the problem here? Why? I don't understand. 
    }else { 
     return accumulator = callback(accumulator, element); 
    }; 
}); 

return accumulator; 
}; 

console.log(reduce(myArray, multiply)); // 120 
console.log(reduce(myArray, multiply, 5)); // 160 
console.log(reduce(myObject, multiply)); // returns undefined 
console.log(reduce(myObject, multiply, 5)); // returns 5 
+0

FWIW,MDN有[可用polyfill](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/Reduce#Polyfill) - 可能能够获得一些想法。 –

+3

'if(collection ===“object”)'永远不会是真的。它不应该是'typeof(collection)==='object')'? –

+0

是的,你完全正确。有人告诉我,当我做每个功能时都会选择这种类型,因为它不是必需的。谢谢! – Masha

回答

0

正如@Yeldar Kurmangaliyev指出的那样,你需要改变这一点:

if(collection === "object") 

到:

if(typeof(collection) === "object") 

这里是固定代码:

var myArray=[1,2,3,4,5]; 

var myObject={ 
    num1:1, 
    num2:2, 
    num3:3 
}; 

function each(collection, callback){ 
    if(Array.isArray(collection)) { 
    for(var i=0, l=collection.length; i<l; i++){ 
     callback(collection[i]); 
    }; 
    }else if(typeof(collection) === "object") { 
    for(var prop in collection){ 
     callback(collection[prop]); 
    }; 
    } 
} 

function multiply(num, num2){ 
    return num*num2; 
} 

function reduce(collection, callback, accumulator){ 
    each(collection, function(element){ 
    if(accumulator === undefined) { 
     return accumulator = element; // is the problem here? Why? I don't understand. 
    }else { 
     return accumulator = callback(accumulator, element); 
    }; 
}); 

return accumulator; 
}; 

console.log(reduce(myArray, multiply)); // 120 
console.log(reduce(myArray, multiply, 5)); // 600 
console.log(reduce(myObject, multiply)); // 6 
console.log(reduce(myObject, multiply, 5)); // 30