Какой из этих двух примеров лучший дизайн кода?

Question

Пример # 1

public void mouseClicked(MouseEvent e) {
    if (isShowParameters)
       isShowParameters = false;
    else
       isShowParameters = true;

    ParametersWindows.getInstance().setVisible(isShowParameters);
}  

Пример № 2

public void mouseClicked(MouseEvent e) {
    isShowParameters = !isShowParameters;
    ParametersWindows.getInstance().setVisible(isShowParameters);
}

Answer 1

Второй более понятный и логичный.

Answer 2

Я предпочитаю isShowParameters = !isShowParameters;, потому что он короче.Более короткие программы легче понять и поддерживать.Но что с твоим огромным скоплением непревзойденных правых скобок?Это похоже на плохой стиль кодирования.

Answer 3

Существует третий вариант:

isShowParameters ^= true;

или более компактно:

ParametersWindows.getInstance().setVisible(isShowParameters ^= true);

Answer 4

ParametersWindows.getInstance().setVisible(!isShowParameters);

;)

Обновление

Работает, только если вам не нужно обновлять само поле.