-
Notifications
You must be signed in to change notification settings - Fork 11
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
Task 3 #2
base: master
Are you sure you want to change the base?
Task 3 #2
Conversation
1. Написал еще 10 событий 2. Реализовал хранилище событий ^^ некоторая база данны� 3. Реализовал несколько функций для работы с хранилищем: 1.pastEventBase() - создать бд с пропущенными событиями 2.nextEventBase() - создать бд с грядущими событиями 3.nowEventBase() - создать бд с происходящими событиями 4.withFriend(friend) - создать бд с событиями, в которых принимал учисатия друг 5.getEventAfterMontrh() - создать бд с событиями через месяц 6. getEventAfterDay() - - создать бд с событиями через день 7. getEventAfterWeek() - - создать бд с событиями через неделю 8. sortByStar() - создать бд с отсортированными событиями по количеству звезд 9. sortByDate() - создать бд с отсортированными событиями по дате 3. Реализовал функциональный синтаксис ^^ 4. Написал к каждой функции по тесту Осталось: Пройти проверку на JSLint, JSDoc
Исключение функция initTestBase - cодержит только данные.
1) getEventFromPeriod - протестировал. 2) Задокументировал JSDoc.
Да, будет |
Ну ты и понаписал :D |
Попробуй в следующий раз не включать QUnit и jQuery - подключай их через сабмодули |
*/ | ||
function Event(start, end, location, id) { | ||
"use strict"; | ||
var dateValidator = function (date) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Вынеси dateValidator отдельно (Event.dateValidator) проблема в том, что при каждом вызове Event будет создаваться эта функция.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
т. е. вынести в свойство функции Event
function Event (start, end, location, id) {
.............
Event.dateValidator(..);
.............
}
Event.dateValidator = new function (...) {....};
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Да
Вроде сделал. Жду критики ^^
И за одно у меня есть вопрос:
Я опоздал со сроком к Task 2. Будет ли возможность по ней услышать комментарии?