C#反射批量设置对象布尔属性值逻辑错误排查求助

咱们先拆解一下你的代码为什么没达到预期效果，主要是两个关键问题：

1. 条件判断的逻辑错误 #

先看这行代码：

if(propNames.Length > 0 || propNames != null)

这里有两个明显问题：

• 顺序错误：如果propNames是null，先访问Length会直接抛出NullReferenceException，必须先判断propNames != null再检查长度
• 运算符错误：这里应该用&&而不是||，咱们的意图是「当排除列表不为空且有元素时」才执行后续逻辑，||会导致只要其中一个条件满足就执行，完全不符合预期。

2. 嵌套循环的逻辑漏洞 #

你用了两层嵌套循环：外层遍历所有属性，内层遍历排除的属性名。这个逻辑会导致致命问题：
比如RecordExist这个属性，当它和第一个排除名"RecordExist"比对时，因为名字匹配，不会赋值；但当它和第二个排除名"HasDamage"比对时，名字不匹配，就会触发property.SetValue把它设为true！这就是为什么原本要排除的属性最后还是被改成了true。

修正后的代码 #

咱们把逻辑改成「遍历所有属性，只要属性类型是目标类型，且不在排除列表中，就赋值」，这样就避免了嵌套循环的问题，同时修正条件判断：

public T SetBoolPropertyValueOfObject<T>(string[] propsToExclude, Type propertyType, object value) where T : new()
{
    // 用new()约束创建实例，更安全且不需要强制转换
    T obj = new T();
    // 先判断排除列表是否有效
    if (propsToExclude != null && propsToExclude.Length > 0)
    {
        PropertyInfo[] properties = typeof(T).GetProperties();
        foreach (var property in properties)
        {
            // 检查属性类型匹配，且不在排除列表中
            if (property.PropertyType == propertyType && !propsToExclude.Contains(property.Name))
            {
                property.SetValue(obj, value, null);
            }
        }
    }
    return obj;
}

调用方式保持不变： #

public Foo GetFoo() 
{ 
    string[] propsToExclude = new string[] { "RecordExist", "HasDamage" }; 
    var foo = SetBoolPropertyValueOfObject<Foo>(propsToExclude, typeof(bool), true); 
    return foo; 
}

额外优化建议 #

• 用where T : new()泛型约束，替代Activator.CreateInstance，不仅代码更简洁，还能在编译期就检查T是否有无参构造函数，避免运行时异常。
• 如果排除列表经常使用或元素较多，可以把它转成HashSet<string>，这样Contains的查询效率会更高：

  var excludeSet = new HashSet<string>(propsToExclude);
  // 然后判断：!excludeSet.Contains(property.Name)
  

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