求助：Do While循环在条件不满足时仍未停止运行

咱们来拆解一下你的循环持续运行的问题，主要有两个核心原因：

1. 方法签名缺失async关键字 #

你的DoSomething方法里用到了await _myRepository.Save(someData);，但方法签名没有标记async——这不仅会导致编译错误，还会让异步代码的执行逻辑完全偏离预期，甚至引发奇怪的循环行为，这是首先要修复的基础问题。

2. 未处理循环体中的异常 #

你的代码只处理了someData.SomeValue == null的情况，但完全没考虑数据库操作（Get或Save）抛出异常的场景。一旦这些步骤报错，代码会直接跳过PopulateErrorData的调用，myObject.IsValid会保持true，循环就会无限执行下去。

另外，当你已经标记对象为无效时，还继续执行Save操作也是不合理的，应该跳过这一步。

修复后的代码 #

// 修正方法签名，添加async关键字
public async Task DoSomething(MyObjectType myObject) 
{ 
    do { 
        try
        {
            // 调用数据库获取数据
            var someData = _myRepository.Get(myObject.Id); 
            // 处理数据
            someData.SomeValue = SomeFunction(someData.Property1, someData.Property2); 
            // 设置myObject的相关值
            if(someData.SomeValue == null) 
            {
                MyUtils.PopulateErrorData(myObject, "Something went wrong!!!"); 
                // 跳过保存步骤，直接进入循环条件检查
                continue;
            }
            // 仅当数据有效时执行保存
            await _myRepository.Save(someData); 
        }
        catch (Exception ex)
        {
            // 捕获所有异常，标记对象为无效并记录错误信息
            MyUtils.PopulateErrorData(myObject, $"Operation failed: {ex.Message}");
        }
    } while(myObject.IsValid); 
}

// 去掉多余的ref参数：MyObjectType是引用类型，修改属性无需传递引用的引用
public static void PopulateErrorData(MyObjectType myObject, string comments) 
{ 
    if(myObject == null) throw new ArgumentNullException(nameof(myObject));
    
    myObject.IsValid = false; 
    myObject.CompletionTimeStamp = DateTime.UtcNow; 
    myObject.Comments = comments; 
}

额外说明 #

• do-while循环会先执行一次循环体再检查条件，所以即使第一次执行就标记IsValid为false，循环也会完成第一次执行后退出，完全符合你“任意步骤失败就终止流程”的需求。
• 添加try-catch后，任何步骤抛出的异常都会被捕获，确保IsValid被设为false，彻底终止循环。
• 移除PopulateErrorData的ref参数：因为MyObjectType是类（引用类型），传递的是对象的引用，修改其属性会直接反映到原对象，ref在这里是多余的，反而会增加代码复杂度。

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