delphi - TDataModule 后代 'created' 没有 .Create 但没有问题？

Question

我突然注意到我们的代码库中有一个TDataModuleTestExchange(nil)“构造函数调用”:

procedure TDialoogConfigExchange.ButtonTestClick(Sender: TObject);
var
   lDataModuleTestExchange: TDataModuleTestExchange;
   lResult                : Boolean;
begin
   inherited;
   [snip]
   begin
      lDataModuleTestExchange := TDataModuleTestExchange(nil);  // *** HERE ***
      try
         lResult := lDataModuleTestExchange.GetCalendarFolder(EditHost.Text,EditGebruiker.Text,EditWachtwoord.Text);
         if lResult then
            ToonMelding(sExchangeTestGelukt, mtInformation, [mbOk])
         else
            ToonMelding(Meldingen.Text, mtError, [mbOK]);
      finally
         lDataModuleTestExchange.Free;
      end;
   end;
end;

因此，代替 TDataModuleTestExchange.**Create**(nil) 这工作得很好!

unit dmTestExchange;

interface

uses
  System.SysUtils, System.Classes,
  Xml.XMLDoc, Xml.XMLIntf, Xml.XMLDOM,
  TimeTellDM;

type
  TDataModuleTestExchange = class(TTimeTellDataModule)  // TDataModule descendant
  private
  public
    function GetCalendarFolder(const AExchangeServerURL,AExchangeLoginName,AExchangePass: String): Boolean;
  end;

没有编译器错误，没有运行时问题。怎么会这样？

Answer 1

首先，值得指出的是， Actor 阵容是虚假的，除了混淆视听之外没有任何目的。该代码相当于:

lDataModuleTestExchange := nil;

TDataModuleTestExchange.GetCalendarFolder 是一个实例方法，您在 nil 引用上调用它。如果该方法尝试访问实例中的任何字段，或者调用虚拟方法，或者实际上依赖于实例的任何内容，这将导致运行时错误。因此，TDataModuleTestExchange.GetCalendarFolder 的实现似乎不依赖于实例。尽管您似乎在这里逃脱了惩罚，但编写这样的代码显然是非常糟糕的形式。

该类可能应该被重写以声明静态类方法，如下所示:

type
  TDataModuleTestExchange = class(TTimeTellDataModule)  
  public
    class function GetCalendarFolder(const AExchangeServerURL,  
      AExchangeLoginName, AExchangePass: string): Boolean; static;
  end;

然后这样调用:

lResult := TDataModuleTestExchange.GetCalendarFolder(EditHost.Text,
  EditGebruiker.Text, EditWachtwoord.Text);
if lResult then
  ToonMelding(sExchangeTestGelukt, mtInformation, [mbOk])
else
  ToonMelding(Meldingen.Text, mtError, [mbOK]);