C#中优雅实现属性空值检查与异步赋值的性能疑问

我在C#项目里频繁写这类代码：

if (null != classX.user || await classX.getUserAsync())
{
     // getUserAsync()会合理设置classX.user并返回true，否则返回false。
     // 后续调用仅在classX.user有效时允许的classX成员及其他函数
}

为了提升可读性，我想把判断逻辑封装成成员方法：

private async Task<bool> IsUserSet()
{
     if (null != user)
         return await Task.FromResult<bool>(user);
     
     return await getUserAsync();
}

这样就能用if (await IsUserSet())替代原判断，但我担心引入额外Task会带来性能损耗，想知道有没有无需额外Task的实现方式，或是更简洁的ASP.NET写法。

优化方案与性能优化要点 #

• 用ValueTask替代Task降低开销：对于这类可能同步完成的异步方法，使用ValueTask<bool>替代Task<bool>，能避免不必要的Task对象分配，减少性能损耗。修改后的方法可写成：

private ValueTask<bool> IsUserSet()
{
    if (user != null)
        return ValueTask.FromResult(true);
    
    return getUserAsync(); // 建议将getUserAsync也改为返回ValueTask<bool>
}

• 将空值检查整合到原异步方法：直接修改getUserAsync（建议重命名为SetUserAsync更贴合语义），在方法内部先检查user是否已存在，存在则直接返回成功，无需重复执行异步逻辑。这样连中间方法IsUserSet都可以省去：

public async ValueTask<bool> SetUserAsync()
{
    if (user != null)
        return true;
    
    // 原getUserAsync的逻辑：获取并设置user，返回是否成功
    user = await FetchUserFromSourceAsync();
    return user != null;
}

之后直接用if (await classX.SetUserAsync())即可，既简洁又避免额外封装的开销。

• 避免不必要的Task.FromResult：同步返回时直接返回布尔值即可，不需要用Task.FromResult包装——这会额外分配Task对象，增加无意义的开销。比如原方法里的return await Task.FromResult<bool>(user);完全可以改成return true;（因为user != null已经为真）。

内容的提问来源于stack exchange，提问作者Greg L.