why am i getting an infinite loop in this piece of code:
Is it because i should have flushed something and i didn't??Code:while((( *(str+index)) != '(') || ((*(str+index)) ! = '\0'))//while char is not '(' and not end of string, increase index { index++; }
If that's the case how do i do that??
coz i've been trying delete [] str;
and this doesn't work
It can't be both '(' and '\0' at the same time. Thus, it should be "while it isn't this AND it isn't that".
Also, delete is a C++ keyword.
Quzah.
Hope is the first step on the road to disappointment.
try
Code:if( index < strlen(str) && index >= 0) /*be sure...*/ while( str[index] != '\0' && str[index++] != '(' );
Last edited by Salem; 05-24-2006 at 12:56 AM. Reason: Fix code tags - its / not \
> while( str[index] != '\0' && str[index++] != '(' );
Would be better written as
while( str[index] != '\0' && str[index] != '(' ) index++;
Remember, && and || are short-circuited, so if the right hand side is skipped, then the side effect of the ++ DOES NOT HAPPEN!
Needless to say, this is a bizarre bug to find.
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.
you even put in your comment
//while char is not '(' and not end of string, increase index
make it && and it should work.
Also with this line:
while( str[index] != '\0' && str[index++] != '(' );
remember that i++ is postfix and depending on the order that you want things to happen and if you want index to increase, that statement is undefined.
It is undefined because the while could evaluate from the right, then index would be increased for str[index] but not for str[index++]. But since it is postfix, if you want it to increase and it evaluates from the left it won't be increased for the evaluation. Then also what the guy before said, if the left hand side is false the right hand side won't even be evaluated (that is of course if it evaluates from left to right). So I wouldn't be doing that since you don't know what order things are all going to occur in.
> It is undefined because the while could evaluate from the right,
No - the order in which && and || are evaluated is very specifically defined.
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.
This while loop, while it works, seems overly complex & is causing headaches. Why not just search for a '(' using strchr() ? For example, any reason why this wouldn't work:
Code:#include <stdio.h> int main() { char *test = "Test! =) Good day."; int index = 0; char *cfind; cfind = strchr(test, ')'); if(cfind) index = cfind - test; else index = strlen(test); printf("Index is: %d\n", index); return 0; }
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)
Well it's beacause i'm converting my program from C++ to C and i didn't use any available method to find a character. Thanks to you all, i've found my mistake.
A really silly mistake i can say. :P
Sorry for all this trouble.
