这是我的代码。我无法弄清楚为什么这段代码给出了'由于StackOverFlowException导致进程终止'。
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
namespace SadiSDAL4
{
class Program
{
static void Main(string[] args)
{
PrivateVehical privateV = new PrivateVehical("001");
PrivateVehical pVehical = (PrivateVehical)privateV.Clone();
Console.WriteLine("Cloned : {0}", privateV.name);
Console.WriteLine("Cloned : {0}", pVehical.name);
privateV.name = "Sadia's Car";
Console.WriteLine("Cloned : {0}", privateV.name);
Console.WriteLine("Cloned : {0}", pVehical.name);
pVehical.name = "Waheed's Car";
Console.WriteLine("Cloned : {0}", privateV.name);
Console.WriteLine("Cloned : {0}", pVehical.name);
Console.WriteLine(privateV.GetHashCode().ToString());
Console.WriteLine(pVehical.GetHashCode().ToString());
PublicVehical publicV = new PublicVehical("002");
Console.WriteLine(publicV.id);
PublicVehical pubVehi = (PublicVehical)publicV.Clone();
Console.WriteLine("Cloned : {0}", pubVehi.id);
}
}
abstract class Vehical
{
private string vehicalId = "01";
public string name = "Car_1";
public Vehical(string id)
{
this.vehicalId = id;
}
public string id
{
get { return id; }
set { this.vehicalId = id; }
}
public abstract Vehical Clone();
}
class PrivateVehical : Vehical
{
public PrivateVehical(string id)
: base(id)
{
}
public override Vehical Clone()
{
return (Vehical)this.MemberwiseClone();
}
}
class PublicVehical : Vehical
{
public PublicVehical(string id)
: base(id)
{
}
public override Vehical Clone()
{
return (Vehical)this.MemberwiseClone();
}
}
}
这是输出。 有人能解释一下它的原因是什么吗?为什么它在第一部分和第一部分工作不在另一个?
答案 0 :(得分:25)
看看这段代码:
public string id
{
get { return id; }
set { this.vehicalId = id; }
}
具体来说是get { return id; }
。您将财产归还给自己,导致SO错误。
这是你的意图吗?
public string id
{
get { return this.vehicalId; }
set { this.vehicalId = value; }
}
答案 1 :(得分:6)
您的错误发生在这行代码
上Console.WriteLine(publicV.id);
当您尝试从publicV
变量获取属性的值时,它会调用基类的getter。
在您的情况下,Vehicle
类
public string id
{
get { return id; }
set { this.vehicalId = id; }
}
但是,当调用get
时,它会执行return id;
。因此它再次调用id属性的getter,它再次执行return id;
代码行。
因此,有效的代码行无法自行调用自身,从而产生Stack Overflow问题。
(如果您不熟悉此概念,请参阅Recursion)
再次查看Vehicle的id属性。
您对set
版本有正确的想法,您可以将值分配给vehicleId
属性。这也是你应该从中得到它的地方。
我希望如果你改变吸气剂
get { return this.vehicleId; }
这将解决您的问题。
答案 2 :(得分:3)
调试代码显示您有剪切和粘贴错误。
public string id
{
// having this set to id causes endless recursion
//get { return id; }
//use this instead
get { return vehicalId; }
set { this.vehicalId = id; }
}
答案 3 :(得分:0)
您的错误是由以下定义引起的:
public string id
{
get { return id; }
set { this.vehicalId = id; }
}
这是因为get调用不断返回对自身的调用,即get Id调用get Id反复调用,直到您的内存不足并获得StackOverflowException为止。我猜你打算这样做:
public string id
{
get { return vehicalId; }
set { this.vehicalId = id; }
}
顺便说一句,感觉真的很不错,可以通过stackoverflow获得有关调试StackOverflowException的帮助:)