-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
Наред:
Повтаряне на код
Двете анонимни функции в кода за сортирането се повтарят, така че е добра идея да ги изнесеш в един var:
var numericSort = function(a, b) { return b - a ;}И да дадеш тази функция за сортиране.
Всички променливи на един var
Двете променливи най-горе ги изкарай на 1 var.
Правилото е, че се опитваме да имаме по 1 var на scope (на функция)
Малко оптимизация
В това парче код:
return beer.map(function(score){
var index = beer.indexOf(score);
return score * fries[index];
}).reduce(function(a,b){
return a + b;
},0);Като цяло не ми изглежда особено работещо, въпреки, че минава теста.
Може да се възползваш от факта, че map дава като втори аргуемнт и index
и да стане така:
return beer.map(function(score, index){
return score * fries[index];
}).reduce(function(a,b){
return a + b;
},0);Другото е супер 👍
Metadata
Metadata
Assignees
Labels
No labels