Search code, repositories, users, issues, pull requests...

@NovoselovaNatalia NovoselovaNatalia commented Nov 2, 2016

Copy link

Copy Markdown

No description provided.


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

d7abc05

honest-hrundel commented Nov 2, 2016

Copy link

Copy Markdown

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


 поправлен sortBy

046870b

honest-hrundel commented Nov 2, 2016

Copy link

Copy Markdown

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


 поправлен query

26a375d

honest-hrundel commented Nov 2, 2016

Copy link

Copy Markdown

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

@honest-hrundel honest-hrundel assigned vetoshko

Nov 2, 2016

vetoshko

vetoshko reviewed

lego.js Outdated

exports.isStar = true;

exports.isStar = false;

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

@vetoshko vetoshko Nov 4, 2016

Copy link

Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

vetoshko

vetoshko reviewed

lego.js Outdated

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

function copyCollection(array) {

@vetoshko vetoshko Nov 4, 2016

Copy link

Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

vetoshko

vetoshko reviewed

lego.js Outdated

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

function copyCollection(array) {

@vetoshko vetoshko Nov 4, 2016 •

edited

Loading

Copy link

Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

vetoshko

vetoshko reviewed

lego.js Outdated

exports.query = function (collection) {

return collection;

var copy = copyCollection(collection);

var func = [].slice.call(arguments, 1);

@vetoshko vetoshko Nov 4, 2016

Copy link

Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

vetoshko

vetoshko reviewed

lego.js Outdated

exports.query = function (collection) {

return collection;

var copy = copyCollection(collection);

@vetoshko vetoshko Nov 4, 2016

Copy link

Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

vetoshko

vetoshko reviewed

lego.js Outdated

exports.sortBy = function (property, order) {

console.info(property, order);

return function sortBy(collection) {

var newCollection = copyCollection(collection);

@vetoshko vetoshko Nov 4, 2016

Copy link

Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

vetoshko

vetoshko reviewed

lego.js Outdated

console.info(property, formatter);

return function format(collection) {

return collection.map(function (element) {

var copy = Object.assign({}, element);

@vetoshko vetoshko Nov 4, 2016

Copy link

Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@vetoshko

vetoshko commented Nov 4, 2016

Copy link

Copy Markdown

🍅


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

2d47144

honest-hrundel commented Nov 5, 2016

Copy link

Copy Markdown

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

NovoselovaNatalia commented Nov 5, 2016

Copy link

Copy Markdown

Author

🍏

@vetoshko

vetoshko commented Nov 6, 2016

Copy link

Copy Markdown

🚀

@honest-hrundel honest-hrundel assigned hrundik and unassigned vetoshko

Nov 6, 2016

hrundik

hrundik requested changes

Nov 7, 2016

@hrundik hrundik left a comment

Copy link

Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍅

lego.js Outdated

function getCopyCollection(collection) {

return collection.map(function (element) {

return Object.assign({}, element);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

lego.js

})

.forEach (function (query) {

copyCollection = query(copyCollection);

});

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

lego.js Outdated

}

return;

return first[property] < second[property];

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

lego.js Outdated

console.info(property, formatter);

return function format(collection) {

return collection.map(function (element) {

var copyCollection = Object.assign({}, element);

@hrundik hrundik Nov 7, 2016

Copy link

Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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


 update getCopyCollection, sortBy, format

331c0df

honest-hrundel commented Nov 7, 2016

Copy link

Copy Markdown

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


 update getCopyCollection

d984d85

honest-hrundel commented Nov 7, 2016

Copy link

Copy Markdown

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


 update sortBy

d5093d7

honest-hrundel commented Nov 7, 2016

Copy link

Copy Markdown

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


 Update lego.js

ee4b074

honest-hrundel commented Nov 7, 2016

Copy link

Copy Markdown

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


 Update lego.js

d228c8b

honest-hrundel commented Nov 7, 2016

Copy link

Copy Markdown

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


 update format

7be0eba

honest-hrundel commented Nov 7, 2016

Copy link

Copy Markdown

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


 update format

9d416d2

honest-hrundel commented Nov 7, 2016

Copy link

Copy Markdown

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


 update sortBy

35e3d2e