Чистый внутренний класс, реализующий Runnable

Question

У меня есть класс для CustomView, который имеет внутренние классы, и реализовать Runnable для выполнения задания в отдельном потоке.

public class ValueSelector extends LinearLayout{ 

..... 

private class AutoIncrementer implements Runnable { 

    @Override 
    public void run() { 
     if (plusButtonIsPressed) { 
      incrementValue(); 
      mHandler.postDelayed(new AutoIncrementer(), REPEAT_INTERVAL_MS); 
     } else { 
      mHandler.removeCallbacks(this); 
      Thread.currentThread().interrupt(); 
     } 
    } 
} 

private class AutoDecrementer implements Runnable { 
    @Override 
    public void run() { 
     if (minusButtonIsPressed) { 
      decrementValue(); 
      mHandler.postDelayed(new AutoDecrementer(), REPEAT_INTERVAL_MS); 
     } else { 
      mHandler.removeCallbacks(this); 
      Thread.currentThread().interrupt(); 
     } 
    } 
} 
} 

Как правильно их очистить? Получают ли они Разрушенные автоматически, когда действие, в котором находятся эти CustomViews, будет уничтожено?

Приветствия

Answer 1

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

Сделать внутренний класс статическим и удерживать слабую ссылку на переменные, которые вам нужны в методе запуска.

Вторая вещь, которую вы можете сделать, это прервать вашу нить, когда вы просматриваете, отсоединяться от окна и проверять метод запуска, если поток был прерван или нет, но не нужен, если ваш поток не слишком много работает.

Вот что ваш работоспособным должен выглядеть

private static class AutoDecrementer implements Runnable { 

    AutoDecrementer (ValueSelector valueSelector){ 
     this.weakRef = new WeakReference<>(valueSelector); 
    }  


    @Override 
    public void run() { 
     ValueSelector valueSelector = (ValueSelector)weakRef.get(); 
     if(valueSelector == null){ 
      return ; 
      } 

     if (valueSelector.minusButtonIsPressed) { 
      valueSelector .decrementValue(); 
      valueSelector .mHandler.postDelayed(new AutoDecrementer(), REPEAT_INTERVAL_MS); 
     } else { 
      valueSelector.mHandler.removeCallbacks(this); 
      Thread.currentThread().interrupt(); 
     } 
    } 
} 

Я не проверяется на наличие ошибок.

Answer 2

Нет, это вызовет ошибку, если активность будет уничтожен во время события таймера еще не завершено. Чтобы избежать этого, используйте WeakReference для некоторого объекта, уменьшающего значение. Но, как правило, это плохая практика - смешивать UI и некоторые лигические, потому что это сложно проверить. Рассмотрите возможность использования библиотеки rxJava, это будет выглядеть

Subscriptioin s = Observable.just(100, TimeUnit.Milliseconds) 
.subscribeOn(Schedulers.io()) 
.observeOn(AndroidSchedulers.computation()) 
.subscribe(t -> decrementValue()); 

в вашем методе OnPause() Отмена что actioin по

if (s != null && !s.inUnsubscribed()) { 
s.unsubscribe(); 
s = null; 
}