C# 中的堆栈溢出错误 - 但如何修复它？

Question

我遇到了一个非常有趣的运行时错误，它会生成恶意堆栈溢出。

我定义了一个结构如下：

public enum EnumDataType { Raspberry, Orange, Pear, Apple };

public class DataRequest
{
    public long DataSize 
    { 
        get { return 0; } 
        set { DataSize = value; } 
    }

    public EnumDataType DataType  
    { 
        get { return EnumDataType.Apple; } 
        set { DataType = value; } 
    }
}

以下几行工作完美：

DataRequest request = new DataRequest();
request.DataSize = 60;

但是，当我在代码中跳过以下行时，它会生成堆栈溢出：

request.DataType = EnumDataType.Raspberry;

当然，我可以通过删除默认值或使用 auto get/ 来修复它设置，但我需要它既可读又可写，并返回默认值 - 有什么想法吗？

Answer 1

public long DataSize { get { return 0; } set { DataSize = value; } } 

您不断地设置DataSize 的值。您需要创建一个局部变量并使用它。例如

private long dataSize;

public long DataSize
{
    get { return this.dataSize; }
    set { this.dataSize = value; }
}

编辑
我已经编写了 DataSize 但同样适用于 DataType

Answer 2

正如其他人所说，发生堆栈溢出是因为您的属性设置器只是调用自身。如果您将其视为一种方法，可能会更容易理解：

// This obviously recurses until it blows up
public void SetDataType(long value)
{
    SetDataType(value);
}

据我了解，您正在尝试创建普通属性但具有默认值，对吧？

在这种情况下，您需要由 setter 设置的支持变量 - 并且 getter 也应该返回这些变量。 变量应该获得默认值：

private long dataSize = 0;
public long DataSize {
  get { return dataSize; }
  set { dataSize = value; }
}

private EnumDataType dataType = EnumDataType.Apple;
public EnumDataType DataType { 
  get { return dataType; }
  set { dataType = value; }
}

或者，使用自动属性，但在构造函数中设置默认值：

public long DataSize { get; set; }
public EnumDataType DataType { get; set; }

public DataRequest()
{
    DataSize = 0; // Not really required; default anyway
    DataType = EnumDataType.Apple;
}

Answer 3

发生堆栈溢出是因为在设置器中您将属性设置为一个值（即您试图将某些东西设置为某个值...这会导致无限循环）...这意味着它尝试将自己设置为value 这意味着它尝试将自己设置为一个值，直到繁荣

您的属性将永远不会获得您正在设置的值，因为它们总是返回相同的值（而不是存储的值）

public enum EnumDataType { Raspberry, Orange, Pear, Apple }; 

public class DataRequest 
{ 
private long _dataSize = 0;
private EnumDataType _dataType = EnumDataType.Apple;

public long DataSize { get { return _dataSize ; } set { _dataSixe= value; } } 
public EnumDataType DataType  { get { return _dataType; } set { _dataType= value; } } 
} 

是您真正想要的

Answer 4

你必须用后备存储来实现它：

private EnumDataType dataType;
public EnumDataType DataType  { get { return EnumDataType.Apple; } set { dataType = value; } }

}

你应该在任何时候在 getter 和 setter 中执行一些操作时这样做。顺便说一句，为什么你甚至可以设置变量？你无法读出它们，你总是得到 EnumDataType.Apple。如果你想要一个起始值，你可以这样做：

private EnumDataType dataType = EnumDataType.Apple;
public EnumDataType
{
   get
   {
       return dataType;
   }
   set
   {
       dataType = value;
   }
 }

Answer 5

我不明白第一行如何：
请求数据大小 = 60;

不会导致堆栈溢出 - 我的建议是使用支持属性：

public class DataRequest
{
    protected int dataSize = 0;
    protected EnumDataType enumDataType;
    public long DataSize { get { return 0; } set { dataSize = value; } }
    public EnumDataType DataType  { get { return EnumDataType.Apple; } set { enumDataType = value;} 
}