Warning: file_get_contents(/data/phpspider/zhask/data//catemap/2/csharp/282.json): failed to open stream: No such file or directory in /data/phpspider/zhask/libs/function.php on line 167

Warning: Invalid argument supplied for foreach() in /data/phpspider/zhask/libs/tag.function.php on line 1116

Notice: Undefined index: in /data/phpspider/zhask/libs/function.php on line 180

Warning: array_chunk() expects parameter 1 to be array, null given in /data/phpspider/zhask/libs/function.php on line 181
C#中的重载构造函数类似于Delphi(具有多个名称)_C#_Delphi_Constructor_Overloading_Fxcop - Fatal编程技术网

C#中的重载构造函数类似于Delphi(具有多个名称)

C#中的重载构造函数类似于Delphi(具有多个名称),c#,delphi,constructor,overloading,fxcop,C#,Delphi,Constructor,Overloading,Fxcop,我正在尝试将一些代码从Delphi移植到C#,我发现了一个无法以合理方式实现的构造,同时遵守了.NET Framework设计指南(我在问题的最后讨论了该指南) P>显然,C++、java、C++和许多其他语言提供了方法/构造函数重载的含义,但是Delphi构造函数可以另外有多个名称。这样就可以编写直接表示意图的代码: var Data, ParI, ParD, Locl: TDataElement; begin Data := TDataElement.Create('Element

我正在尝试将一些代码从Delphi移植到C#,我发现了一个无法以合理方式实现的构造,同时遵守了.NET Framework设计指南(我在问题的最后讨论了该指南)

<> P>显然,C++、java、C++和许多其他语言提供了方法/构造函数重载的含义,但是Delphi构造函数可以另外有多个名称。这样就可以编写直接表示意图的代码:

var
  Data, ParI, ParD, Locl: TDataElement;
begin
  Data := TDataElement.Create('Element');
  ParI := TDataElement.CreateParam('IntElement', 22);
  ParD := TDataElement.CreateParam('DblElement', 3.14);
  Locl := TDataElement.CreateLocal('LocalElement');
  // ... use the above objects ...
end;
简化代码如下:

unit DataManager;

interface

TDataElement = class
  FName: string;
  FPersistent: Boolean;
public
  constructor Create(AName: string);
  constructor CreateParam(AName: string; DefaultInt: Integer); overload;
  constructor CreateParam(AName: string; DefaultDouble: Double); overload;
  constructor CreateLocal(AName: string);
  property Name: string read FName;;
  property Persistent: Boolean read FPersistent;
end;

implementation

constructor TDataElement.Create(AName: string);
begin
  FName := AName;
  FPersistent := True;
  // ... other initialization ...
end;

constructor TDataElement.CreateParam(AName: string; DefaultDouble: Double);
begin
  Create(AName); 
  // ... use DefaultInt ...
end;

constructor TDataElement.CreateParam(AName: string; DefaultInt: Integer);
begin
  Create(AName); 
  // ... use DefaultDouble...
end;

constructor TDataElement.CreateLocal(AName: string);
begin
  Create(AName); 
  FPersistent := False;
  // ... other code for local (non-persistent) elements ...
end;
特别是在C#中,构造函数必须与类具有相同的名称,因此首先我尝试将行为与枚举区分开来。唉,我偶然发现了几个问题:

  • 每个构造函数中的第一个参数都是相同的类型(ElementKind)
  • 构造函数不像Delphi中那样容易识别(Create vs.CreateParam vs.CreateLocal)
  • 在DataElement的子类中需要特别小心
  • 可能出现错误,如指定ElementKind.DoubleParam和传递整数值
  • 需要一个额外的bool参数来处理本地元素
第一次尝试如下:

public enum ElementKind
{
    Regular, IntParam, DoubleParam, Local
}

public class DataElement
{
    private string FName;
    public string Name { get { return FName; } }

    private bool FPersistent;
    public bool Persistent { get { return FPersistent; } }

    public DataElement(ElementKind kind, string name)
    {
        FName = name;
        // ugly switch :-(
        switch (kind)
        {
            case ElementKind.Regular:
            case ElementKind.IntParam:
            case ElementKind.DoubleParam:
                FPersistent = true;
                break;
            case ElementKind.Local:
                FPersistent = false;
                break;
        }
        // ... other initialization ...
    }

    public DataElement(ElementKind kind, string name, int defaultInt)
        : this(kind, name)
    {
        // ... use defaultInt ...
    }

    public DataElement(ElementKind kind, string name, double defaultDouble)
        : this(kind, name)
    {
        // ... use defaultDouble ...
    }

    // Redundant "bool local" parameter :-(
    public DataElement(ElementKind kind, string name, bool local)
        : this(kind, name)
    {
        // What to do when "local" is false ???

        // ... other code for local (non-persistent) elements ...
    }
}

public class Program
{
    public void Run()
    {
        DataElement data = new DataElement(ElementKind.Regular, "Element");
        DataElement parI = new DataElement(ElementKind.IntParam, "IntElement", 22);
        DataElement parD = new DataElement(ElementKind.DoubleParam, "DblElement", 3.14);
        DataElement locl = new DataElement(ElementKind.Local, "LocalElement");
    }
}
然后,我尝试了更多面向对象的方法,通过类型区分构造函数,同时在Run()方法中保留相同的初始化代码:

一切都很好地编译和运行。那么我的问题是什么。NET框架设计指南,以及一个名为.NET的工具。通过这个工具运行最后一段代码后,我遇到了多个中断问题(见下文)

问题是:如何设计我的类以符合.NET设计指南和最佳实践?

中断-确定性90%-嵌套类型不应可见-ElementKind+RegularElement 中断-确定性90%-嵌套类型不应可见-ElementKind+IntParameterElement 中断-确定性90%-嵌套类型不应可见-ElementKind+DoubleParameterElement 中断-确定性90%-嵌套类型不应可见-ElementKind+LocalElement

断开-确定性90%-静态保持架类型不应具有构造函数-ElementKind

中断-确定性75%-标识符不应包含类型名称-DataElement.#.ctor(ElementKind+IntParameterElement,System.String,System.Int32)

Breaking-75%-标识符不应包含类型名称-DataElement.#.ctor(ElementKind+DoubleParameterElement,System.String,System.Double)

非中断-确定性25%-不声明只读可变引用类型-ElementKind.#常规

非中断-确定性25%-不声明只读可变引用类型-ElementKind.#IntParam

非中断-确定性25%-不声明只读可变引用类型-ElementKind.#DoubleParam

非中断-确定性25%-不声明只读可变引用类型-ElementKind.#Local


首先,我将用一个
enum
替换嵌套的“ElementKind”
class

public enum ElementKind
{
    RegularElement,
    IntParamElement,
    DoubleParamElement,
    LocalElement
}
此外,我认为您的Delphi代码不需要映射到构造函数。您最好使用返回
数据元素的static。例如:

public static DataElement Create(string name)
{
    return new DataElement(ElementKind.Regular, name);
}

public static DataElement CreateParam(string name, int defaultInt);
{
    return new DataElement(ElementKind.IntParam, name);
    // ... use defaultInt ... 
}

// similar to above
public static DataElement CreateParam(string name, double defaultDouble); 
public static DataElement CreateLocal(string name);
因为您使用工厂函数来创建数据元素对象,所以应该将构造函数设置为私有的

然后,将更新Run()函数以使用这些函数,如下所示:

public void Run() 
{ 
    DataElement data = DataElement.Create("Element");
    DataElement parI = DataElement.CreateParam("IntElement", 22);
    DataElement parD = DataElement.CreateParam("DblElement", 3.14); 
    DataElement locl = DataElement.CreateLocal("LocalElement"); 
} 

更新:我包括了对
Run()
函数的建议更改,并更正了基本的
Create()
工厂方法(我相信它应该返回一个“常规的”
DataElement
)。

您可以使用静态工厂方法:

public class TDataElement {
    private TDataElement(){}

    public static TDataElement Create(string name ) {  
        return new TDataElement {
                                   FName = name,
                                   FPersistent = true
                                   // ... other initialization ...
                                };
    }

    public static TDataElement CreateParam(string name, double defaultDouble){
         var element = Create(name);
         // ... use DefaultInt ...
         return element;
    }
//...    
}
您可以这样使用它:

Data = TDataElement.Create("Element");
ParI = TDataElement.CreateParam("IntElement", 22);

这是众所周知的做法。例如,在.NET framework的类
Image
中有一个工厂方法:
var Image=Image.FromFile(“test.jpg”)

我认为,此Delphi代码是C#中泛型的很好候选者:

类数据元素
{
公共字符串名称{get;set;}
公共布尔持久化{get;set;}
公共数据元素(/*ctor params*/)
{
}
}
类DataElement:DataElement
{
公共数据元素(字符串名称,T defaultValue/*其他参数*/)
:base(/*base-ctor-params*/)
{
}
}
类InteElement:DataElement{}
类DoubleElement:数据元素{}

关于TDataElement的更多细节会很有用。

最后,由于@JonSenchyna的想法和@Dennis的建议,我提出了以下通用实现

作为一个额外的特性,我在混合中添加了
IElementFactory
接口,以表达实现子类的要求,使其具有所有
InitXyz()
方法,这在原始的Delphi代码中是不存在的

在这个设计中,我唯一不喜欢的是要求将
DataElement()
ByteElement()
的默认构造函数(来自
new()
约束)以及
InitXyz()
方法(因为它们是public
IElementFactory
接口的一部分)公开

现在FxCop只抱怨四种工厂方法,但我认为我可以接受:

中断-确定性95%-不在泛型类型上声明静态成员-数据元素'1.#Create(System.String)

中断-确定性95%-不在泛型类型上声明静态成员-数据元素'1.#CreateParam(System.String,System.Int32)

中断-确定性95%-不在泛型类型上声明静态成员-数据元素'1.#CreateParam(System.String,System.Double)

中断-确定性95%-不在泛型类型上声明静态成员-数据元素'1.#CreateLocal(System.String)


工厂方法是我在发布问题几个小时后想到的。非常感谢您确认:)顺便说一句,
ElementKind
本来是一个枚举,事实上它是在C#code的第一个版本中。
Data = TDataElement.Create("Element");
ParI = TDataElement.CreateParam("IntElement", 22);
class DataElement
{
  public string Name { get; set; }
  public bool Persistent { get; set; }

