А у меня вопрос по красивому коду. Как вы относитесь

я за семантичность, коли назвался тогглом, то переключай из одного в другое для open/close - обычно отдельные ивенты юзаю (потом в случае чего, проще найти концы)

0

19.04.2020

Vadim Ivanov

Думаю должно быть без параметров. Иначе это более низкоуровневый setState(boolean)

0

19.04.2020

Ruslan

крайне отрицательно... лучше сделать toggle() turnOn() turnOff() toggle(false) - это говнокод классический

0

19.04.2020

Nikita Gritsai

Нормальный лаконичный код, toggle и так работает как нужно без аргументов. Имеет право на жизнь.