c# - C# override in Finalize()

Question

protected override void Finalize()
        {
            try
            {
                this.FtcpSock.Close();
                this.FudpSock6800.Close();
                this.FudpSock6801.Close();
                this.FudpSock6802.Close();
                this.FudpSock6803.Close();
                this.FudpSock6804.Close();
                this.FudpSock6806.Close();
            }
            finally
            {
                this.Finalize();
            }
        }

I m getting this Error Message:

Error 1 Do not override object.Finalize. Instead, provide a destructor.

by the way, this is Original Code complied by third party company.
how to solve this problem? how to Finalize with override?

Answer 1

As the message states, you shouldn't override Finalize directly but instead provide a destructor. Ignoring that though the code has one other fatal problem

protected override void Finalize() {
  try { 
    ...
  } finally { 
    this.Finalize();
  }
}

This code will eventually lead to a stack overflow in the code. It is recursively calling itself in a finally block and hence will just loop until you run out of stack space. The code was likely meant to use the following instead

base.Finalize();

This will chain to the implementation of Finalize on the base type

Answer 2

如果您可以编辑代码，则可以将代码移至析构函数。对于名为“Form1”的表单，析构函数如下所示：

~Form1()
{
    this.FtcpSock.Close();
    this.FudpSock6800.Close();
    this.FudpSock6801.Close();
    this.FudpSock6802.Close();
    this.FudpSock6803.Close();
    this.FudpSock6804.Close();
    this.FudpSock6806.Close();
}

http://msdn.microsoft.com/en-us/library/66x5fx1b.aspx