Question

我试图完成表达式功能替代

private static Expression<Func<UserProfile, bool>> CompareFilter(FilterViewModel f)
{
       ...
}

关于这一个：

private static bool CompareFilter(UserProfile profile, FilterViewModel filter)
{
        if (filter.FirstName != null)
        {
            if (profile.FirstName != null)
            {
                if (profile.FirstName.CompareTo(filter.FirstName) != 0)
                {
                    return false;
                }
            }
            else
            {
                return false;
            }
        }

        if (filter.TownId != null)
        {
            if (profile.TownId != filter.TownId)
            {
                return false;
            }
        }

        // true if at least one of the filter interests match
        if (filter.InterestsIds != null)
        {
            var firstInterestFound = profile.Interests
                 .Where(i => filter.InterestsIds.Contains(i.Id))
                 .FirstOrDefault();

            if (firstInterestFound == null)
            {
                return false;
            }
        }
        ...

        return true;
}

有没有办法将这么多嵌套的if语句适用于lambda表达式或其他适用于表达式函数的东西？这个想法是通过所有if语句进行验证，并且只有当所有if语句都没有返回false时才返回true。

提前致谢！

Answer 1

首先，你当前的方法太长了，因为你没有充分利用布尔逻辑。它可以简化为：

private static bool CompareFilter(UserProfile profile, FilterViewModel filter)
{
    if (filter.FirstName != null && filter.FirstName != profile.FirstName)
    {
        return false;
    }

    if (filter.TownId != null && filter.TownId != profile.TownId)
    {
        return false;
    }

    // true if at least one of the filter interests match
    if (filter.InterestsIds != null &&
        !profile.Interests.Any(i => filter.InterestsIds.Contains(i.Id)))
    {
        return false;
    }
    ...

    return true;
}

你可以把它变成一个很大的笨重的表达by inverting all the logic ¹：

private static bool CompareFilter(UserProfile profile, FilterViewModel filter)
{
    return (filter.FirstName == null || filter.FirstName == profile.FirstName) &&

           (filter.TownId == null || filter.TownId == profile.TownId) &&

           (filter.InterestsIds == null ||
            profile.Interests.Any(i => filter.Interests.Contains(i.Id)));

    // etc. etc.
}

一旦你拥有了它，将它变成一个lambda并获得你的Expression<T>是一块蛋糕：

private static Expression<Func<UserProfile, bool>> CompareFilter(FilterViewModel f)
{
    return profile => 
           (filter.FirstName == null || filter.FirstName == profile.FirstName) &&

           (filter.TownId == null || filter.TownId == profile.TownId) &&

           (filter.InterestsIds == null ||
            profile.Interests.Any(i => filter.Interests.Contains(i.Id)));
}

从技术上讲，您可以||将第一种方法中的所有条件放在一起，并在其周围放置一个大的!( )：return !( (...) || (...) || (...));，但是反转所有条件并使用{{1}加入条件好多了。

Lambda表达式替代了几个if语句

1 个答案: