Solved

Code Problem

Posted on 2001-09-17
7
146 Views
Last Modified: 2010-04-06
howdy...

could someone please inspect the following code, and tell me why the program just hangs on the line "Categories[0].Items[0].Create;"?

maybe im doing something wrong, but im not extirely sure if what im doing is correct way either...

Basically i need to have an array [0..5] each element containing an array of TStrings.

if there is a better way of doing that, i'd apprecaite being filled in! :P

Thanks

X

-----------------------

unit mainform;

interface

uses
  Windows, Messages, SysUtils, Classes, Graphics, Controls, Forms, Dialogs,
  StdCtrls;

type
  TCategoryItems = record
    Items: array of TStringList;
  end;


type
  TForm1 = class(TForm)
    Button1: TButton;
    procedure Button1Click(Sender: TObject);
  private
    Categories: array[0..5] of TCategoryItems;
  public
    { Public declarations }
  end;

var
  Form1: TForm1;

implementation

uses StringRoutines;

{$R *.DFM}

procedure TForm1.Button1Click(Sender: TObject);
var
  Count1, Count2, Count3 : integer;
begin
  SetLength(Categories[0].Items, 6);
  SetLength(Categories[1].Items, 6);
  SetLength(Categories[2].Items, 6);
  SetLength(Categories[3].Items, 6);
  SetLength(Categories[4].Items, 6);
  SetLength(Categories[5].Items, 6);

  Categories[0].Items[0].Create;
  Categories[0].Items[0].Add('HEH!');

  Categories[2].Items[5].Create;
  Categories[2].Items[5].Add('wei');
  Categories[2].Items[5].Add('diddy');

  Showmessage('Category 0, Item 0 is ' + Categories[0].Items[0].Strings[0]);
  Showmessage('Category 2, Item 5 is ' + Categories[0].Items[2].Strings[0]);
  Showmessage('Category 2, Item 5 is ' + Categories[0].Items[2].Strings[1]);

{  try
    for Count1 := 0 to High(Categories) do
      for Count2 := 0 to High(Categories[Count1].Items) do
        if Assigned(Categories[Count1].Items[Count2]) then
          for Count3 := 0 to Categories[Count1].Items[Count2].Count do
            if Categories[Count1].Items[Count2].Strings[Count3] <> '' then
              Showmessage('Category ' + IntToStr(Count1) + ', Item ' + IntToStr(Count2) + ' is ' + Categories[Count1].Items[Count2].Strings[Count3]);
  except
    showmessage('we died');
  end;
}
end;

end.
0
Comment
Question by:X_Kalibur
  • 2
  • 2
  • 2
  • +1
7 Comments
 
LVL 1

Author Comment

by:X_Kalibur
ID: 6486896
note that the second array of TStringList needs to be a dynamic resizable array...

the values of 6 for the size of the arrays were just something i set for testing purposes.
0
 
LVL 17

Accepted Solution

by:
geobul earned 200 total points
ID: 6486907
Hi,
I think:

Categories[0].Items[0] := TStringList.Create;

Regards, Geo
0
 
LVL 22

Expert Comment

by:Mohammed Nasman
ID: 6486928
Hello

  You need to call create method for each Categories[?].items[?] before you can use it

  and items is TStringList, so you need to create it as Geo  suggested to you

 also you try to access items not added in some Categories

you added items to  Categories[2].Items[5]
 Categories[2].Items[5].Add('wei');
 Categories[2].Items[5].Add('diddy');

but u try to access to
 Showmessage('Category 2, Item 5 is ' + Categories[0].Items[2].Strings[0]);

so change it to
 Showmessage('Category 2, Item 5 is ' + Categories[2].Items[5].Strings[0]);
 Showmessage('Category 2, Item 5 is ' + Categories[2].Items[5].Strings[1]);


0
Free Tool: SSL Checker

Scans your site and returns information about your SSL implementation and certificate. Helpful for debugging and validating your SSL configuration.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

 
LVL 22

Expert Comment

by:Mohammed Nasman
ID: 6487005
also  change this line
for Count3 := 0 to Categories[Count1].Items[Count2].Count
to
for Count3 := 0 to Categories[Count1].Items[Count2].Count -1 // count -1

ur code will look like this after the few modifications

procedure TForm1.Button1Click(Sender: TObject);
var
 Count1, Count2, Count3 : integer;
begin
 SetLength(Categories[0].Items, 6);
 SetLength(Categories[1].Items, 6);
 SetLength(Categories[2].Items, 6);
 SetLength(Categories[3].Items, 6);
 SetLength(Categories[4].Items, 6);
 SetLength(Categories[5].Items, 6);

 Categories[0].Items[0] :=TStringList.Create;

 Categories[0].Items[0].Add('HEH!');

 Categories[2].Items[5] :=TStringList.Create;;
 Categories[2].Items[5].Add('wei');
 Categories[2].Items[5].Add('diddy');

 Showmessage('Category 0, Item 0 is ' + Categories[0].Items[0].Strings[0]);
 Showmessage('Category 2, Item 5 is ' + Categories[2].Items[5].Strings[0]);
 Showmessage('Category 2, Item 5 is ' + Categories[2].Items[5].Strings[1]);

 try
   for Count1 := 0 to High(Categories) do
     for Count2 := 0 to High(Categories[Count1].Items) do
       if Assigned(Categories[Count1].Items[Count2]) then
         for Count3 := 0 to Categories[Count1].Items[Count2].Count -1 do
           if Categories[Count1].Items[Count2].Strings[Count3] <> '' then
             Showmessage('Category ' + IntToStr(Count1) + ', Item ' + IntToStr(Count2) + ' is ' + Categories[Count1].Items[Count2].Strings[Count3]);
 except
   showmessage('we died');
 end;

end;
0
 
LVL 14

Expert Comment

by:AvonWyss
ID: 6487188
You may also want to enable the range checks in the compiler options while developing such stuff, it will detect bounds errors in arrays.

Very important, create the TStringList's as shown by geobul:
    Categories[0].Items[0] := TStringList.Create;
instead of
    Categories[0].Items[0].Create;

Why do you need as many TStringList's for anyways?
0
 
LVL 1

Author Comment

by:X_Kalibur
ID: 6489270
Thanks!

You wouldnt belive it, but this was the second time ive done something stupid like that ever, and posted a question on EE, cause i overlooked it each time!!


Basically i need to maintain a list of categories which will be read in from a file...and each category will have a variable list of sub-categories - hence the array of dynamic array..


Thanks everyone for all your help!
0
 
LVL 14

Expert Comment

by:AvonWyss
ID: 6489399
I would have solved this by makinc a custom TStringList descendant which has another TStringList assigned to each string... this would allow to build a "tree" of TStringLists without the hassle of mixing arrays and lists.
0

Featured Post

Networking for the Cloud Era

Join Microsoft and Riverbed for a discussion and demonstration of enhancements to SteelConnect:
-One-click orchestration and cloud connectivity in Azure environments
-Tight integration of SD-WAN and WAN optimization capabilities
-Scalability and resiliency equal to a data center

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

The uses clause is one of those things that just tends to grow and grow. Most of the time this is in the main form, as it's from this form that all others are called. If you have a big application (including many forms), the uses clause in the in…
Introduction The parallel port is a very commonly known port, it was widely used to connect a printer to the PC, if you look at the back of your computer, for those who don't have newer computers, there will be a port with 25 pins and a small print…
Microsoft Active Directory, the widely used IT infrastructure, is known for its high risk of credential theft. The best way to test your Active Directory’s vulnerabilities to pass-the-ticket, pass-the-hash, privilege escalation, and malware attacks …

856 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question