Xamarin转MAUI后属性值变更触发无限循环求助

无限循环根源 #

你的代码触发无限循环核心原因有两点：

1. ItemID属性setter逻辑错误：setter开头直接修改私有字段_itemID，而非先处理输入值再与当前字段比较，导致属性更新逻辑混乱。执行ItemID = ""时，错误的setter逻辑可能意外触发重复属性变更通知，进而反复调用IsValid方法。
2. IsValid方法内的属性修改闭环：IsValid中修改ItemID的操作会触发setter，若属性变更后又触发了调用IsValid的扫描命令，就会形成IsValid → 修改ItemID → 触发setter → 再次调用IsValid的循环。

代码修复步骤 #

1. 修正ItemID的setter逻辑

正确的MVVM属性setter应先处理输入值，再与当前私有字段比较，仅当值确实变化时才更新字段并触发通知：

private string _oldItemID = string.Empty;
private string _itemID = string.Empty;
public string ItemID
{
    get => _itemID;
    set
    {
        // 先处理输入值的trim操作
        var trimmedValue = !string.IsNullOrEmpty(value) ? value.Trim() : value;
        // 与当前私有字段比较，避免无意义的更新
        if (trimmedValue != _itemID)
        {
            _oldItemID = _itemID;
            // 通过SetProperty更新字段并触发属性变更通知
            SetProperty(ref _itemID, trimmedValue, nameof(ItemID));
        }
    }
}

2. 修复IsValid方法的逻辑问题

原代码存在未定义变量、语法错误等问题，同时优化属性赋值逻辑避免循环：

private async Task<bool> IsValid()
{
    try
    {
        bool lValid = false;
        // 提前获取trim后的值，避免重复触发setter
        var trimmedItemID = ItemID.Trim();

        // 替换为你实际的验证逻辑，示例中仅保留结构
        // lValid = 你的验证规则（如数据库校验、格式检查等）;

        if (!lValid)
        {
            // 修正字符串拼接的语法错误
            await DisplayAlert("提示", $"Item '{trimmedItemID}' is not valid", "确定");
            // 修复后的setter不会触发无效循环
            ItemID = "";
        }    

        // 原代码中_Item变量未定义，需替换为实际业务变量或删除该逻辑
        // if (!lValid)
        // {
        //     ItemID = _Item.Trim();
        // }

        return lValid;
    }
    catch (Exception ex)
    {
        return false;
    }
}

3. 避免属性变更触发重复扫描

检查扫描命令的触发条件：如果扫描命令绑定到ItemID的PropertyChanged事件，需添加判断逻辑，仅当输入来自扫描设备时才调用IsValid，而非每次属性变更都执行验证。

内容的提问来源于stack exchange，提问作者Eric