Garbage collection and Finalization

• improper implementation of the Dispose pattern
• usage of managed objects in .NET finalizators, and so on.

public class FileHandler
	{
		private StreamReader sr;
		private string _fname;
 
		public FileHandler(string filename)
		{
			sr = new StreamReader(filename);
		}
 
		public void PerformRead()
		{
			string line = null;
			while ((line = sr.ReadLine()) != null)
			{
				if (line.Contains("declare"))
					line = line.Trim();
			}
		}
 
		public void Close()
		{
			sr.Close();
		}
	}
 
	public sealed class FileReader
	{
		private FileHandler fileHandler;
 
		public FileReader(string fileName)
		{
			fileHandler = new FileHandler(fileName);
		}
 
		public void Read()
		{
			FileTools.ReadFile(fileHandler);
		}
 
		~FileReader()
		{
			Console.WriteLine("Closing vie finalization");
			fileHandler.Close();
		}
 
		public void Dispose()
		{
			fileHandler.Close();
			GC.SuppressFinalize(this);
		}
	}
 
	public static class FileTools
	{
		public static void ReadFile(FileHandler fileHandler)
		{
			if (fileHandler != null)
				fileHandler.PerformRead();
		}
	}
	
	class Program
	{
		static void Main(string[] args)
		{
			FileReader fr = new FileReader("myfile.txt");
			fr.Read();
		}
	}