?
Solved

Help me in chat application !

Posted on 2006-07-19
10
Medium Priority
?
204 Views
Last Modified: 2010-04-05
I used indy TCP to make chat application both in client and server.

after a time (about 3 hours upper), CPU of server.exe near 100%, make my system LAG

I know function sleep() can solve this problem but I dont know where to put function sleep in.
 
in client code or in server code,

or is there a better way for server.exe than use sleep ?

please help me in detail.

thank all experts.

0
Comment
Question by:lhquang
[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
  • 4
  • 2
  • 2
  • +2
10 Comments
 
LVL 28

Expert Comment

by:2266180
ID: 17137269
you will put a sleep(1) tipically or other small value in  loopes that take "forever" without doing anything. like

while not terminated do
  slee(100);

it is also advisable that you put application.processmessages; calls in such repetitive structures.
also, when doing client-server applications, you must use threads at least on teh server side to habdle client connections.
there are also other general issues that can cause a cpu eating up 100%.

so better post the code here so we can see exactly where those issues are. (maybe sleep is not the answer ;) )
0
 
LVL 17

Accepted Solution

by:
Wim ten Brink earned 1500 total points
ID: 17137656
I don't think Sleep() will solve this problem. All sleep does is halt the current thread and give another thread some CPU time. In general you would use Sleep(0) and not Sleep(1) if you just want another thread to start processing a bit more. (A Sleep(1) would add a delay of one millisecond, which might slow down your app considerably if you execute it often enough.)

The question is of course why your system needs so much CPU time. Maybe you're running out of memory because of a memory leak. Then it would be swapping to the swapfile. Or maybe something else is going wrong. Are you testing on multiple computers, where one is running the server and the other the client(s)? If so, which computer gets the 100% CPU?

It could also be that your application is calling Application.ProcessMessages or Sleep() too often. But I don't think Sleep() will solve it.

0
 
LVL 17

Expert Comment

by:TheRealLoki
ID: 17140667
the server runs its' clent connections inside a thread, so you do not need sleep
I would look for the problem elsewhere
can you post some of your code, and we might be able to spot the real problem
0
Industry Leaders: 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 5

Expert Comment

by:Scay7
ID: 17140897
Have to agree here with TheRealLoki, we do not have enough information to help you. Also as Loki said its not a thread problem more likely a source twitch...
So paste you code where you think it use the most CPU time...

Peace Scay7
0
 

Author Comment

by:lhquang
ID: 17142850
the source code I developed from the chat demo in INDY 9. thank experts
0
 
LVL 17

Expert Comment

by:TheRealLoki
ID: 17143273
The Indy 9 chat demo is very buggy
firstly, the client can lock up on the
Msg := IdTCPClient1.ReadLn('', 5);
line in the Timer1Timer() event

This can be resolved by putting a TIdAntiFreeze on the form, but you still get a memory leak
(just watch the task maanger while your client is connected)
To resolve this you need to disable the TTimer while doing the ReadLn, so your Timer1Timer routine ends up looking something like this :-

procedure TForm1.Timer1Timer(Sender: TObject);
var
  Com,
  Msg : String;
begin
  if not IdTcpClient1.Connected then
    exit;
  Timer1.Enabled := False;
  try
    Msg := IdTCPClient1.ReadLn('', 5);

    if Msg <> '' then
      if Msg[1] <> '@' then
        begin
        { Not a system command }
          memLines.Lines.Add(Msg);
        end
      else
        begin
        { System command }
          Com := UpperCase(Trim(Copy(Msg, 2, Pos(':', Msg) -2)));
          Msg := UpperCase(Trim(Copy(Msg, Pos(':', Msg) +1, Length(Msg))));
          if Com = 'CLIENTS' then
            lbClients.Items.CommaText := Msg;
        end;
    finally
      Timer1.Enabled := True;
    end;
  end;

anyway, this may be the cause of your problems...
0
 

Author Comment

by:lhquang
ID: 17143537
anyone can give me the better demo than INDY DEMO ?
thanks
0
 

Author Comment

by:lhquang
ID: 17151179
oh, now I found out the my problem and fixed it.

thank everybody
0
 
LVL 17

Expert Comment

by:Wim ten Brink
ID: 17152576
Am just curious... What was the error exactly? :-)
Which reminds me... memlines.lines.add just keeps adding strings to a memo field. Does it also clean up this memo a bit? If not, lots of memory usage there...
0
 

Author Comment

by:lhquang
ID: 17154681
Workshop_Alex , u r right, memlines.lines.add just keeps adding strings to a memo field and I solved it
0

Featured Post

[Webinar] How Hackers Steal Your Credentials

Do You Know How Hackers Steal Your Credentials? Join us and Skyport Systems to learn how hackers steal your credentials and why Active Directory must be secure to stop them.

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…
In this tutorial I will show you how to use the Windows Speech API in Delphi. I will only cover basic functions such as text to speech and controlling the speed of the speech. SAPI Installation First you need to install the SAPI type library, th…
Michael from AdRem Software outlines event notifications and Automatic Corrective Actions in network monitoring. Automatic Corrective Actions are scripts, which can automatically run upon discovery of a certain undesirable condition in your network.…
In this video, Percona Director of Solution Engineering Jon Tobin discusses the function and features of Percona Server for MongoDB. How Percona can help Percona can help you determine if Percona Server for MongoDB is the right solution for …
Suggested Courses

764 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