  public DataElement(/* ctor params */)
  {
  }
}

class DataElement<T> : DataElement
{
  public DataElement(string name, T defaultValue /* other ctor params */)
    : base(/* base ctor params */)
  {
  }
}

class IntElement : DataElement<int> {}
class DoubleElement : DataElement<double> {}
public enum ElementKind
{
    RegularElement,
    IntParamElement,
    DoubleParamElement,
    LocalElement
}

public interface IElementFactory<TElement>
{
    void InitElement(string name);
    void InitParam(string name, int defaultValue);
    void InitParam(string name, double defaultValue);
    void InitLocal(string name);
}

public class DataElement<TElement>
    where TElement : class, IElementFactory<TElement>, new()
{
    private ElementKind FKind;
    public ElementKind Kind { get { return FKind; } }

    private string FName;
    public string Name { get { return FName; } }

    private int FDefaultInt;
    protected int DefaultInt
    {
        get { return FDefaultInt; }
        set { FDefaultInt = value; }
    }

    private double FDefaultDouble;
    protected double DefaultDouble
    {
        get { return FDefaultDouble; }
        set { FDefaultDouble = value; }
    }

    protected DataElement()
    {
    }

    public virtual void InitElement(ElementKind kind, string name)
    {
        FKind = kind;
        FName = name;
    }

    public static TElement Create(string name)
    {
        TElement obj = new TElement();
        obj.InitElement(name);
        return obj;
    }

    public static TElement CreateParam(string name, int defaultValue)
    {
        TElement obj = new TElement();
        obj.InitParam(name, defaultValue);
        return obj;
    }

    public static TElement CreateParam(string name, double defaultValue)
    {
        TElement obj = new TElement();
        obj.InitParam(name, defaultValue);
        return obj;
    }

    public static TElement CreateLocal(string name)
    {
        TElement obj = new TElement();
        obj.InitLocal(name);
        return obj;
    }
}

public class ByteElement : DataElement<ByteElement>, IElementFactory<ByteElement>
{
    public ByteElement()
    {
    }

    public void InitElement(string name)
    {
        base.InitElement(ElementKind.RegularElement, name);
    }

    public void InitParam(string name, int defaultValue)
    {
        base.InitElement(ElementKind.IntParamElement, name);
        // Range checking
        if ((defaultValue >= Byte.MinValue) && (defaultValue <= Byte.MaxValue))
        {
            DefaultInt = defaultValue;
        }
    }

    public void InitParam(string name, double defaultValue)
    {
        base.InitElement(ElementKind.DoubleParamElement, name);
        // Range checking
        if ((defaultValue >= Byte.MinValue) && (defaultValue <= Byte.MaxValue))
        {
            DefaultDouble = defaultValue;
        }
    }

    public void InitLocal(string name)
    {
        base.InitElement(ElementKind.LocalElement, name);
    }

}
public class Program
{
    public static void Run()
    {
        ByteElement data = ByteElement.Create("Element");
        ByteElement parI = ByteElement.CreateParam("IntElement", 22);
        ByteElement parD = ByteElement.CreateParam("DblElement", 3.14);
        ByteElement locl = ByteElement.CreateLocal("LocalElement");
    }
}