编写基于条件的 if 语句的优雅方式

Question

我试图根据一组标准在列表中查找对象。基本逻辑如下所示

for (objectx obj : list)
{
    if (object.property1 > criteria1)
        //accept object
    else (object.property1 == criteria1)
    {
        if (object.property2 > criteria2)
            //accept object
        else (object.property2 == criteria2)
        {
            if (object.property3 > criteria3)
                  ... etc
        } 
    }                                  
}

有没有一种优雅的方法来简化这个混乱？

Answer 1

boolean accept = false;
for (int i = 0; i < object.numProps(); i++) {
   if (object.prop[i] > criteria[i]) {
       accept = true;
       break;
   } else if (object.prop[i] < criteria[i]) 
       break;
   // loop continues only if object.prop[i] == criteria[i]
}

Answer 2

我将创建一个方法来检查所有属性，然后在循环中您可以简单地说：

for (objectx obj : list)
{
    if(checkProperties(obj))
    {
        //Do Stuff
    }
}

Answer 3

添加以您的业务逻辑命名的方法可能会有所帮助。然后阅读代码应该像英语一样读起来，而实际的比较在其他地方。这也让思考变得更容易。

Answer 4

如果您想做像我之前那样的事情，那会有所帮助。我想像你一样使用 NHibernate 搜索数据库中的对象。我创建了查询对象来收集用户输入的每个条件。当我在查询对象上设置相关属性时，我添加了这样的条件。

public bool Archive
    {
        set
        {
            if(value)
            {
                criteria.Add(Restrictions.Eq("Archive", true));
            }
        }
    }

    public bool IsFavorite
    {
        set
        {
            if (value)
            {
                criteria.Add(Restrictions.Eq("IsFavorite", true));
            }
        }
    }

如果你只想过滤内存中的对象。您还可以使用规范模式。