Problème de libération de ressource sur un tableau d'objet

**ffert2907** · 31/03/2006, 13h02

Mon prog sert à afficher un agenda paramètrable en vue, jour, semaine, mois, année.. et multi année...
(il n'est pas encore terminé.. donc.. indulgence. mais là je bloque !! HELP PLEASE !!!)

J'ai une erreur bizarre lors de la libération de la ressource d'un tableau d'objet. en effet, mon tableau contient au départ 48 objets.
la libération de l'objet[0] se passe correctement dans la boucle et la libération de l'objet[1] donne un access violation.
J'en perds mon latin !!!!

(notez au passage la classe TDateTimeEx : qui est une extension du Datetime pour permettre plein de fonctions de calcul sur les dates...
non terminée encore... des bugs ici aussi !! mais moins important pour l'instant.)

Voici le code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
 
unit Unit1;
 
interface
 
uses
  Windows, Messages, SysUtils, Variants, Classes, Graphics, Controls, Forms,
  Dialogs, ExtCtrls, Buttons, StdCtrls, Math, ComCtrls, DateUtils;
 
 
const
  TJoursemaine : array[1..7] of string = ('Lundi', 'Mardi', 'Mercredi', 'Jeudi', 'Vendredi', 'Samedi', 'Dimanche');
  TAgendaType : array[0..4] of string = ('Jour', 'Semaine', 'Mois', 'Année', 'Multi Années');
 
type
 
  TCaseType = (ctday, ctweek, ctmonth, ctyear, ctMultiyear);
 
  TDTExtract = record
    Year  : word;
    Month : word;
    Day   : word;
    Hour  : word;
    Min   : word;
    Sec   : word;
    Milli : word;
  end;
 
  TDateTimeEx = Class(TComponent)
  Private
    FDate : TDate;
    FTime : TTime;
    FDateTime : TDateTime;
    FValue : Double;          // Valeur numérique de la date
    FDTExtract : TDTExtract; // Date décomposée... accés à la modif de date par la décomposition : year, Month, Day, Hour, Min, Sec, Milli
    FDayOfTheWeek : Word;   // 1..7
    FDayOfTheMonth : word;  // 1..31
    FDayOfTheYear : word;   // 1..366
    FWeekOfTheMonth : word; // 1..6
    FWeekOfTheYear : word;  // 1..53
    FItems : TStrings;
    FType : TStrings;
 
    Procedure SetDateTime(InVal : TDateTime);
    Procedure SetDate(InVal : TDate);
    Procedure SetTime(InVal : TTime);
    Function  ReadDayOfTheWeek : word;
    Procedure SetDayOfTheWeek(InVal : word);
    Function  ReadDayOfTheMonth : word;
    Procedure SetDayOfTheMonth(InVal : word);
    Function  ReadDTExtract : TDTExtract;
    Procedure SetDTExtract(InVal : TDTExtract);
 
  Public
    constructor Create(AOwner: TComponent); override;       // ; dn : TDateTime
 
    Property DDateTime : TDateTime read FDateTime Write SetDateTime;
    Property DDate : TDate read FDate Write SetDate;
    Property DTime : TTime read FTime Write SetTime;
    Property DExtract : TDTExtract read ReadDTExtract Write SetDTExtract;
    Property DDayOfTheWeek : word read ReadDayOfTheWeek Write SetDayOfTheWeek;
    Property DDayOfTheMonth : word read ReadDayOfTheMonth Write SetDayOfTheMonth;
 
    Function ReadDecodeDateTime(InVal : TDateTime) : TDTExtract;
    Function ReadEncodeDateTime(InVal : TDTExtract) : TDateTime;
 
  end;
 
 
  TAgenda = Class(TPanel)  //ScrollBox)
  private
    Drawing : boolean;   // si l'on est en train de dessiner
    SB : TScrollBox;
    TB : TPanel;
    NBNext : TSpeedButton;
    NBPrevious: TSpeedButton;
    DTP : TDateTimePicker;
    CBSTyle : TComboBox;
 
    FCreate : boolean;      // si l'on est en mode création
    FDateActive : TDateTime;
    FSTartDate : TDateTime;
    FType : TCaseType;      // type de vue : ctday, ctweek, ctmonth, ctyear, ctMultiyear
    FColorFond : Tcolor;
    FYearCount : integer;
    DCase : Array of TPanel;   //Array of
    Capt : TLabel;
    FDaTiEx : array of TDateTimeEx;
    FLegend : array of TLabel;
    FDayOfTheWeekName : Array [1..7] of string;
    FAgendaType : Array [0..4] of string;
 
    procedure AgendaResize(Sender : TObject);
 
  Public
    constructor Create(AOwner: TComponent); override;
 
    Property Date : TDateTime read FDateActive Write FDateActive;
    Property StartDate : TDateTime read FSTartDate Write FSTartDate;
    Property CaseType : TCaseType read FType Write FType;
    Property YearCount : Integer read  FYearCount write FYearCount;
    Property Color; // : Tcolor read FColorFond Write FColorFond;
 
    Procedure ResizeAll(NewSizeLegend, NewSizeDCase, NewSizeDaTiEx : integer);
    Procedure ClearSetDaTiEx(NewSize : integer);
    Procedure ClearSetLegend(NewSize : integer);
    Procedure CreateLegend(ai, aLeft, aTop : integer ; Caption : String );
    Procedure ClearSetDCase(NewSize : integer);
    Procedure Draw(SelectDateTime : TDateTime);
    Procedure CBStyleChange(Sender: TObject);
  end;
 
 
 
  TForm1 = class(TForm)
    SB: TScrollBox;
    SpeedButton1: TSpeedButton;
    Panel1: TPanel;
    Button1: TButton;
    DateTimePicker1: TDateTimePicker;
    ComboBox1: TComboBox;
    procedure SpeedButton1Click(Sender: TObject);
  private
    { Private declarations }
  public
    { Public declarations }
  end;
 
var
  Form1: TForm1;
  Agenda : Tagenda;
 
implementation
 
{$R *.dfm}
 
 
// *****************************************************************************
 
constructor TDateTimeEx.Create(AOwner: TComponent);
begin
  // créer Ma Date Etendue : DateTimeEx
  Inherited Create(AOwner);
  DDateTime := now;
end;
 
Procedure TDateTimeEx.SetDateTime(InVal : TDateTime);
begin
  // Mettre date et heure à jour
  FDateTime := Inval;
  FDate := FDateTime;
  FTime := FDateTime;
end;
 
Procedure TDateTimeEx.SetDate(InVal : TDate);
begin
  // Mettre la date à jour
  FDate := Inval;
  FDateTime := FDate + FTime;
end;
 
Procedure TDateTimeEx.SetTime(InVal : TTime);
begin
  // Mettre l'heure à jour
  FTime := Inval;
  FDateTime := FDate + FTime;
end;
 
Function TDateTimeEx.ReadDecodeDateTime(InVal : TDateTime) : TDTExtract;
var
  tmp : TDTExtract;
begin
  // exploser la date et l'heure.
  DecodeDateTime(Inval, tmp.Year, tmp.Month, tmp.Day, tmp.Hour, tmp.Min, tmp.Sec, tmp.Milli);
  ReadDecodeDateTime := tmp;
end;
 
Function TDateTimeEx.ReadEncodeDateTime(InVal : TDTExtract) : TDateTime;
begin
  // exploser la date et l'heure.
  ReadEncodeDateTime := EncodeDateTime(InVal.Year, InVal.Month, InVal.Day, InVal.Hour, InVal.Min, InVal.Sec, InVal.Milli);
end;
 
 
Procedure TDateTimeEx.SetDTExtract(InVal : TDTExtract);
Begin
  // Faire un Set de la date par la composition des date extraite...
  FDateTime := ReadEncodeDateTime(FDTExtract);
  FDate := FDateTime;
  FTime := FDateTime;
end;
 
Function TDateTimeEx.ReadDTExtract : TDTExtract;
Begin
  // Extraire la date décomposée
  FDTExtract := ReadDecodeDateTime(FDateTime);
end;
 
Procedure TDateTimeEx.SetDayOfTheMonth(InVal : word);
begin
  // Set de la valeur du jour du moi
  FDTExtract := ReadDecodeDateTime(FDateTime);
  FDTExtract.Day := InVal;
  FDateTime := ReadEncodeDateTime(FDTExtract);
  FDate := FDateTime; //FDate + Inval;
end;
 
Function TDateTimeEx.ReadDayOfTheMonth : word;
begin
  // retourner la valeur DayOfTheMonth
  ReadDayOfTheMonth := DayOfTheMonth(FDateTime);
end;
 
Procedure TDateTimeEx.SetDayOfTheWeek(InVal : word);
var
  adotw : word;
begin
  // Set de la valeur du jour du moi
  adotw := DayOfTheMonth(FDateTime);
  FDateTime := FDateTime + (Inval - adotw);
  FDate := FDateTime; //FDate + Inval;
end;
 
Function TDateTimeEx.ReadDayOfTheWeek : word;
begin
  // retourner la valeur DayOfTheMonth
  ReadDayOfTheWeek := DayOfTheWeek(FDateTime);
end;
 
// *****************************************************************************
//                 L'AGENDA
// *****************************************************************************
 
 
constructor TAgenda.Create(AOwner: TComponent);
begin
  Drawing := True;
  FCreate := true;
  inherited Create(AOwner);
 
  SetLength(FDaTiEx, 0);
 
  // Couleur de fond
  TB := TPanel.Create(self);
  TB.Parent := Self;
  TB.Top := 0;
  TB.Left := 0;
  TB.Align := alTop;
  TB.Height := 44;
 
  // Boutons Next
  NBNext := TSpeedButton.Create(TB);
  NBNext.Parent := TB;
  NBNext.Top := 2;
  NBNext.Width := 38;
  NBNext.Height := 38;
  NBNext.Left := TB.Width - 2 - NBNext.Width;
  NBNext.Anchors := [akTop,akRight];  // [akLeft,akTop,akRight,akBottom];
 
  // Boutons Previous
  NBPrevious := TSpeedButton.Create(TB);
  NBPrevious.Parent := TB;
  NBPrevious.Top := 2;
  NBPrevious.Width := 38;
  NBPrevious.Height := 38;
  NBPrevious.Left := 2;
  NBPrevious.Anchors := [akLeft,akTop];  // [akLeft,akTop,akRight,akBottom];
 
  // créer le combo de date
  DTP := TDateTimePicker.Create(TB);
  DTP.Parent := TB;
  DTP.Name := 'DTP_1';
  DTP.DateTime := FDateActive;
  DTP.Top := 2;
  DTP.Left := 5;
  DTP.Width := 150;
  DTP.Left := trunc((TB.Width / 2) - (DTP.Width / 2));
 
  // créer le combo du type d'affichage
  CBSTyle := TComboBox.Create(TB);
  CBSTyle.Parent := TB;
  CBSTyle.Name := 'CBStyle_1';
  CBSTyle.Name := '';
  CBSTyle.Top := 1 + DTP.Top + DTP.Height;
  CBSTyle.Left := 5;
  CBSTyle.Width := 150;
  CBSTyle.Left := trunc((TB.Width / 2) - (DTP.Width / 2));
  CBSTyle.OnChange := CBStyleChange;
 
  FDayOfTheWeekName[1] := TJoursemaine[1];
  FDayOfTheWeekName[2] := TJoursemaine[2];
  FDayOfTheWeekName[3] := TJoursemaine[3];
  FDayOfTheWeekName[4] := TJoursemaine[4];
  FDayOfTheWeekName[5] := TJoursemaine[5];
  FDayOfTheWeekName[6] := TJoursemaine[6];
  FDayOfTheWeekName[7] := TJoursemaine[7];
 
  FAgendaType[0] := TAgendaType[0];
  FAgendaType[1] := TAgendaType[1];
  FAgendaType[2] := TAgendaType[2];
  FAgendaType[3] := TAgendaType[3];
  FAgendaType[4] := TAgendaType[4];
 
  // créer la scrollBox
  SB := TScrollBox.Create(self);
  SB.Parent := self;
  SB.Color := Color;
  SB.Align := alClient;
 
  CaseType := ctmonth;
 
  Color := FColorFond;
  OnResize := AgendaResize;
 
  // Fin de création...
  FCreate := False;
  Drawing := False;
end;
 
procedure TAgenda.AgendaResize(Sender : TObject);
begin
  // redimensionner
//  If not(FCreate) then
//    Draw(FDateActive);
  DTP.Left := trunc((TB.Width / 2) - (DTP.Width / 2));
  CBSTyle.Left := trunc((TB.Width / 2) - (DTP.Width / 2));
 
end;
 
Procedure TAgenda.ResizeAll(NewSizeLegend, NewSizeDCase, NewSizeDaTiEx : integer);
begin
  // Tout redimensionner facilement !! ;)
  ClearSetLegend(NewSizeLegend);
  ClearSetDCase(NewSizeDCase);
  ClearSetDaTiEx(NewSizeDaTiEx);
end;
 
 
 
 
// --------------------------------------------------------------------------------
// --------------------------------------------------------------------------------
//     ERREUR ICI, ClearLegend[0], passe bien, ressource bien supprimée,
//                            [1], Access violation !!!
//                            alors que mon tableau fait 48 objet dés le départ de l'App
// --------------------------------------------------------------------------------
// --------------------------------------------------------------------------------
 
 
Procedure TAgenda.ClearSetLegend(NewSize : integer);
var
  i : integer;
begin
  // vider et libérer les ressources de DaTiEx
  If High(FLegend) > -1 then
  begin
    i := 0;
    While i < High(FLegend) Do
    begin
  Showmessage('Clear FLegend : ' + inttostr(i) + ' - ' + inttostr(High(FLegend)) + ' - caption=' + FLegend[i].Caption);
      If FLegend[i] <> nil then
        FLegend[i].Destroy;
//  Showmessage(inttostr(i) + ' - ' + inttostr(High(FLegend)) + ' - caption=' + FLegend[i].Caption);
      inc(i);
//     Showmessage(inttostr(i) + ' - ' + inttostr(High(FLegend)));
    end;
  end;
  // Taille du tableau := NewSize;
  SetLength(FLegend, NewSize);
end;
 
Procedure TAgenda.CreateLegend(ai, aLeft , aTop: integer ; Caption : String );
begin
  // créer une ligne de légende
  Showmessage('Create FLegend=' + inttostr(ai) + ' - ' + Caption);
  If FLegend[ai] = nil then
  begin
    If High(FLegend) <= ai then
      SetLength(FLegend, ai+1);
    FLegend[ai] := TLabel.Create(SB);
    FLegend[ai].Parent := SB;
    FLegend[ai].name := 'Label_' + inttostr(ai);
  end;
  // affecter les nouvelles valeurs
  FLegend[ai].caption := Caption;
  FLegend[ai].Left := aLeft;
  FLegend[ai].Top := aTop;
end;
 
Procedure TAgenda.ClearSetDaTiEx(NewSize : integer);
var
  i : integer;
begin
  // vider et libérer les ressources de DaTiEx
  If High(FDaTiEx) > -1 then
  begin
    i := 0;
    While i < High(FDaTiEx) Do
    begin
      If FDaTiEx[i] <> nil then
        FDaTiEx[i].Destroy;
      inc(i);
    end;
  end;
  // Taille du tableau := NewSize;
  SetLength(FDaTiEx, NewSize);
end;
 
Procedure TAgenda.ClearSetDCase(NewSize : integer);
var
  i : integer;
begin
  // vider et libérer les ressources de DCase
  If High(DCase) > -1 then
  begin
    i := 0;
    While i < High(DCase) Do
    begin
      If DCase[i] <> nil then
        DCase[i].Destroy;
      inc(i);
    end;
  end;
  // Taille du tableau := NewSize;
  SetLength(DCase, NewSize);
end;
 
procedure TAgenda.CBStyleChange(Sender: TObject);
begin
  // changement de format : jour, semaine, année.. multi année....
  // ctday, ctweek, ctmonth, ctyear
  If not(FCreate) then
  begin
    If not(Drawing) then
    begin
      If CBSTyle.Text = CBSTyle.Items[0] then
      begin
        CaseType := ctday;
        ResizeAll(48, 48, 48);    // NewSizeLegend, NewSizeDCase, NewSizeDaTiEx
        Draw(self.FDateActive);
      end
      Else If CBSTyle.Text = CBSTyle.Items[1] then
      begin
        CaseType := ctweek;
        ResizeAll(7, 7, 7);    // NewSizeLegend, NewSizeDCase, NewSizeDaTiEx
        Draw(self.FDateActive);
      end
      Else If CBSTyle.Text = CBSTyle.Items[2] then
      begin
        CaseType := ctmonth;
        ResizeAll(7, 42, 42);    // NewSizeLegend, NewSizeDCase, NewSizeDaTiEx
        Draw(self.FDateActive);
      end
      Else If CBSTyle.Text = CBSTyle.Items[3] then
      Begin
        CaseType := ctyear;
        ResizeAll(12, 372, 372);    // NewSizeLegend, NewSizeDCase, NewSizeDaTiEx
        Draw(self.FDateActive);
      end;
    end;
  end;
end;
 
Procedure TAgenda.Draw(SelectDateTime : TDateTime);
var
  NoCase : integer;   // N° de la case crée actuellement.
//  AffCase : boolean;  // si l'on doit afficher la case
//  DayOffset : integer;
  Nbre : integer;
  PasX : integer;
  PasY : integer;
  NbCol : integer;
  NbLig : integer;
  OffsetX : integer;
  OffsetY : integer;
  i, ii : integer;
  PasTime : Double;
begin
  // redessiner...
  FDateActive := SelectDateTime;
  Drawing := True;
 
  If CBSTyle.Items.Count < 2 then
  begin
    // Si la liste des différentes vues possible
    //  n'est pas encore renseignée on la renseigne... une fois pour toute.
    CBStyle.Items.Clear;
    CBSTyle.Items.Add(FAgendaType[0]);
    CBSTyle.Items.Add(FAgendaType[1]);
    CBSTyle.Items.Add(FAgendaType[2]);
    CBSTyle.Items.Add(FAgendaType[3]);
    CBSTyle.Items.Add(FAgendaType[4]);
  end;
 
  StartDate := DTP.DateTime;
  If CaseType = ctday then
  begin
    CBSTyle.Text := CBSTyle.Items[0];
    Nbre := 48;
    NbCol := 1;
    NbLig := Nbre;
    OffsetX := 20;
    OffsetY := 60;
    PasX := Max(OffsetX, trunc((SB.Height - (OffsetX + 10)) / NbCol));
    PasY := Max(OffsetY, trunc((SB.Width - (OffsetY + 10)) / NbLig));
    PasTime := (1 / (60 * 60 * 24 * 1000)) / 48;
    // Commentaires et repères et légende
    For i := 0 to  NbLig -1 Do
    begin
      CreateLegend(i, 4, 50 + Trunc(PasY / 2) + i * PasY,  inttostr(trunc(i / 2)));
    end;
  end
  else If CaseType = ctweek then
  begin
    CBSTyle.Text := CBSTyle.Items[1];
    Nbre := 7;
    NbCol := 1;
    NbLig := Nbre;
    OffsetX := 60;
    OffsetY := 30;
    PasTime := 1;
    PasX := Max(OffsetX, trunc((SB.Height - (OffsetX + 10)) / NbCol));
    PasY := Max(OffsetY, trunc((SB.Width - (OffsetY + 10)) / NbLig));
    If DayOfTheWeek(StartDate) > 1 then
      StartDate := StartDate - (DayOfTheWeek(StartDate) - 1);
    // Commentaires et repères et légende
    For i := 0 to  NbLig -1 Do
    begin
      CreateLegend(i, 4, 20 + Trunc(PasY / 2) + i * PasY,  FDayOfTheWeekName[1 + i]);
    end;
  end
  else if CaseType = ctmonth then
  begin
    CBSTyle.Text := CBSTyle.Items[2];
    Nbre := 31;
    NbCol := 7;
    NbLig := 6;
    OffsetX := 20;
    OffsetY := 30;
    PasX := 70; //Max(OffsetX, trunc((SB.Height - (OffsetX + 10)) / NbCol));
    PasY := Max(OffsetY, trunc((SB.Width - (OffsetY + 10)) / NbLig));
    PasTime := 1;
    // Se caler sur le 1er lundi précédent :
    StartDate := StartDate - (1 - DayOfTheWeek(StartDate));
    // Commentaires et repères et légende
    For i := 0 to  NbCol -1 Do
    begin
      CreateLegend(i, 30 + i * PasX, 10,  Copy(FDayOfTheWeekName[1 + i],0, 3) + '.');
    end;
 
  end
  else if CaseType = ctyear then
  begin
    CBSTyle.Text := CBSTyle.Items[3];
    Nbre := 366;
    NbCol := 31;
    NbLig := 12;
    OffsetX := 20;
    OffsetY := 60;
    PasX := Max(OffsetX, trunc((SB.Height - (OffsetX + 10)) / NbCol));
    PasY := Max(OffsetY, trunc((SB.Width - (OffsetY + 10)) / NbLig));
    PasTime := 1;
  end
  else if CaseType = ctMultiyear then
  begin
    CBSTyle.Text := CBSTyle.Items[4];
    Nbre := 366 * YearCount;
    NbCol := 366;
    NbLig := YearCount;
    OffsetX := 20;
    OffsetY := 60;
    PasX := Max(OffsetX, trunc((SB.Height - (OffsetX + 10)) / NbCol));
    PasY := Max(OffsetY, trunc((SB.Width - (OffsetY + 10)) / NbLig));
    PasTime := 1;
  end;
 
  // créer les cases...
  NoCase := 0;
  For i := 0 To NbCol - 1 Do    // Colonnes
  begin
    For ii := 0 to NbLig  Do    // Lignes
    begin
      FDaTiEx[NoCase] := TDateTimeEx.Create(self);
      FDaTiEx[NoCase].FDateTime := (StartDate ) + (NoCase * PasTime);  //((1 + i) * (1 + ii) * PasTime);
      DCase[NoCase] := TCase.Create(SB);
      DCase[NoCase].Name := 'Case_' + inttostr(i) + '_' + inttostr(ii);
      DCase[NoCase].Tag := NoCase;
      DCase[NoCase].Caption := DateTimeTostr(FDaTiEx[NoCase].FDateTime); //'Case_' + inttostr(i) + '_' + inttostr(ii);
      DCase[NoCase].Parent := SB;
      DCase[NoCase].Font.Size := 6;
      if CaseType = ctmonth then
      begin
        // Horizontal
        DCase[NoCase].Left := OffsetX + ii * PasX;
        DCase[NoCase].Top  := OffsetY + i * PasY;
      end
      else
      begin
        // vertical
        DCase[NoCase].Left := OffsetX + i * PasX;
        DCase[NoCase].Top  := OffsetY + ii * PasY;
      end;
      DCase[NoCase].Width := PasX;
      DCase[NoCase].Height := PasY;
      inc(NoCase);
    end;
  end;
 
  // fin de dessins
  Drawing := False;
 
end;
 
 
procedure TForm1.SpeedButton1Click(Sender: TObject);
var
  DCase : TCase;
begin
  // Créer une case
  Agenda := Tagenda.Create(SB);
  Agenda.Parent := Form1;
  Agenda.Top := 10;
  Agenda.Left := 10;
  Agenda.Width := 400;
  Agenda.Height := 400;
  Agenda.Color := ClSkyBlue;
  Agenda.Anchors := [akLeft,akTop,akRight,akBottom];
  Agenda.Fcreate := True;
  Agenda.FDateActive := now;
  Agenda.CaseType := ctmonth; //ctweek; //ctday, ctweek, ctmonth, ctyear
  Agenda.Fcreate := False;
  Agenda.CBStyleChange(Sender);
 
end;
 
end.

**Philippe Gormand** · 31/03/2006, 13h28

Bonjour.

Il ne faut pas utiliser la méthode Destroy, mais la méthode Free.
C'est très bien expliqué dans l'aide de DELPHI (que l'on ne consulte
jammais assez).

Salut.

**ffert2907** · 31/03/2006, 14h27

Merci pour le conseil que j'ai appliqué expressément...

Cependant j'ai toujours une erreur bizarre :
"Invalid Pointer Operation" !!!!!!!!!!!!!!

le nouveau code de la procédure est :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
Procedure TAgenda.ClearSetLegend(NewSize : integer);
var
  i : integer;
begin
  // vider et libérer les ressources de FLegend
  If High(FLegend) > -1 then
  begin
    i := 0;
    While i < High(FLegend) Do
    begin
        FLegend[i].Free;
      inc(i);
    end;
  end;
  // Taille du tableau := NewSize;
  SetLength(FLegend, NewSize);
end;

Et pour infos mon tableau Flegend contient alors : 48 objets.
Mais le FLegend[0].Free;.... se passe sans aucune erreur
alors que le FLegend[1].Free me donne une erreur : "Invalid Pointer Operation" !!!! c'est à y perdre son latin !!!

merci d'avance.

**TicTacToe** · 31/03/2006, 15h41

Pour boucler sur la libération, tu manques un elt non ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

While i <= High(FLegend)

J'utilise rarement High(). Mais c'est Count-1 il me semble.
si ton tableau possede 48 elts, High() = 47 ? dont il faut le prendre

mais pour l'instant je pense pas que j'explique ton pb

**ffert2907** · 31/03/2006, 15h45

Ouiii, c'est exact.

Mais de 0 à 47 ça fait bien 48 élément.... non ?

le problème c'est qu'il plante pas aux bornes 0 ou 47.

Mais l'erreur apparaît aprés... à l'élément N°1

Alors que l'élément 0 à bien été détruit correctement et sans erreur.

???

Mais merci quand même.

**Philippe Gormand** · 31/03/2006, 15h46

Il faut utiliser la procedure Length qui donne le nombre d'élément du
tableau. High done la plus grande valeure contenue dans le tableau.
Donc :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
Procedure TAgenda.ClearSetLegend(NewSize : integer); 
var 
  i : integer; 
begin 
  // vider et libérer les ressources de FLegend 
  If Length(FLegend) > 0 then 
  begin 
       For i :=0 To Length(FLegend) - 1 Do FLegend[i].Free; 
  end; 
  // Taille du tableau := NewSize; 
  SetLength(FLegend, NewSize); 
end;

**ffert2907** · 31/03/2006, 15h54

Perso, j'ai l'habitude de travailler avec High.
Cependant, J'ai fait les modifs nécessaire...

Mais là c'est PIRE !!!!!
Il ne me supprime même plus l'élément FLegend[0]

et j'ai l'erreur "Invalid Pointer Operation" tout de suite.
alors qu'avant il libérait au moins l'élément 0.

désolé..... ça ne marche pas.

**Andry** · 31/03/2006, 16h10

Et si on inverse le parcours, du dernier vers le premier :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
Procedure TAgenda.ClearSetLegend(NewSize : integer); 
var 
  i : integer; 
begin 
  // vider et libérer les ressources de FLegend 
  If Length(FLegend) > 0 then 
  begin 
       For i :=Length(FLegend) - 1 DownTo 0 
             Do FLegend[i].Free; 
  end; 
  // Taille du tableau := NewSize; 
  SetLength(FLegend, NewSize); 
end;

Essaie avec ça.
A+

**Philippe Gormand** · 31/03/2006, 16h22

Comprends pas !

J'utilise courament la méthode que j'ai donnée, dans plusieurs
programmes, sans problème.

Es tu sur que l'élément que tu veux libéré a bien été créé ? Tu peux le
tester avec Nil.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
Procedure TAgenda.ClearSetLegend(NewSize : integer); 
var 
  i : integer; 
begin 
  // vider et libérer les ressources de FLegend 
  If Length(FLegend) > 0 then 
  begin 
       For i :=0 To Length(FLegend) - 1 Do If FLegend[i] <> Nil Then FLegend[i].Free; 
  end; 
  // Taille du tableau := NewSize; 
  SetLength(FLegend, NewSize); 
end;

En tout cas, avec ça je n'ai jammais de problème.
Encore un mystère de l'informatique à éclaircir.

**TicTacToe** · 31/03/2006, 16h24

Je spécifiais bien que je corrigeais pas le pb, mais juste la longueur du parcours, ou il manquais la libération du dernier Elt. à cause du signe inferieur strict

Sinon, parcourir à l'envers, je comprends pas pourquoi cela ferait grand chose.
Etant donné que les élts alloués par le setlength sont toujours là, mais pointant sur rien (pas comme effectivement un 'Length' qui descendrait à chaque libération).

D'ailleurs, il serait preferable de faire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

FreeAndNil( FLegend[ x ] ) plutot que FLegend[ x ].Free

Si tu as ciblé la ligne à pb, et que le pb est constant, tu peux vérifier que ton pointeur FLegend[ x ] ( ou x vaut 0 puisque ca plante d'office), juste avant la libération, pointe toujours sur ton TLabel alloué (vérif par le caption etc...).

bon courage !

[EDIT]
Je viens de voir le post de Phil. Gormand
Le pb, c'est que meme si FLegend[ x ] est désalloué, il ne sera pas à Nil et provequera une exception si on le libère à nouveau (passant le test du Nil), par contre, pas avec FreeAndNil

**Philippe Gormand** · 31/03/2006, 18h53

bien vue TicTacToe.

Mais je pense que Free ou FreeAndNil n'est pas la raison du problème.
Il faudra que je travaille le sujet pour mieux le cerner.

Encore une fois, j'utilise ma méthode sans problème pour créer des
composants et pour les libérer. Cela marche sans problème, et pour
notamment un composant (pour le web) que j'ai fait et que je gère en
tableau.

Je suggère à "ffert2907" de choisir une méthode (la mienne ou celle d'un
autre) et de remonter l'origine du problème (peut être à la création).

L'expérience m'a montrée que souvent on but sur un problème, et plus
on s'y accroche et plus on bute. Alors, dans ce cas, on ferme tout, on va
à la pêche (quoi que en ce moment c'est l'époque des morilles) et le
landemain on se dit -"mais bon sang, c'est bien sur"-

Bon WE à tous.

**Laurent Dardenne** · 31/03/2006, 21h18

Pourquoi ne pas utiliser une liste ce qui faciliterai la maintenance de ton code ?

**ffert2907** · 01/04/2006, 07h05

MERCI à tous....

Justement pour les listes j'y songe... En fait je me donne jusqu'a lundi matin. si à midi j'ai pas trouvé, refonte complète du code avec les listes. J'ai trouvé un programme qui les généres automatiquement !!!!!!
ici : http://waskol.developpez.com

Et encore merci à tous... Pour les morilles j'ai pas eu le temps, ni d'aller à la pêche d'ailleur !!!

mais là je me prends un WE pour penser à autre chose... qui sait, la solution viendra peut être lundi matin....

merci. bye.

**Philippe Gormand** · 03/04/2006, 10h01

Ca m'a tracassé hier.

Effectivement, l'objet TList semble bien apporter la solution. De plus, c'est
aussi simple (apparemment) à gérer qu'un tableau. Donc un minimum
de modification.

A+

Problème de libération de ressource sur un tableau d'objet

Langage Delphi

Discussions similaires

Partager

Partager