Skip to content

dz-2-basics #2

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

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

dz-2-basics #2

wants to merge 2 commits into from

Conversation

axlerk
Copy link

@axlerk axlerk commented Oct 14, 2012

Перешёл на пробельную табуляцию, т.к. она используется в JSDoc. Можно ли
перемешивать пробелы в комментариях и табы в коде? Валидатор съедает, но неединообразно же.

Перешёл на пробельную табуляцию, т.к. она используется в JSDoc. Можно ли
перемешивать пробелы в комментариях и табы в коде? Валидатор съедает, но
как-то неединообразно...
@azproduction
Copy link
Member

Не нужно мешать. В JSDoc можно тоже табы заюзать (только все может поехать)

*
* @return {Object}
*/
function Event(name, start, end, allDay, location, withPeople, repeat, stars, color, uri, notes) {
Copy link
Member

Choose a reason for hiding this comment

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

А если аргументов будет 50? Подумай как можно сделать нагляднее.

@axlerk
Copy link
Author

axlerk commented Oct 14, 2012

Да, дело именно в том, что всё едет: там всё подогнано пробелами, поэтому их и оставил :)

@axlerk
Copy link
Author

axlerk commented Oct 15, 2012

А можно намекнуть в какую сторону думать?
Мне сейчас в голову приходит вариант получения каждой формы и проход if else, пример для рейтинга:

if (document.form.stars.value === "") {
    document.form.stars.value = 0;
} else {
    document.form.stars.value = +document.form.stars.value;
}

Но я как-то не уверен, что это то, что требуется. :(

@azproduction
Copy link
Member

/**
 * @param {Object} eventData
 * @param {String} eventData.name
 * ...
 */
function Event(eventData) {
    eventData = eventData || {};
    eventData.name = eventData.name || 'Default name';
    // ...
}

Event({
    name: name.
    start: start,
    ...
});

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

@axlerk
Copy link
Author

axlerk commented Oct 16, 2012

Т.е. функция проверяет, есть ли что-то в eventData, а в Event лежит список объектов-аргументов, там же делается приведение к типу (то, что раньше происходило в return)? Вызывая функцию, мы передаём в неё eventData.name и так далее в любом порядке? Например: function Event(eventData.name, eventData.start) и так далее?

P.S. И ведь пока думал вчера, дошёл до мысли, что в аргументе нужно что-то вроде "Data.*", а уже в звёздочку передавать name, start, которые будут выписаны отдельно. Но не знал как это записывается. :)

@azproduction
Copy link
Member

В функцию будет передаваться уже готовый объект - фактически функция просто валидирует поля

@azproduction azproduction mentioned this pull request Oct 18, 2012
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