Skip to content

Issues за beersAndFries #1

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);

Другото е супер 👍

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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