Would there be a way to do this simplier?
Instead of this
Something likeCode:if (pre2 == "0\n"||pre2 == "1\n"||pre2 == "2\n"|| pre2 == "3\n"||pre2 == "4\n"||pre2 == "5\n"|| pre2 == "6\n"||pre2 == "7\n"||pre2 == "8\n"||pre2 == "9\n")
(pre2 == int'\n')
I know its not good but you can get the idea what i mean.
Wouldn't it be possible to have a function which took an int and a string and returned the int and the string as one string and then inset the function to replace the "0/n" part.
Something like:
Edit: Sorry i just noticed you said || and not &&.
besides, Elysia got a way easier solution.
Last edited by Zigs; 12-28-2007 at 02:17 PM. Reason: Yeah, I'm a noob >_>"
One way:
Or:Code:bool bMatch = false; for (int i = 0; i <= 9; i++) { if ( pre2[0] != ('0' + i) ) { bMatch = true; break; } } if (pre[1] == '\n' && bMatch); // Your code
Code:if (isdigit(pre2[0]) && pre2[0] == '\n')
Last edited by Elysia; 12-28-2007 at 05:02 PM. Reason: Typo
Originally Posted by Adak
Nice! Thank you Elysia!
Why not just
Code:if (pre2.length() == 2 && isdigit(pre2[0]) && pre2[1] == '\n')
Incorrect code?
Last edited by Elysia; 12-28-2007 at 05:00 PM.
Because out-of-bounds memory accesses are A-ok?
Not definitely not, but no-one says it IS out of bounds. I'll just have to assume you're doing some basic checks before.
Depending on the context of the OP's code, it's very possible that one or two of the conditions in my test might be known true and could be skipped, but at the moment, only the OP knows that.
Yes Robatino on second thought you're right , it depends on the context, that just i forgot to specify.
Actually i cant use either one because im gonna read a whole file once and pre2[0] would be the first character in the file however what i want is to find the combination of a number with \n in the file.
Thanks though for the help , it helped me to understand more the problem.
