Skip to content

Issues за beersAndFries #1

@RadoRado

Description

@RadoRado

Наред:

Повтаряне на код

Двете анонимни функции в кода за сортирането се повтарят, така че е добра идея да ги изнесеш в един 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

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions