Wo zum speichern von Programm-Einstellungen/Status in einer MVVM-Anwendung

Ich generell ein schlechtes Gefühl bekommen über code, der einen Blick Modell die direkte Kommunikation mit anderen. Ich mag die Idee, dass die VVM Teil des Musters sollte grundsätzlich steckbar und nichts in diesem Bereich der code sollte, hängt von der Existenz von irgendetwas anderem in diesem Abschnitt. Die Argumentation hinter diesem ist, dass ohne eine Zentralisierung der Logik kann es schwierig werden, zu definieren, Verantwortung.

Auf der anderen Seite, basierend auf Ihre tatsächlichen code, kann es auch einfach sein, dass die ApplicationViewModel ist schlecht benannt, es nicht machen eine Modell zugänglich auf einen Blick, so kann dies einfach eine schlechte Wahl der Namen.

Entweder Weg, die Lösung kommt zu einem brechen der Verantwortung. So wie ich das sehe, haben Sie drei Dinge zu erreichen:

1. Mit der der Benutzer eine Verbindung zu einer Adresse
2. Verwenden Sie diese Adresse zu einem server herstellen,
3. Bestehen, dass-Adresse.

Ich würde vorschlagen, dass Sie drei Klassen statt Ihrer zwei.

public class ServiceProvider
{
    public void Connect(Uri address)
    {
        //connect to the server
    }
} 

public class SettingsProvider
{
   public void SaveAddress(Uri address)
   {
       //Persist address
   }

   public Uri LoadAddress()
   {
       //Get address from storage
   }
}

public class ConnectionViewModel 
{
    private ServiceProvider serviceProvider;

    public ConnectionViewModel(ServiceProvider provider)
    {
        this.serviceProvider = serviceProvider;
    }

    public void ExecuteConnectCommand()
    {
        serviceProvider.Connect(Address);
    }        
}

Das nächste, was zu entscheiden ist, wie die Adresse bekommt der SettingsProvider. Man könnte es passieren, von der ConnectionViewModel wie Sie tun derzeit, aber ich bin nicht so scharf auf, weil es erhöht die Kopplung von view-Modell, und es ist nicht die Verantwortung des ViewModel zu wissen, dass es muss durchhalten. Eine weitere option ist, um den Anruf aus der ServiceProvider, aber es hat nicht wirklich das Gefühl, zu mir, wie es sein sollte der Dienstleister die Verantwortung. In der Tat fühlt es sich nicht wie jemand, der Verantwortung anderer als der SettingsProvider. Das führt mich zu glauben, dass die Einstellung für die Anbieter hören sich für änderungen an der verbundenen Adresse und dem persistieren, ohne intervention. In anderen Worten ein Ereignis:

public class ServiceProvider
{
    public event EventHandler<ConnectedEventArgs> Connected;
    public void Connect(Uri address)
    {
        //connect to the server
        if (Connected != null)
        {
            Connected(this, new ConnectedEventArgs(address));
        }
    }
} 

public class SettingsProvider
{

   public SettingsProvider(ServiceProvider serviceProvider)
   {
       serviceProvider.Connected += serviceProvider_Connected;
   }

   protected virtual void serviceProvider_Connected(object sender, ConnectedEventArgs e)
   {
       SaveAddress(e.Address);
   }

   public void SaveAddress(Uri address)
   {
       //Persist address
   }

   public Uri LoadAddress()
   {
       //Get address from storage
   }
}

Damit wird die enge Kopplung zwischen dem ServiceProvider und dem SettingsProvider, die Sie vermeiden wollen, wenn möglich, und ich würde verwenden einen EventAggregator hier, die ich besprochen habe in einer Antwort auf diese Frage

Zur Adresse die Probleme der Testbarkeit, haben Sie jetzt eine sehr definierte Erwartung für das, was jede Methode tun. Die ConnectionViewModel rufen wird eine Verbindung, Die ServiceProvider zu verbinden und die SettingsProvider bestehen bleiben. Zum testen der ConnectionViewModel werden Sie wahrscheinlich wollen, zu konvertieren, die Kopplung an die ServiceProvider, von einer Klasse zu einem interface:

public class ServiceProvider : IServiceProvider
{
    ...
}

public class ConnectionViewModel 
{
    private IServiceProvider serviceProvider;

    public ConnectionViewModel(IServiceProvider provider)
    {
        this.serviceProvider = serviceProvider;
    }

    ...       
}

Dann können Sie mit einem mocking-framework einzuführen, verspottet IServiceProvider, dass Sie können überprüfen, um sicherzustellen, dass die connect-Methode aufgerufen wurde, mit den erwarteten Parametern.

Testen der anderen beiden Klassen ist schwieriger, da Sie darauf angewiesen, dass ein richtiger server und real-persistent-storage-Gerät. Sie können mehr hinzufügen, Ebenen der Dereferenzierung zu verzögern (zum Beispiel einem PersistenceProvider, dass die SettingsProvider verwendet), aber schließlich verlassen Sie die Welt von unit-Tests und geben Sie den Integrationstests. In der Regel, wenn ich code mit dem Muster über die Modelle und Modelle kann eine gute unit test-Abdeckung, aber die Anbieter erfordern komplizierte Testmethoden.

Natürlich, wenn Sie einen EventAggregator zu brechen, Kupplung und IOC, um Tests zu vereinfachen, ist es wahrscheinlich lohnt ein Blick in eines der dependency injection-frameworks wie Microsoft Prism, aber auch wenn Sie zu spät sind entlang in der Entwicklung zu re-Architekten eine Menge von Regeln und Muster können angewendet werden, um bestehenden code in eine einfachere Möglichkeit.