Les meilleures pratiques pour Android MVVM startActivity

Je suis de la construction d'une Application Android utilisant MVVM et la liaison de données. Et j'ai une fonction à l'intérieur de mon ViewModel qui démarre une Activité.
Est-il normal d'avoir un onClick appel à l'intérieur d'un ViewModel?

Comme ça.

public class MyViewModel {
    public void onClick(View view, long productId) {
        Context context = view.getContext();
        Intent intent = new Intent(context, ProductDetailActivity.class);
        intent.putExtra("productId", productId);
        context.startActivity(intent);
    }
}

Et dans mon XML:

...
android:onClick="@{(v) -> viewModel.onClick(v, viewModel.product.id)}">

Ou serait-il une meilleure pratique pour la déplacer vers la Vue et de l'appeler à partir d'EventBus ou Rx et ont seulement POJO dans mon ViewModel?

OriginalL'auteur Felipe Kenji Shiba | 2016-11-07

17

La réponse à votre question-quel est votre objectif?

Si vous souhaitez utiliser MVVM pour la séparation des préoccupations de sorte que vous pouvez unité de tester votre Viewmodel alors vous devriez essayer de garder tout ce qui nécessite un Context séparé de votre Viewmodel. Le Viewmodel contient le noyau de la logique métier de votre application et ne devrait pas avoir de dépendances externes.

Cependant j'aime où l'on va 🙂 Si la décision dont l'Activité est ouverte réside dans la Vue, c'est très très dur d'écrire un test Unitaire. Cependant, vous pouvez passer un objet dans le Viewmodel qui effectue la startActivity() appel. Maintenant, dans votre de test de l'Unité vous pouvez tout simplement se moquer de cet objet et de vérifier que la bonne Activity est ouvert

"Le Viewmodel contient le noyau de la logique métier de votre application et ne devrait pas avoir de dépendances externes." - Je pense qu'il aura toujours des dépendances externes. Je ne vois pas un point en utilisant ViewModel sans liaison de données et c'est android spécifique, il suffit de voir l'exemple de l'importation: "import android.la liaison de données.ObservableBoolean;"
Vous avez raison dans une certaine mesure. La question est de savoir quoi ne voulez atteindre. Pour moi la chose la plus importante est la testabilité. Si j'ai une dépendance à View ou Context je ne peux vérifier à l'aide de Robolectric. Ces tests sont lents et nécessitent beaucoup de frais généraux. Donc ce que je cherche, c'est la plaine de l'Unité de test qui s'exécutent dans une fraction du temps un sont faciles à mettre en œuvre. La dépendance à la liaison de données de la bibliothèque ne diminue en rien cette.
Donc, il n'y a aucune façon de mettre en œuvre une telle fonctionnalité. Je ne veux pas garder Contexte dans ma classe ViewModel, car il peut créer des fuites de Mémoire. Ni que je veux passer toute référence. Donc, si je veux passer à quelque ressource de Chaîne à partir de view viewModel ou je veux appeler une méthode en Vue de lancer une activité, alors comment puis-je la mettre en œuvre.
Vous pouvez faire quelque chose de similaire à ceci: stackoverflow.com/questions/31883415/... Donc ce que vous voulez faire est de passer une couche d'abstraction dans votre Dernier ou de l'animateur. Cette abstraction(interface) pourrait avoir une méthode String getString(@StringRes int id). Et dans la mise en œuvre concrète de cette interface, vous pouvez implémenter les getString et peut déléguer ce contexte. getString(). De cette façon, ce Dernier ne voit qu'une abstraction qui vous pourriez maquette pour tester
Merci pour la réponse. En fait je connais déjà cette approche, et je suis de l'utiliser. Je ne suis pas vraiment concerné par les tests. Mon seul souci est de ne pas utiliser toute référence à vue parce que cela peut conduire à des fuites de mémoire et je n'en veux pas. Alors, dites-moi si nous utilisons une interface en œuvre en vue et référencé par ViewModel d'obtenir ou de définir des données en Vue, serait-il conduire à la fuite de mémoire?

OriginalL'auteur Kaskasi
8

C'est absolument parfait pour le mettre à l'intérieur ViewModel, cependant, vous devez définir votre ViewModel de Activity/Fragment.

Voici quelques liens que vous pouvez suivre pour apprendre architecture MVVM.

Approche Android avec MVVM

