【问题标题】:Delphi dictionary freeingDelphi 字典释放
【发布时间】:2015-08-26 08:22:55
【问题描述】:

我实现了以下类:

type
  TUtilProcedure = procedure(var AJsonValue: TJSONObject);

  TCallback = class
  private
    FName: string;
    FProcedure: TUtilProcedure;
    FAnnotation: string;
  public
    constructor Create(AName: string; AProcedure: TUtilProcedure; AAnnotation: string); overload;
    constructor Create(ACallback: TCallback); overload;
    property Name: string read FName;
    property Proc: TUtilProcedure read FProcedure;
    property Annotation: string read FAnnotation;
 end;

然后我有一个全局变量:

procedures: TDictionary<string, TCallback>;

OnFormActivate 过程中,我初始化了procedures 变量:

procedures := TDictionary<string, TCallback>.Create();
procedures.Add('something', TCallback.Create('sth', @proc, 'annotation')); 
// ....

然后在OnFormClose我释放它:

procedures.Clear;
procedures.Free;

我的代码会泄漏内存吗?如果是这样,释放dictionary 的正确方法是什么? 据我所知,迭代不是好主意。

【问题讨论】:

  • 使用对象时使用TObjectDictionary,如果要自动释放值,请在创建字典时使用[doOwnsValues]
  • TCallback 是否需要 destructor
  • TCallback 现在不需要析构函数
  • @fantaghirocco 谢谢
  • 不要在过程中使用@。这会抑制编译器的类型安全。启用 Embarcadero 仍然拒绝启用的类型化地址编译器选项,这样做会让所有 Delphi 初学者遭受不必要的痛苦。

标签: delphi dictionary memory-leaks tdictionary


【解决方案1】:

代码泄漏内存,因为 TDictionary 中包含的对象未自动释放。

如果您需要将对象存储在字典中,采用TObjectDictionary 是一种更好的方法。

如果您希望字典中包含的对象自动释放,请在创建集合实例时使用doOwnsValues 标志。

  • 当变量是真正的全局变量时(即在单元的interface 部分的var 下声明),它应该在initializationfinalization 中创建和销毁单元本身的部分。

    . . .
    var
      procedures: TObjectDictionary<string, TCallback>;
    . . .
    initialization
      procedures:= TObjectDictionary<string, TCallback>.Create([doOwnsValues]);
    finalization
      procedures.Free;
    
  • 当您的变量属于表单类本身时,您应该在表单的OnCreate 事件中创建字典。

    . . .
    public
      procedures: TObjectDictionary<string, TCallback>;
    . . .
    procedure TForm1.FormCreate(Sender: TObject);
    begin
      procedures:= TObjectDictionary<string, TCallback>.Create([doOwnsValues]);
    end;
    

    在表单的OnDestroy 事件中释放字典:

    procedure TForm1.FormDestroy(Sender: TObject);
    begin
      procedures.Free;
    end;
    
  • 此外,如果您希望在不需要类本身的实例的情况下访问属于该类的变量(这在许多编程语言中称为静态变量),您可以将字典声明为class var,并可选择通过class property 访问它;在这种情况下,最好在class constructorclass destructor 中创建和销毁集合。

    . . .
    TMyClass = class
      private
        class constructor Create;
        class destructor Destoy;
      public
        class var procedures: TObjectDictionary<string, TCallback>;
    end;
    . . .
    class constructor TMyClass.Create;
    begin
      procedures := TObjectDictionary<string, TCallback>.Create([doOwnsValues]);
    end;
    
    class destructor TMyClass.Destoy;
    begin
      procedures.Free;
    end;
    

TCallback = class
  private
    FName: string;
    FProcedure: TUtilProcedure;
    FAnnotation: string;
  public
    constructor Create(AName: string; AProcedure: TUtilProcedure; AAnnotation: string); overload;
    constructor Create(ACallback: TCallback); overload;
    property Name: string read FName;
    property Proc: TUtilProcedure read FProcedure;
    property Annotation: string read FAnnotation;
end;

附带说明,TCallback 类不需要指定析构函数,因为它只拥有两个字符串和一个指向过程的指针。因此,从TObject 继承的默认析构函数就足够了。

【讨论】:

  • 在这种情况下没问题,只有一个表单实例。如果有多个表单实例,使用 FormCreate 和 FormDestroy 来构造或销毁这样的变量可能不是那么好。但是OTOH,无论如何都不推荐使用全局变量。
  • 嗯...如果变量必须是全局的,出于某种原因,为什么不在初始化和终结部分分别构造和销毁它?
  • @Rudy Velthuis 谢谢你的cmets:我已经根据你的评论更新了答案; @David Heffernan 已经指出了多表单实例问题,但我错误地认为 global 变量可能是 private one
猜你喜欢
  • 2023-03-30
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2018-03-14
相关资源
最近更新 更多