我正在开发一个工厂,基于两个标准创建具体的实例:
1)类必须继承自特定的抽象类
2)类必须在重写的属性中具有特定值
我的代码如下所示:
public abstract class CommandBase
{
public abstract string Prefix { get; }
}
public class PaintCommand : CommandBase
{
public override string Prefix { get; } = "P";
}
public class WalkCommand : CommandBase
{
public override string Prefix { get; } = "W";
}
class Program
{
static void Main(string[] args)
{
var paintCommand = GetInstance("P");
var walkCommand = GetInstance("W");
Console.ReadKey();
}
static CommandBase GetInstance(string prefix)
{
try
{
var currentAssembly = Assembly.GetExecutingAssembly();
var concreteType = currentAssembly.GetTypes().Where(t => t.IsSubclassOf(typeof(CommandBase)) &&
!t.IsAbstract &&
t.GetProperty("Prefix").GetValue(t).ToString() == prefix).SingleOrDefault();
if (concreteType == null)
throw new InvalidCastException($"No concrete type found for command: {prefix}");
return (CommandBase)Activator.CreateInstance(concreteType);
}
catch (Exception ex)
{
return default(CommandBase);
}
}
}
我得到了一个错误:
{System.Reflection.TargetException: Object与目标类型不匹配。在System.Reflection.RuntimeMethodInfo.CheckConsistency(Object目标处)在System.Reflection.RuntimeMethodInfo.InvokeArgumentsCheck(Object obj处,BindingFlags invokeAttr,活页夹,Object[]参数,CultureInfo文化)在System.Reflection.RuntimeMethodInfo.InvokeArgumentsCheck(Object obj处,CultureInfo invokeAttr,活页夹,Object[]参数,CultureInfo文化)
发布于 2018-05-06 02:07:47
一种更简洁的方法是定义自己的属性来存储前缀。
[AttributeUsage(AttributeTargets.Class)]
public class CommandAttribute : Attribute
{
public String Prefix { get; set; }
public CommandAttribute(string commandPrefix)
{
Prefix = commandPrefix;
}
}
然后像这样使用它们:
[CommandAttribute("P")]
public class PaintCommand : CommandBase
{}
[CommandAttribute("W")]
public class WalkCommand : CommandBase
{}
在反射中:
static CommandBase GetInstance(string prefix)
{
var currentAssembly = Assembly.GetExecutingAssembly();
var concreteType = currentAssembly.GetTypes().Where(commandClass => commandClass.IsDefined(typeof(CommandAttribute), false) && commandClass.GetCustomAttribute<CommandAttribute>().Prefix == prefix).FirstOrDefault();
if (concreteType == null)
throw new InvalidCastException($"No concrete type found for command: {prefix}");
return (CommandBase)Activator.CreateInstance(concreteType);
}
发布于 2018-05-06 06:35:22
正如spender在他的评论中提到的,你得到这个特定错误的原因是下面这行:
t.GetProperty("Prefix").GetValue(t)
在这里,t
是一个包含类的类型变量,比如WalkCommand
。您将为该类的Prefix
属性获取PropertyInfo
对象,然后尝试使用GetValue()
从WalkCommand
对象的实例中读取该属性的值。
问题是你没有传递给GetValue()
一个WalkCommand
类的实例,而是传递了一个Type
,所以反射抛出了这个异常。
有几种方法可以解决这个问题:
1)动态创建每种类型的实例,只是为了读取它的前缀(我真的不建议这样做):
var instance = currentAssembly.GetTypes()
.Where(t => t.IsSubclassOf(typeof(CommandBase)) && !t.IsAbstract)
.Select(t => new { t, i = (CommandBase)Activator.CreateInstance(t) })
.Where(x => x.t.GetProperty("Prefix").GetValue(x.i).ToString() == prefix)
.Select(x => x.i)
.SingleOrDefault();
return instance;
2)将整个过程改为使用属性,例如在SwiftingDuster的答案中
3)使用静态构造函数创建将字符串首选项映射到具体类型的Dictionary。反射的开销很大,而且类也不会改变(除非你是在动态加载程序集),所以只需要做一次。
我们可以通过(Ab)使用我前面的“创建一个实例来丢弃它”代码来做到这一点,所以我们仍然为每个类创建一个实例来读取属性,但至少我们只做了一次:
static Dictionary<string, Type> prefixMapping;
static Program()
{
prefixMapping = currentAssembly.GetTypes()
.Where(t => t.IsSubclassOf(typeof(CommandBase)) && !t.IsAbstract)
.Select(t => new { t, c = (CommandBase)Activator.CreateInstance(t) })
.ToDictionary(x => x.t.GetProperty("Prefix").GetValue(x.c).ToString(), x => x.t);
}
static CommandBase GetInstance(string prefix)
{
Type concreteType;
if ( prefixMapping.TryGetValue(prefix, out concreteType) )
{
return (CommandBase)Activator.CreateInstance(concreteType);
}
return default(CommandBase);
}
请注意,如果您有多个具有相同前缀的类,这将抛出一个非常可怕的异常,并且由于它将是静态构造函数中的一个异常,它很可能会炸毁您的应用程序。要么捕获异常(将prefixMapping
保留为null
),要么修改Linq表达式,使每个前缀只返回一种类型(如下所示)。
4)既使用SwiftingDuster的属性方法,也使用字典的预计算。这将是我首选的解决方案:
static Dictionary<string, Type> prefixMapping;
static Program()
{
prefixMapping = currentAssembly.GetTypes()
.Where(t => t.IsSubclassOf(typeof(CommandBase)) && t.IsDefined(typeof(CommandAttribute), false) && !t.IsAbstract)
.Select(t => new { t, p = t.GetCustomAttribute<CommandAttribute>().Prefix })
.GroupBy(x => x.p)
.ToDictionary(g => g.Key, g => g.First().t);
}
static CommandBase GetInstance(string prefix)
{
Type concreteType;
if ( prefixMapping.TryGetValue(prefix, out concreteType) )
{
return (CommandBase)Activator.CreateInstance(concreteType);
}
return default(CommandBase);
}
希望这能有所帮助
https://stackoverflow.com/questions/50192150
复制相似问题