LiveData и ViewModel не обновляют текстовое представление

Question

У меня небольшая проблема, когда я закомментирую binding.setData (dataContainer); в onChanged при просмотре Livedata в своей активности, я не могу обновить Пользовательский интерфейс, , когда я раскомментирую его, пользовательский интерфейс обновляется. Пожалуйста, ознакомьтесь и сделайте небольшой обзор кода, если вы чувствуете необходимость.

У меня есть Runnable, который запускается несколько раз через каждые x секунд. Это сделано Livedata, который я наблюдаю в своей деятельности.

Спасибо.

public class MainActivity extends AppCompatActivity {

  TextOnScreen dataContainer;
  ActivityMainBinding binding;
  ViewModelClass modelClass;

  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);

    modelClass = ViewModelProviders.of(this).get(ViewModelClass.class);
    binding = DataBindingUtil.setContentView(this, R.layout.activity_main);
    dataContainer = new TextOnScreen("hello");
    binding.setData(dataContainer);

    modelClass.getLiveData().observe(this, new Observer<String>() {
      @Override
      public void onChanged(String s) {
        Log.e("TAG", "--onChanged--" + s);
        dataContainer.onscreen.set(s);

        // this line allows the textview to update
        // other wise no change on UI is seen
        binding.setData(dataContainer);
      }
    });
  }
}


public class TextOnScreen {

  public final ObservableField<String> onscreen=new ObservableField<>();

  public TextOnScreen(String t) {
    onscreen.set(t);
  }

  public String getOnscreen() {
    return onscreen.get();
  }

  public void setOnscreen(String onscreen) {
    this.onscreen.set(onscreen);
  }
}


    public class ViewModelClass extends ViewModel{

      private MutableLiveData<String> liveData;

      public ViewModelClass() {
        ModelClass modelClass = new ModelClass();
        liveData= modelClass.generateName();
      }

      public LiveData<String> getLiveData() {
        return liveData;
      }
    }


public class ModelClass {

  MutableLiveData<String > data =new MutableLiveData<>();


  MutableLiveData<String> generateName(){
    final android.os.Handler handler=new android.os.Handler();
    Runnable runnable=new Runnable() {
      @Override
      public void run() {

        data.setValue(data.getValue()+"*");
        handler.postDelayed(this,1000);
      }
    };
    handler.postDelayed(runnable,2000);
    return data;
  }

}

<?xml version="1.0" encoding="utf-8"?>
<layout>
  <data>
    <variable
      name="data"
      type="dsfa.drish.com.livedatasample.TextOnScreen"/>
  </data>
<android.support.constraint.ConstraintLayout
  xmlns:android="http://schemas.android.com/apk/res/android"
  xmlns:app="http://schemas.android.com/apk/res-auto"
  xmlns:tools="http://schemas.android.com/tools"
  android:layout_width="match_parent"
  android:layout_height="match_parent"
  tools:context=".MainActivity">

  <TextView
    android:id="@+id/textview"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:text="@{data.onscreen}"
    app:layout_constraintBottom_toBottomOf="parent"
    app:layout_constraintLeft_toLeftOf="parent"
    app:layout_constraintRight_toRightOf="parent"
    app:layout_constraintTop_toTopOf="parent"/>

</android.support.constraint.ConstraintLayout>
</layout>

Answer 1

Я мог бы достичь того же желаемого результата, сохраняя большую часть вашего кода, удалив TextOnScreen и изменив MainActivity на:

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    ViewModelClass modelClass = ViewModelProviders.of(this).get(ViewModelClass.class);

    ActivityMainBinding binding = DataBindingUtil.setContentView(this, R.layout.activity_main);

    binding.setData(modelClass); // <-- set the ViewModel to be binded
    binding.setLifecycleOwner(this); // <-- this enables MutableLiveData to be update on your UI

    modelClass.getLiveData().setValue("hello");

    modelClass.getLiveData().observe(this, new Observer<String>() {
        @Override
        public void onChanged(String s) {
            Log.e("TAG", "--onChanged--" + s);
        }
    });
}

и на вашем activity_main.xml :

<?xml version="1.0" encoding="utf-8"?>
<layout>
<data>
    <variable
        name="data"
        type="com.rafaelfukuda.stackoverflow.ViewModelClass"/>
</data>
    <android.support.constraint.ConstraintLayout
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context=".MainActivity">

    <TextView
        android:id="@+id/textview"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="@{data.liveData}"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintLeft_toLeftOf="parent"
        app:layout_constraintRight_toRightOf="parent"
        app:layout_constraintTop_toTopOf="parent"/>
    </android.support.constraint.ConstraintLayout>
</layout> 

Другие улучшения кода:

Я бы оставил свои модели очень простыми (POJO без свойств MutableLiveData) и переместил бы generateName в ViewModel. Это будет поддерживать правильную структуру MVVM , и в будущем будет очень легко понять и расширить ваш код.

Результат: