使用反射清理代码
使用反射清理代码
提问于 2011-06-17 21:52:28
我想使用反射遍历我的Car类，以删除所有空值，并用更好的值替换它们，如string = "“。
[Serializable()]
public class Car
{
public string model;
public int year;
public List<Owner> owner;
}
[Serializable()]
public class Owner
{
public string firstName;
public string lastName;
}到目前为止，我已经做到了
public void LoopEverythingAndFix(object type)
{
var prop = type.GetType().GetFields();
foreach (var fieldInfo in prop)
{
if (GetType(fieldInfo))
{
var value = fieldInfo.GetValue(type);
if (value == null)
fieldInfo.SetValue(type, GetDefaultValue(fieldInfo));
}
else
{
LoopEverythingAndFix(fieldInfo);
}
}
}
public bool GetType(System.Reflection.FieldInfo fieldInfo)
{
if (fieldInfo.FieldType == typeof(string))
return true;
if (fieldInfo.FieldType == typeof(bool))
return true;
if (fieldInfo.FieldType == typeof(int))
return true;
if (fieldInfo.FieldType == typeof(decimal))
return true;
return false;
}GetType方法是知道当前字段是像"owner“类一样的类，还是像int/string这样的值/引用字段，如果它是"owner”类型，那么我想循环到并修复那些属性。
问题是当它在car类中找到"owner“并执行时：
LoopEverythingAndFix(fieldInfo);这就是问题所在，因为我将fieldInfo发送给LoopEverythingAndFix方法，当它返回循环时，它在type.GetType().GetFields()处得到0个字段。这是一个列表，我希望循环listitems项并将它们发送到LoopEverythingAndFix方法
回答 3
Stack Overflow用户
回答已采纳
发布于 2011-06-17 21:59:10
您试图在反射类型FieldInfo上调用LoopEverythingAndFix，而不是您想要修复的实际对象。
要修复此问题，请替换以下内容：
LoopEverythingAndFix(fieldInfo);有了这个：
LoopEverythingAndFix(fieldInfo.GetValue(type));Stack Overflow用户
发布于 2011-06-17 22:00:19
看起来问题不是传递一个所有者对象，而是传递一个对静态类所有者的引用。因此，您已经拥有了所有者的类型。
Stack Overflow用户
发布于 2011-06-17 22:26:19
您的代码有几个问题需要解决：
- 您正在对已知值类型执行null检查。
Nullable<T>以外的ValueType实例永远不会为空。此操作是不必要的，并且会影响性能。我已经在下面修复了这个问题。 - 你没有考虑到
string以外的空引用类型字段的情况。例如，如果存在空Owner或空List<Owner>会发生什么？ - 由于您没有发布
GetDefaultValue()函数，因此它的签名有点不清楚。我的猜测是，它接受一个FieldInfo并根据字段类型解析默认的非空值。由于这实际上只适用于string参数，因此最好用string.Empty替换该调用。我在下面的评论中指出了这一点。
下面，我已经合并、优化和重写了下面的函数，并添加了注释，以指示必须更改或插入不同行为的位置。
public void LoopEverythingAndFix(object instance)
{
var fields = instance.GetType().GetFields(BindingFlags.Public|BindingFlags.Instance)
foreach(var fieldInfo in fields){
var fieldType = fieldInfo.FieldType;
if(!fieldType.IsValueType){
bool isString = fieldType == typeof(string);
var fieldValue = fieldInfo.GetValue(instance);
if(fieldValue != null){
if(!isString){
// This should recursion be called when the type is a
// complex (non-string) reference type that is not null
LoopEverythingAndFix(fieldValue);
}
// You don't need to fix a non-null string value
}
else{
if(isString){
// since you didn't post the code for this, I am assuming
// it works correctly, but it may be that you can just replace
// GetDefaultValue(fieldInfo) with string.Empty
fieldInfo.SetValue(instance, GetDefaultValue(fieldInfo));
}
else{
// It is unclear how you want to handle a complex reference type
// field with a null value. That code should go here.
}
}
}
}
}页面原文内容由Stack Overflow提供。腾讯云小微IT领域专用引擎提供翻译支持
原文链接：
https://stackoverflow.com/questions/6386808
复制相关文章
相似问题
腾讯云开发者
