RelayCommand Lambda Syntax Problem

Question

Ich verwende das MVVM-Muster pro Josh Smith und Schwierigkeiten haben. Ich habe das Problem hier untersucht und kann die Syntax nicht ganz richtig finden.

Der folgende Code sieht für mich aus, als ob es die erforderliche Syntax folgt, aber Visual Studio meldet Fehler "Delegate 'System.Action' nicht '2' Argumente" auf der angegebenen Zeile.

Kann jemand sehen, wo ich einen Fehler mache? Vielen Dank!
+ tom

RelayCommand _relayCommand_MoveUp; 
    public ICommand RelayCommand_MoveUp 
    { 
     get 
     { 
     if (_relayCommand_MoveUp == null) 
     { 
      _relayCommand_MoveUp = new RelayCommand(
      (sender, e) => this.Execute_MoveUp(sender, e),  **ERROR REPORTED HERE** 
      (sender, e) => this.CanExecute_MoveUp(sender, e)); 
      return _relayCommand_MoveUp; 
     } 
     } 
    } 

    private void Execute_MoveUp(object sender, ExecutedRoutedEventArgs e) 
    { 
     if (_selectedFolder != null) 
     { 
     _selectedFolder.SelectParent(); 
     } 
    } 

    private void CanExecute_MoveUp(object sender, CanExecuteRoutedEventArgs e) 
    { 
     e.CanExecute = (_selectedFolder != null) && (_selectedFolder.Parent != null); 
     } 


//And from Josh Smith: 

    public class RelayCommand : ICommand 
    { 
    public RelayCommand(Action<object> execute); 
    public RelayCommand(Action<object> execute, Predicate<object> canExecute); 

    public event EventHandler CanExecuteChanged; 

    [DebuggerStepThrough] 
    public bool CanExecute(object parameter); 
    public void Execute(object parameter); 
    } 

Answer 1

Die RelayCommand ist kein RoutedCommand, was ich denke, ist, wo Sie verwirrt enden.

Die Konstruktoren für den Relaisbefehl nehmen eine Action delegate und optional Predicate delegate. Diese Delegaten verwenden keine EventArgs, sondern nur den einzelnen Object-Parameter. Aus diesem Grund tritt ein Fehler auf. Das Prädikat erfordert auch einen Rückgabetyp von bool, was der nächste Fehler ist, den Sie erhalten. Im CanExecute-Vergleichselement anstelle von e.CanExecute wie bei einem RoutedCommand geben Sie einfach true/false zurück.

Hier ist, wie es aussehen sollte:

public ICommand RelayCommand_MoveUp 
{ 
    get 
    { 
    if (_relayCommand_MoveUp == null) 
    { 
     _relayCommand_MoveUp = new RelayCommand(Execute_MoveUp, CanExecute_MoveUp); 

    } 
    return _relayCommand_MoveUp; 
    } 
} 

private void Execute_MoveUp(object sender) 
{ 
    if (_selectedFolder != null) 
    { 
    _selectedFolder.SelectParent(); 
    } 
} 

private void CanExecute_MoveUp(object sender) 
{ 
    return (_selectedFolder != null) && (_selectedFolder.Parent != null); 
} 

EDIT (Hinzugefügt von Diskussion in den Kommentaren):

Wenn Sie etwas mehr wie die RoutedCommands verwenden möchten, was die machen ViewModels sind stärker abhängig von WPF-spezifischen Ansichten. Es sind einige gute Optionen verfügbar.

Diese discussion hat die ganze Idee der Verwendung von RoutedCommands in Verbindung mit MVVM gestartet.

Und here's eine sehr solide Lösung für die Probleme von Josh Smith und Bill Kempf vorgestellt.

Answer 2

An diesem Wochenende (22. August) checkte Josh Smith neue Änderungen in Codeplex für sein MvvmFoundation-Projekt ein, das die Funktionsweise von RelayCommand für Delegaten mit einem Parameter ändert. In acht nehmen!

public ICommand GotoRegionCommand 
    { 
     get 
     { 
      if (_gotoRegionCommand == null) 
       _gotoRegionCommand = new RelayCommand<String>(GotoRegionCommandWithParameter); 
      return _gotoRegionCommand; 
     } 
    } 
    private void GotoRegionCommandWithParameter(object param) 
    { 
     var str = param as string; 
    }