[英]What is the best way to communicate that your constructor has failed in C#?
在C#中,我想与调用方法进行通信,传递给对象的参数导致其实例化失败。
// okay
Banana banana1 = new Banana("green");
// fail
Banana banana2 = new Banana("red");
抛出异常? 如果是这样的话?
throw new ArgumentException("Reason", "param name");
很多(所有?)的答案都说要抛出异常,但我确信我已经看到框架设计团队的官方声明建议不要抛弃构造函数中的异常。
请注意,.NET框架中的行为类似于“Banana”示例(其中只有某些值适合实例化对象)不使用构造函数,而是使用静态工厂方法。 例如, System.Net.WebRequest没有公共构造函数,而是使用静态Create方法,如果提供的字符串不是有效URI,则可以引发异常。 除了一些例外 - 请参阅下面的更新。
因此,对于您的代码,我会将Banana的构造函数更改为protected,并引入如下方法:
public static Banana Create(string color)
{
if (color != "green" && color != "yellow")
{
throw new ArgumentException("Color must be 'green' or 'yellow'",
"color");
}
return new Banana(color);
}
更新
好吧,从构造函数中抛出异常似乎并不是一个坏主意。 事实上,如果您将无效的文件名传递给其构造函数,System.IO.FileStream就是这样做的。 我想使用静态工厂方法的想法只是更明确地说明如何创建实例(例如,如果上面的方法被称为“FromColor”)。
最常见的解决方案是抛出异常。 为了证明这一点,打开反射器并查看BCL中的大多数类以及它们可以在构造上抛出的异常。
举个例子。 如果collection为null,List(IEnumerable collection)将抛出异常。 向调用者传达错误的完美有效方式。
如果构造函数失败,则应抛出异常。 通过defenition,构造函数不返回值,因此您不能返回错误代码。
public class Banana
{
public Banana(string color)
{
if ( color == "red" )
throw new SomeException("intialization failed, red is an invalid color");
}
}
//////////////////////////////////////////////////////////
try
{
Banana banana1 = new Banana("green");
banana1.whatever();
}
catch( SomeException error )
{
// do something
}
finally
{
// always do this stuff
}
我想你想要ArgumentException ......
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.