Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Дз4 #7

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Дз4 #7

wants to merge 9 commits into from

Conversation

Yunnii
Copy link

@Yunnii Yunnii commented Nov 1, 2012

Только мне кажется, с фуункциями sort для коллекций я полный хардкор устроила... может как-то по-аккуратнее можно?

Создан конструктор + методы для абстрактного объекта model
Класс Event унаследовался от Model
Написаны тесты на 1 и 2 пункт
1. добавлены новые тесты, выделены в отделный каталог
2. создан конструктор + фильтр + фции для коллекции
1. допилен 4 пункт
2. остался jslint + jsdoc
return this.coming()
.filter(function (events) {
return events.start < now;
});
Copy link
Member

Choose a reason for hiding this comment

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

Вот так более читаемо

var result = this.coming()
    .filter(function (events) {
        return events.start < now;
    });

return result;

@azproduction
Copy link
Member

Только мне кажется, с фуункциями sort для коллекций я полный хардкор устроила... может как-то по-аккуратнее можно?

про форматирование отписал

@Yunnii
Copy link
Author

Yunnii commented Nov 2, 2012

в коллекции Events, когда вызывается, например фция

Events.prototype.byEndTime = function () {

return this.sort(function (a, b) {
    return a.end - b.end;
});

};

идет вызов функции sort, которая возвращает объект типа Collection, это ведь не верно? должно Events возвращать?

@azproduction
Copy link
Member

должно Events возвращать?

Да, должно возвращать инстанс текущего конструктора. Можешь где-нибудь в прототипе сохранить ссылку "на себя" (prototype.constructor = Events), а в функции sort использовать.

@@ -0,0 +1,81 @@
((function (exports) {
Copy link
Member

Choose a reason for hiding this comment

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

Лишняя ( скобка.

@azproduction
Copy link
Member

Последний коммит все испортил. Я более чем уверен, что у тебя ничего не работает ;-)

@Yunnii
Copy link
Author

Yunnii commented Nov 2, 2012

меня опечалило, что фции возвращают совсем не то, что надо и я даже тесты не прогнала. позор мне)

@azproduction
Copy link
Member

Вот так и познается вся мощь автоматических юнит-тестов!

@azproduction
Copy link
Member

Если интересно попробуй начать использовать Qunit PhantomJS Runner https://github.com/jquery/qunit/tree/master/addons/phantomjs с TravisCI ;-)

1. Ошибули поправлены
2. Тесты работают
3. Фции возвращают для Collection - новую коллекцию типа Collection,
для Events - типа Events
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants