aDialog created inside a Try/Finally, bad practise?

[Skrim 11]

try
    aDialog:=TMyDialog.Create(self);
   ...

   ...
finally
    aDialog.Free;
end;

 

I keep getting warnings about a dialog created inside a Try/Finally, why is it bad practise?

 

[Uwe Raabe 2064]

Because aDialog is not assigned when an exception is raised inside TMyDialog.Create. That causes Free called for an uninitialized variable inside the finally block.

 

A workaround would be to set aDialog to nil before the try, but the cleaner way is to move the Create outside the try-finally.