Хотим exception, когда нет параметров нужных для ключа модели

[chestozo]

ns.Model.define('ququ', {
    params: {
        'id': null
    }
});

// Тут всё ок.
ns.Model.get('ququ', { id: 1 });
// { key: "model=ququ&id=1" ... }

// А вот тут хочется exception (потому что не хватает параметра id для ключа):
ns.Model.get('ququ', {});
// { key: "model=ququ" ... }

Получается, параметров для ключа не хватает, но ключ строится и вообще всё хорошо. Мне кажется, где-то это обсуждалось уже, но я не помню где и какой итог.

Всем кажется, что это ок? /cc @i2r

[doochik]

Да, обсуждалось. Честно пытался найти таск и не нашел Тогда я был против, потому что у меня есть модели с кучей необязательных параметров. Надо придумать синтаксис для обязательности. Я готов это сделать, мне бы тоже пригодилось :)

[doochik]

Нашел! #452

[doochik]

Предлагаю =. Его хорошо видно, он отличается от других и символизирует "присвоение", т.е. наличие значение

ns.Model.define('ququ', {
    params: {
        'id=': null
    }
});

Если я правильно помню, параметры у нас разбираются в объект, поэтому там можно сделать флаг

[chestozo]

= означает только "опциональные параметры" в роутере https://yandex-ui.github.io/noscript/single-page/#index_md_14

[chestozo]

Мои варианты:

params: {
    '!id': null, // аналогия из https://developers.google.com/closure/compiler/docs/js-for-compiler?hl=ru
    'id': ns.paramType.required // аналогия из http://facebook.github.io/react/docs/reusable-components.html#prop-validation
}

[chestozo]

А ещё вариант по дефолту делать параметры обязательными, а для опциональных придумать обозначение.

[doochik]

Мне ! принципиально не нравится, потому что его плохо видно, он может слится с l или i

[alexeyten]

А почему бы необязательным параметрам не задать дефолтное значение ''? А все остальные параметры сделать обязательными?

[chestozo]

А почему бы необязательным параметрам не задать дефолтное значение ''? А все остальные параметры сделать обязательными?

Раньше было что-то такое для обозначения обязательности:

params: {
    p1: true, // обязательный
    p2: false // необязательный
}

оно чего-то не прижилось. Плюс начнётся всякое '' == 0 и т.п.

А чем не нравится, к примеру, 'id': ns.paramType.required? Можно будет накрутить валидаторы параметров потом, к примеру.

[doochik]

Мне нравится идея @alexeyten. Просто и понятно.

[chestozo]

Т.е. поумолчанию параметры обязательны, а для опциональных указываем p: ''? Мне тоже ок. @edoroshenko ?

[alexeyten]

Не обязательно ''. Любое разумное значение. Может 0, может 'default' и т.п.

[chestozo]

Ну оно и сейчас так работает, да. Просто сейчас остальные параметры опциональные, а хочется обязательные.

[edoroshenko]

+1 к @alexeyten

[doochik]

Итого: Если параметр в декларации null - обязательное поле Любое другое значение - необязательное

Все правильно?

[edoroshenko]

да

[chestozo]

Итого: Если параметр в декларации null - обязательное поле Любое другое значение - необязательное Все правильно?

Т.е. прямо value === null будем проверять?

[doochik]

Да