Упростить код условия

Question

Есть код, который в зависимости от значения или заставляет мигать иконку, или скрывает её. Помогите упростить и более правильнее и лаконичнее переписать его

if (_belt_left) {
    if (belt_left.getVisibility() == View.VISIBLE) {
        belt_left.setVisibility(View.INVISIBLE);
    } else {
        belt_left.setVisibility(View.VISIBLE);
    }
} else {
    if (belt_left.getVisibility() == View.VISIBLE || belt_left.getVisibility() == View.INVISIBLE) {
        belt_left.setVisibility(View.GONE);
    }
}

Answer 1

Нашёл вариант более ёмкий

if (_belt_left) {
    belt_left.setVisibility(belt_left.getVisibility() == View.VISIBLE ? View.INVISIBLE : View.VISIBLE);
} else {
    if (belt_left.getVisibility() == View.VISIBLE || belt_left.getVisibility() == View.INVISIBLE) {
        belt_left.setVisibility(View.GONE);
    }
}

Answer 2

Я бы написал так

if (_belt_left) {
    if (belt_left.getVisibility() == View.VISIBLE) {
        belt_left.setVisibility(View.INVISIBLE);
    } else {
        belt_left.setVisibility(View.VISIBLE);
    }
} else {
    belt_left.setVisibility(View.GONE);
}

void setVisibility(View view) {
  if (view != this.view) {
    ........
  }
}

Answer 3

Если всего три состояния, то

belt_left.getVisibility() == View.VISIBLE || belt_left.getVisibility() == View.INVISIBLE

равнозначно

belt_left.getVisibility() != View.GONE

БЛОГ НА HUSL

Упростить код условия

Ответы (3 шт):