Квашнина Ульяна by Uliana1997 · Pull Request #43 · urfu-2016/javascript-task-4 · GitHub

No description provided.

🍅 Пройдено тестов 10 из 15

sort

🍅 Пройдено тестов 10 из 15

sortby

🍅 Пройдено тестов 14 из 15

check

🍅 Пройдено тестов 14 из 15

format

🍅 Пройдено тестов 14 из 15

(

🍅 Пройдено тестов 14 из 15

last

🍅 Пройдено тестов 14 из 15

laaaast check

🍅 Пройдено тестов 10 из 15

check

🍏 Пройдено тестов 15 из 15

Привет! :) Я посмотрю твой пр сегодня вечером либо завтра, не теряй меня, если что.

Хорошо)

Отправлено с iPhone

3 нояб. 2016 г., в 11:29, Ekaterina Onufrienko notifications@github.com написал(а):

Привет! :) Я посмотрю твой пр сегодня вечером либо завтра, не теряй меня, если что.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Кавычки не нужны

fields или все же functions?

Эту функцию можно сильно сократить, возвращая, например, x - y

По названию можно подумать, что метод копирует несколько коллекций

у тебя получилась достаточно большая функция сортировки, ее можно сделать проще

Копипаст

попробуй метод filter

🍅

🍏 Пройдено тестов 15 из 15

Sort

🍏 Пройдено тестов 15 из 15

🍏

🚀

Зачем поудаляла все jsdoc?

мне с ними было неудобно, тут маленький код и так было понятно( мне их вернуть назад??

Их же можно сворачивать в IDE :) В данном случае, да маленький, но стоит привыкать с ним работать. Писать *doc – хороший тон.

Стоит написать комментарий, что значат эти цифры – больше число выполнится "быстрее" или в последнюю очередь

Не старайся усложнять там, где всё можно сделать проще. Когда функция сортировки достаточно простая, не стоит выносить её в отдельную функцию. К тому же её можно написать сильно короче и проще:

functions.sort(function (a, b) {
    return priority[a.name] - priority[b.name];
});

func стоит переименовать. Нейминг в данном случае ничего не говорит.

Смотри, ты здесь создаёшь массив, бегаешь в цикле, пушишь в результирующий массив. Всё это делает Array.reduce() (MDN), только более красиво.
Посмотри в его сторону. Это то, что тебе здесь нужно.

Ульяна, у тебя есть идеи как оптимизировать эту функцию?

есть, сейчас я исправлю всё

👍
Супер! Спасибо

🍅

🍅 Не пройден линтинг или базовые тесты

🍏 Пройдено тестов 15 из 15

🍏

Только я не знаю как заменить reduce, то что вы описываете делает вроде map, то есть мы создаем новый массив вызывая функцию в нем
А reduce применяет функцию и сводит к одному значению, или я не так что то понимаю??

Итоговым значением может же быть и массив. Вот, прочитай, тут про reduce.

🍏 Пройдено тестов 15 из 15

🍏

исправила все замечания

Ульяна, не все комментарии учтены.
🍅

🍅 Не пройден линтинг или базовые тесты

🍏 Пройдено тестов 15 из 15

🍏

Метод sort – не чистая функция. Только что ты мутировала коллекцию. Требования задания не учтены.

🍅

⬆️ @Uliana1997

First

63da3af

honest-hrundel changed the title First Квашнина Ульяна Nov 2, 2016

Квашнина Ульяна

38a3b74

sort

Квашнина Ульяна

b03ad97

sortby

Квашнина Ульяна

d940bae

check

Квашнина Ульяна

4dbb289

format

Квашнина Ульяна

bbc6609

(

Квашнина Ульяна

5fed378

last

Квашнина Ульяна

69b2a4a

laaaast check

Квашнина ульяна

e4743a7

check

honest-hrundel assigned onufrienko Nov 2, 2016

onufrienko reviewed Nov 4, 2016

View reviewed changes

Квашнина Ульяна

c980067

Квашнина Ульяна

319c9fc

Sort

honest-hrundel assigned evilj0e and unassigned onufrienko Nov 8, 2016

evilj0e reviewed Nov 9, 2016

View reviewed changes

Квашнина Ульяна

f1b443b

Квашнина Ульяна

b0bb7b4

Квашнина Ульяна

799aac8

Квашнина Ульяна

80da510

Квашнина Ульяна

f0e810c

evilj0e reviewed Nov 15, 2016

View reviewed changes

               exports.query = function (collection) {
-                  return collection;
+                  var newCollection = copyCollections(collection);
+                  var fields = Array.prototype.slice.call(arguments).slice(1);

-                  return collection;
+                  var newCollection = copyCollections(collection);
+                  var fields = Array.prototype.slice.call(arguments).slice(1);
+                  function compare(one, another) {

-               * Выбор полей
-               * @params {...String}
-               */
+              function copyCollections(collection) {

-               */
               exports.sortBy = function (property, order) {
-                  console.info(property, order);
+                  return function sortBy(collection) {

+                              collection.sort(function (one, another) {
+                                  var x = one.name;
+                                  var y = another.name;
+                                  if (x < y) {

+              exports.isStar = false;
+              var priority = {
+                  'filterIn': 1,

+              };
+              exports.filterIn = function (property, values) {
+                  return function filterIn(collection) {

+                  return newCollection;
               };
-              /**

+                      return x - y;
+                  }
+                  functions.sort(compare);

-                  console.info(property, values);
+                  return function select(collection) {
+                      var result = [];
+                      collection.forEach(function (friend) {

-                  return;
+                      if (order === 'asc') {
+                          collection.sort(compare);

Квашнина Ульяна by Uliana1997 · Pull Request #43 · urfu-2016/javascript-task-4 · GitHub

Uh oh!

Квашнина Ульяна - #43

Квашнина Ульяна#43Uliana1997 wants to merge 18 commits intourfu-2016:masterurfu-2016/javascript-task-4:masterfrom Uliana1997:masterUliana1997/javascript-task-4:masterCopy head branch name to clipboard

Conversation

Uliana1997 commented Nov 2, 2016

Uh oh!

honest-hrundel commented Nov 2, 2016

Uh oh!

honest-hrundel commented Nov 2, 2016

Uh oh!

honest-hrundel commented Nov 2, 2016

Uh oh!

honest-hrundel commented Nov 2, 2016

Uh oh!

honest-hrundel commented Nov 2, 2016

Uh oh!

honest-hrundel commented Nov 2, 2016

Uh oh!

honest-hrundel commented Nov 2, 2016

Uh oh!

honest-hrundel commented Nov 2, 2016

Uh oh!

honest-hrundel commented Nov 2, 2016

Uh oh!

onufrienko commented Nov 3, 2016

Uh oh!

Uliana1997 commented Nov 3, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

onufrienko commented Nov 4, 2016

Uh oh!

honest-hrundel commented Nov 6, 2016

Uh oh!

honest-hrundel commented Nov 6, 2016

Uh oh!

Uliana1997 commented Nov 6, 2016

Uh oh!

onufrienko commented Nov 8, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

evilj0e commented Nov 9, 2016

Uh oh!

honest-hrundel commented Nov 10, 2016

Uh oh!

honest-hrundel commented Nov 10, 2016

Uh oh!

Uliana1997 commented Nov 10, 2016

Квашнина Ульяна#43
Uliana1997 wants to merge 18 commits into
urfu-2016:masterfrom
Uliana1997:master