Hi all!
I'm trying to get a PONG anwser but it doesn't work
Here is my code from my simple ircbot
I hope somebody can help me.Code:printf("\n%s", timebuffer); if(strstr(timebuffer,"PING")!=NULL) { printf("\nPing recv"); timebuffer[1] = 'O'; // From a PING answer to a PONG but it doesn't work write(fd,timebuffer, strlen(timebuffer)); }
Thanks for all
x77
So what does "doesn't work" mean?
Does it ever print "Ping recv" for example?
Or does timebuffer sometimes contain garbage perhaps?
If you dance barefoot on the broken glass of undefined behaviour, you've got to expect the occasional cut.
If at first you don't succeed, try writing your phone number on the exam paper.
Hi!
It means
This is an example:
The server send a PING like
PING :B335AC1B
The client must answer with the same like
PONG:B335AC1B
So I only must change the I in PING to an O.
And if PING isn't right at the start, which strstr() can give you, then what?
Debug the code and trace out what happens.
If you dance barefoot on the broken glass of undefined behaviour, you've got to expect the occasional cut.
If at first you don't succeed, try writing your phone number on the exam paper.
It only prints out at stdout
:irc.test.net NOTICE AUTH :*** Looking up your hostname...
:irc.test.net NOTICE AUTH :*** Found your hostname
PING :X7019375
-------------------------------------------------------------------------
Does it ever print "Ping recv" for example?
No I don't see this Ping recv
Last edited by x77; 07-22-2007 at 12:48 PM.
perhaps try using strstrn instead of strstr
off the top of my head, it looks like you are comparing "PING" to "PING :X7019375"
try
Code:if(strstrn(timebuffer,"PING", 4)!=NULL)Then the condition in the if statement is returning false(non-zero?)Originally Posted by x77
Environment: OS X, GCC / G++
Codes: Java, C#, C/C++
AOL IM: neandrake, Email: neandrake (at) gmail (dot) com
That function returns a pointer to the first occurance of s2 in s1. AFAIK, there is no function "strstrn".Code:char *strstr(const char *s1, const char *s2);
The OP needs to do some debugging, or post more code. I'd first switch around the printf() statements. Place the \n's at the end, (stdout is buffered) so that you're sure it gets printed. Better, send it to stderr. Better, just fire up a debugger and set a breakpoint inside the if() statement. If it never breaks, then what you think is in the buffer doesn't correspond to what's actually there.
False would mean the condition returned 0. Non-zero in an if statement means true.Then the condition in the if statement is returning false(non-zero?)
long time; /* know C? */
Unprecedented performance: Nothing ever ran this slow before.
Any sufficiently advanced bug is indistinguishable from a feature.
Real Programmers confuse Halloween and Christmas, because dec 25 == oct 31.
The best way to accelerate an IBM is at 9.8 m/s/s.
recursion (re - cur' - zhun) n. 1. (see recursion)
Hi!
Thank you for your help.
