RelayCommands多次执行

时间:2013-07-09 15:05:40

标签: .net windows-phone-7.1 mvvm-light async-await relaycommand

我在Windows Phone 7(使用wp8 SDK和VS Ultimate 2012的7.1)应用程序中使用MVVM Light,该应用程序异步检索来自Web服务应用程序的数据。我在每个执行异步方法的页面上使用RelayCommands来获取数据,然后导航到下一页。例如,在我的一个ViewModel中,我声明了以下ICommand:

public ICommand ShowTreatmentDetailsCommand { get; set; }

然后,在VM的构造函数中,我以这种方式分配它:

ShowTreatmentDetailsCommand = new RelayCommand(ShowTreatmentDetails);

这是该命令调用的方法:

private async void ShowTreatmentDetails()
{
    try
    {
        Treatment refreshedTreatment = await treatmentService.LoadSingle(SelectedTreatment.id, LoggedUser.logon, LoggedUser.pwHash);

        if (refreshedTreatment != null)
        {
            DrugGroup anaestGroup = null;
            DrugGroup surgGroup = null;

            IEnumerable<DrugGroup> groups = await drugGroupService.Load(
                refreshedTreatment.id,
                LoggedUser.logon,
                LoggedUser.pwHash);

            anaestGroup = groups
                .Where(g => g.type == DrugType.Anaesthetic)
                .SingleOrDefault<DrugGroup>();

            surgGroup = groups
                .Where(g => g.type == DrugType.Surgical)
                .SingleOrDefault<DrugGroup>();

            Dictionary<string, object> parameters = new Dictionary<string, object>();
            parameters.Add(Keys.AnaestDrugGroup, anaestGroup);
            parameters.Add(Keys.SurgDrugGroup, surgGroup);
            parameters.Add(Keys.SelectedTreatment, refreshedTreatment);

            Messenger.Default.Send(parameters);
        }
        else
        {
            // Display error message
        }

        RefreshData();
    }
    catch (NullReferenceException) { }
}

当使用EventTrigger和EventToCommand类从ListBox中选择一个项目时,从View的xaml代码调用此命令(但问题与按钮相关的命令保持不变。以防万一,这是我的ListBox元素:

<ListBox x:Name="lbxTreatmentList"
         ItemsSource="{Binding Treatments}"
         SelectedItem="{Binding SelectedTreatment, Mode=TwoWay}">
    <int:Interaction.Triggers>
        <int:EventTrigger EventName="SelectionChanged">
            <com:EventToCommand Command="{Binding ShowTreatmentDetailsCommand}"
                                PassEventArgsToCommand="True" />
        </int:EventTrigger>
    </int:Interaction.Triggers>
    <ListBox.ItemTemplate>
        <DataTemplate>
            <custom:TreatmentListItem PatientName="{Binding patient}"
                                      OpeDescription="{Binding description}"
                                      StartedAt="{Binding startedAt}" />
        </DataTemplate>
    </ListBox.ItemTemplate>
</ListBox>

另一个带按钮的问题:

<Button Grid.Row="2"
    HorizontalAlignment="Center"
    Foreground="{StaticResource BeldicoBlue}"
    BorderBrush="{StaticResource BeldicoBlue}"
    Margin="0,12,0,0"
    Padding="24,12"
    Command="{Binding ValidateCommand}">
    <TextBlock Text="Validate this treatment" FontWeight="ExtraBold" />
</Button>

问题在于,每次触发命令时,相关方法都会执行越来越多次。即:一次是第一次通话,然后是两次,然后是三次,四次,五次......次。由于方法中存在异步服务调用,因此很快就会变成带宽和时间。

我绝对不明白这种行为背后的原因,有人可以帮忙吗?

1 个答案:

答案 0 :(得分:1)

这里有几件可能发生的事情:

  1. 您正在创建附加到页面的多个viewmodel实例,并且每个实例都在执行命令。

    或者,更有可能:

  2. 您还没有完全考虑SelectionChanged事件的工作原理。选择更改时,可以多次触发此事件。如果已经存在选定的项目,则在删除该项目时将触发更改的事件,然后在添加新的所选项目时再次触发。 如果您直接使用事件处理程序,则可以直接查询SelectionChangedEventArgs以确定触发事件的内容。你的代码忽略了这些args,即使你的xaml说你正在传递它们。

  3. 如果您更改操作以便它支持这些参数,您可以在那里添加支票 e.g。

    ShowTreatmentDetailsCommand =
         new RelayCommand<SelectionChangedEventArgs>(ShowTreatmentDetails);
    

    private async void ShowTreatmentDetails(SelectionChangedEventArgs args)
    {
        if (args.AddedItems.Count >= 1)
        {
            // your code
        }
    }