Переключение между двумя атрибутами?

Question

[Сергей]

Можно ли как-то сократить данный код или это наиболее верный и оптимальный вариант записи?

if (this.getAttribute('aria-expanded') == 'false') {
  this.setAttribute('aria-expanded', 'true');
} else {
  this.setAttribute('aria-expanded', 'false');
}

То есть по сути надо что-то типа toggle, но между двумя атрибутами.

Answer 1

[0xD34F]

this.setAttribute('aria-expanded', this.getAttribute('aria-expanded') === 'false');

Comments

[Сергей]

Не перестаю восхищаться вашим знанием Javascript.

[Ярослав Иванов]

а разве без const будет работать ?

[Модератор]

Ярослав Иванов, если у вас есть сомнения по использованию const, задайте вопрос.
если вы продолжите бегать по комментам, я буду вынужден ограничить ваше присутствие на ресурсе.

[Ярослав Иванов]

Модератор, Чтобы вы потом его удалили за "невозможно дать ответ"?

[Модератор]

Ярослав Иванов, вы уже создавали и его удалили?

[Ярослав Иванов]

Модератор, нет, я просто знаю наперед, что будет. Вы мои и другие удаляли.

[Модератор]

Ярослав Иванов, у меня нет ни малейшего желания переубеждать вас.
Вы можете:
- задать свой вопрос касательно значимости и целесообразности использования const в js
- не задавать, но прекратить бегать по коментам 0xD34F с этим вопросом
- не задавать и продолжить бегать по коментам 0xD34F с этим вопросом с предсказуемым итогом

Какой из этих вариантов выбрать решайте сами.

[Ярослав Иванов]

Модератор, А кому мешают мои комменты ? это ваш аккаунт 0xD34F ?

[DevMan]

Ярослав Иванов,

это ваш аккаунт 0xD34F ?

нет.

А кому мешают мои комменты ?

мешают не коменты, а их регулярность в ответах этого юзера.

Answer 2

[Dima Polos]

Можно вместо таких проверок тернарный оператор использовать.