Является ли определение значения по умолчанию для логического входного свойства в Angular плохой практикой?

Question

Я просматриваю старый проект Angular, и у одного компонента есть свойство ввода:

@Input()
isNotAdmin = true; 

Является ли непосредственное присвоение значения свойству в его объявлении, как это считается плохой практикой?Я понимаю, что это зависит от контекста, например, если мы хотим, чтобы у каждого из этих компонентов было установлено для этого свойства значение true, если не указано, но мне кажется, что логические свойства должны быть явно указаны в любом случае (true или false).Должен ли подход вместо этого иметь метод установки в качестве входных данных и проверить, передано ли значение, а если нет, назначить свойству значение по умолчанию?

Answer 1

Нет, это не плохая практика.

На самом деле, это было бы даже лучшей практикой.

Причина в том, что входы устанавливаются только при ngOnChanges прогонах, что означает, что до тех пор, пока эта функция не будет запущена, ваше значение не определено.

Для логического значения это не имеет большого значения. Но представьте себе сложный объект, и вы выполняете операции над этим объектом, что-то вроде

if (this.myInput.type.ref.date.modifiedAt.includes('+1'))

Тогда ваше состояние выдаст ошибку.

Более того, это позволяет вводить данные без значений: представьте, что вы создали пользовательский выбор, и вы хотите назначить вход multiple: каков наилучший подход для разработчика?

Personnaly, мне нравится первый. Чтобы достичь этого, вы можете использовать это:

@Input('multiple') doNotShowMultipleValues = true;

Когда разработчик будет использовать первый синтаксис, значение станет '', что неверно, что инвертирует условие. То, что вы не можете обойтись без первоначального значения!

Но, в конце концов, все зависит от личных предпочтений, и если вы не уверены, что можете присвоить значение по умолчанию, вам просто не следует этого делать.

Answer 2

Если вы хотите указать значение по умолчанию, если ввод не связан, то, что вы написали, является правильным подходом.

Однако, если вы вместо этого намереваетесь предоставить значение по умолчанию, если значение привязки равно null или undefined, лучше использовать установщик:

private _isNotAdmin = true;

@Input()
set isNotAdmin(value: boolean) {
    if (value != null) {
        this._isNotAdmin = value;
    } else {
        this._isNotAdmin = true;
    }
}

get isNotAdmin(): boolean {
    return this._isNotAdmin;
}

Answer 3

Да, это плохо

Там вы можете управлять любым свойством ввода и реагировать соответственно