Новоселова Наталья by NovoselovaNatalia · Pull Request #57 · urfu-2016/javascript-task-4 · GitHub

No description provided.

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

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

Прочитай гайды по оформлению, пожалуйста. Нельзя называть переменные транслитом

array -- плохое название переменной
оно не отражает сути

Ты тут возвращаешь значение, названия таких функций принято начинать со слова get

func тоже непонятно название.

copy? Что можно понять по переменной, которая называется копия?

Новая коллекция? Почему она новая? Чем отличается от старой?)

С copy та же самая история, что и выше

🍅

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

🍏

🚀

🍅

Object.assign -- это ES2015. Мы пока остаёмся в рамках ES5, так что эту функцию использовать нельзя

В качестве упражнения можете этот кусочек переделать на reduce, тогда можно будет переменную copyCollection не изменять. Но тут я не настаиваю.

sort ожидает, что компаратор возвращает число. А у вас возвращается логическое значение. И все значения, для которых условие не выполняются, преобразуются в 0, означающее, что значения равны. Так что функция сортировки корректно у вас работать не будет.

Object.assign нельзя использовать

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

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

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

первое решение

d7abc05

поправлен sortBy

046870b

поправлен query

26a375d

honest-hrundel assigned vetoshko Nov 2, 2016

vetoshko reviewed Nov 4, 2016

View reviewed changes

поправила названия переменных и функций

2d47144

honest-hrundel assigned hrundik and unassigned vetoshko Nov 6, 2016

hrundik requested changes Nov 7, 2016

View reviewed changes

update getCopyCollection, sortBy, format

331c0df

update getCopyCollection

d984d85

update sortBy

d5093d7

Update lego.js

ee4b074

Update lego.js

d228c8b

update format

7be0eba

update format

9d416d2

update sortBy

35e3d2e

               exports.query = function (collection) {
-                  return collection;
+                  var copy = copyCollection(collection);
+                  var func = [].slice.call(arguments, 1);

                */
               exports.query = function (collection) {
-                  return collection;
+                  var copy = copyCollection(collection);

               exports.sortBy = function (property, order) {
-                  console.info(property, order);
+                  return function sortBy(collection) {
+                      var newCollection = copyCollection(collection);

-                  console.info(property, formatter);
+                  return function format(collection) {
+                      return collection.map(function (element) {
+                          var copy = Object.assign({}, element);

+                  })
+                  .forEach (function (query) {
+                      copyCollection = query(copyCollection);
+                  });

-              exports.isStar = true;
+              exports.isStar = false;
+              var PRIORITET = ['filterIn', 'sortBy', 'select', 'limit', 'format'];


		var PRIORITET = ['filterIn', 'sortBy', 'select', 'limit', 'format'];

		function copyCollection(array) {

+              function getCopyCollection(collection) {
+                  return collection.map(function (element) {
+                      return Object.assign({}, element);

+                          }
-                  return;
+                          return first[property] < second[property];

Новоселова Наталья by NovoselovaNatalia · Pull Request #57 · urfu-2016/javascript-task-4 · GitHub

Uh oh!

Новоселова Наталья - #57

Новоселова Наталья#57NovoselovaNatalia wants to merge 12 commits intourfu-2016:masterurfu-2016/javascript-task-4:masterfrom NovoselovaNatalia:masterNovoselovaNatalia/javascript-task-4:masterCopy head branch name to clipboard

Conversation

NovoselovaNatalia 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!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

vetoshko Nov 4, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

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!

vetoshko commented Nov 4, 2016

Uh oh!

honest-hrundel commented Nov 5, 2016

Uh oh!

NovoselovaNatalia commented Nov 5, 2016

Uh oh!

vetoshko commented Nov 6, 2016

Uh oh!

hrundik left a comment

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!

honest-hrundel commented Nov 7, 2016

Uh oh!

honest-hrundel commented Nov 7, 2016

Uh oh!

honest-hrundel commented Nov 7, 2016

Uh oh!

honest-hrundel commented Nov 7, 2016

Uh oh!

honest-hrundel commented Nov 7, 2016

Uh oh!

honest-hrundel commented Nov 7, 2016

Uh oh!

honest-hrundel commented Nov 7, 2016

Uh oh!

honest-hrundel commented Nov 7, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Новоселова Наталья#57
NovoselovaNatalia wants to merge 12 commits into
urfu-2016:masterfrom
NovoselovaNatalia:master

vetoshko Nov 4, 2016 •
edited
Loading