Android MVVM

https://github.com/ivacf/archi

Personnes-MVVM

MVVM sur Android: Ce que Vous Devez Savoir

La plupart de ces exemples ont contexte referrences dans le ViewModel. Pour moi, ce n'est pas vrai MVVM.
Je crois que les choses les plus importantes sur le modèle MVVM est la séparation de la logique entre les couches, en conséquence, devrait être responsable de la gestion de l'INTERFACE utilisateur et rien de plus! Donc, Dans ce cas, je crois que Modèle de Vue d'avoir une référence au contexte de l'application est moins nocif que de forcer l'affichage de faire aucune INTERFACE utilisateur des tâches. BTW, il y a de nombreux autres cas que nous avons peut-être besoin de contexte dans le ViewModel c'est pourquoi Android est une version de ViewModel qui a contexte!
pensé que vous étiez censé ne pas avoir de android paquets dans le viewmodel?

OriginalL'auteur Ravi Rupareliya

La façon dont je le fais, c'est, dans votre ViewModel:

val activityToStart = MutableLiveData<Pair<KClass<*>, Bundle?>>()

Cela vous permet de vérifier la classe de l'Activité a commencé, et les données transmises dans le Bundle. Puis, dans votre Activité, vous pouvez ajouter ce code:

viewModel.activityToStart.observe(this, Observer { value ->
    val intent = Intent(this, value.first.java)
    if(value.second != null)
        intent.putExtras(value.second)
    startActivity(intent)
})

OriginalL'auteur gahfy

Que le principe de MVVM souligne que la seule Vue (activité/fragment) contient la référence au ViewModel et le ViewModel ne devrait pas tenir de référence à tout point de Vue.

Dans votre cas, pour démarrer une activité, je vais faire comme ceci:

MyViewModel.class

public class MyViewModel {
public static final int START_SOME_ACTIVITY = 123;

 @Bindable
 private int messageId;

 public void onClick() {
  messageId = START_SOME_ACTIVITY;
  notifyPropertyChanged(BR.messageId); //BR class is automatically generated when you rebuild the project
 }

 public int getMessageId() {
        return messageId;
 }

 public void setMessageId(int message) {
        this.messageId = messageId;
 }

}

Et dans votre MainActivity.class

@BindingAdapter({"showMessage"})
public static void runMe(View view, int messageId) {
    if (messageId == Consts.START_SOME_ACTIVITY) {      
        view.getContext().startActivity(new Intent(view.getContext(), SomeActivity.class));      
    }
}

@Override
protected void onPause() {
    super.onPause();
    finish(); //only call if you want to clear this activity after go to other activity
}

enfin, dans votre activity_main.xml

<Button    
  android:onClick="@{()-> myViewModel.onClick()}"    
  bind:showMessage="@{myViewModel.messageId}" />

OriginalL'auteur lehongphucit

Que par la liaison de données de la documentation.
Il y a 2 façons de le faire:

1- MethodReferences : Vous devez passer l'affichage en tant que paramètre à la fonction, ou vous obtiendrez une erreur de compilation.

Si vous allez utiliser cette façon de faire une catégorie distincte comme exemple ici, qui gèrent de tels événements.

MyHandler

public class MyHandler {
   public void onClick(View view, long productId) {
        Context context = view.getContext();
        Intent intent = new Intent(context, ProductDetailActivity.class);
        intent.putExtra("productId", productId);
        context.startActivity(intent);
    }
}

XML

<data>
        <variable
            name="viewModel"
            type="com.example.ViewModel"

        <variable
            name="myHandler"
            type="com.example.MyHandler" />

    </data>android:onClick="@{myHandler.onClick(viewModel.product.id)}">

2- Auditeur liaisons : Vous n'avez pas besoin de passer de la vue comme un exemple ici.

Mais si vous voulez startActivity faire de votre viewModel s'étend AndroidViewModel et vous allez utiliser l'applicaion objet.

ViewModel

public class MyViewModel extends AndroidViewModel {
    public void onClick(long productId) {
        Intent intent = new Intent(getApplication(), ProductDetailActivity.class);
        intent.putExtra("productId", productId);
        context.startActivity(intent);
    }
}

XML

android:onClick="@{() -> viewModel.onClick(viewModel.product.id)}">

OriginalL'auteur Moaz Rashad

Vous devez vous connecter pour publier un commentaire.