Want to win a PS4? Go Premium and enter to win our High-Tech Treats giveaway. Enter to Win

x
?
Solved

Executing Events in components

Posted on 2001-06-21
6
Medium Priority
?
207 Views
Last Modified: 2010-04-06
Hi, I've just recently launched myself into component-writing.  I have written a sample component with the objective of adding a new event, namely OnNewTransaction
which will be triggered whenever the OnTimer event of a timer is triggered.  Below are some portions of that class declaration :

type
  TRtStkData = class(TComponent)
  private
    FTimer            : TTimer;
    FOnNewTransaction : TNotifyEvent;
    FClose            : TStringList;
    procedure FTimerOnTimer(Sender:TObject);
    procedure SetClose(Value: TStringList);
    procedure AddItem;
  protected
  public
    constructor Create(AOwner : TComponent); override;
    destructor Destroy; override;
    procedure Show(index : integer);
    procedure Addnew;
  published
    property OnNewTransaction : TNotifyEvent read   FOnNewTransaction write FOnNewTransaction;
    property Close : TStringList read FClose write SetClose;
  end;


constructor TRtStkData.Create(AOwner : TComponent);
begin
  inherited Create(AOwner);
  FClose:=TStringList.Create;
  FTimer:=TTimer.Create(Self);
  FTimer.Interval:=2000;
  FTimer.Enabled:=false;
  FTimer.OnTimer:=FTimerOnTimer;
end;

procedure TRtStkData.SetClose(Value: TStringList);
begin
  FClose:=Value;
end;

procedure TRtStkData.add;
begin
  FClose.Add(IntToStr(random(100)));
end;

procedure TRtStkData.FTimerOnTimer(Sender:TObject);
begin
 Add;
 if Assigned(FOnNewTransaction) then
    FOnNewTransaction(Self);
end;


The class compiled successfully and the new component was installed successfully.  I've made a sample calling program into which I've added the new component.
I enabled the timer at one point of the calling program.The  OnNewTransaction event appeared in the Object inspector but that event is never triggered.  
When I traced the program I found out that  the problem is in the method FTimerOnTimer :

the statement Assigned(FOnNewTransaction)is always false.  

Can someone help me out?  
Below are some portions of the sample calling program:

procedure TForm1.FormCreate(Sender: TObject);
begin
  RtStkData1:=TRtStkData.Create(Self);
end;

procedure TForm1.RtStkData1NewTransaction(Sender: TObject);
begin
  Memo1.Lines.Add(RtStkData1.Close[RtStkData1.Close.Count-1]);
end;
0
Comment
Question by:TheBreeze
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
6 Comments
 
LVL 8

Expert Comment

by:TOndrej
ID: 6217128
I can't find in your code FTimer.Enabled := True;
0
 
LVL 1

Accepted Solution

by:
vorisek earned 300 total points
ID: 6217149
If you add the component from component palette
you need not (must not) create it in OnCreate.
So try to omit the code in TForm1.FormCreate.
Or you have to assigne OnNewTransaction manually in
TForm1.FormCreate :OnNewTransaction:=RtStkData1NewTransaction;
0
 
LVL 5

Expert Comment

by:Gwena
ID: 6217157
listening :-)
0
Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 
LVL 3

Expert Comment

by:Stefaan
ID: 6217199
Hi,

One thing, did you add code to the Event Handler ? I mean did you double click on the OnNewTransaction event in the object inspector and write some code in that event handler ?

The following code :

if Assigned(FOnNewTransaction) then
   FOnNewTransaction(Self);

the assigned will only be true if you have added some code in the event handler of the OnNewTransaction event.

One last note, your component has a memory leak.  Why, well it is a good practice in Component Buidling to destroy everything you created yourself.  In the constructor you instantiate a TStringList and a TTimer.  Since the TTimer has an owner it will be cleaned up automatically when the Owner component gets destroyed.  The TStringList on the other hand doesn't have an owner and will not be destroyed automatically, thus it will stay in memory causing a memory leak.

You should override the destructor of your component and in your case destroy the TTimer and TStringList in the destructory of your component.  ( possibly you did it, but you didnt include the destructor in your code sample ).

Hoping that this will be of any help to you.

Best regards,


Stefaan
0
 

Author Comment

by:TheBreeze
ID: 6217207
Thank you vorisek!

I deleted the code inside the OnCreate
procedure of the calling program and it worked!

Thanks again!
0
 

Author Comment

by:TheBreeze
ID: 6223517
TOndrej,
I did enable the Timer but have not included it in the sample code.  

Stefaan,
I did all that you said in your comment, but again, have not included them in the sample code.  Sorry for that.  But I appreciate your help.  
0

Featured Post

New feature and membership benefit!

New feature! Upgrade and increase expert visibility of your issues with Priority Questions.

Question has a verified solution.

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

Have you ever had your Delphi form/application just hanging while waiting for data to load? This is the article to read if you want to learn some things about adding threads for data loading in the background. First, I'll setup a general applica…
In my programming career I have only very rarely run into situations where operator overloading would be of any use in my work.  Normally those situations involved math with either overly large numbers (hundreds of thousands of digits or accuracy re…
This course is ideal for IT System Administrators working with VMware vSphere and its associated products in their company infrastructure. This course teaches you how to install and maintain this virtualization technology to store data, prevent vuln…
This lesson discusses how to use a Mainform + Subforms in Microsoft Access to find and enter data for payments on orders. The sample data comes from a custom shop that builds and sells movable storage structures that are delivered to your property. …
Suggested Courses

610 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