okay, what would the best way to check to make sure that all characters inputted by a user (which is stored as a char array) only consist of a-z, A-Z, and 0-9??
I was thinking a for loop for obvious reasons, but I can't seem to get the logic down to make it work.
This is the last part of my program I need to get to work and it's driving me nuts! heh
Post your loop in [code][/code] tags and somebody may be able to figure out where your logic is off.
Another way might be to use an algorithm combined with isalpha and isdigit, but since you are using C style strings I doubt you'd be allowed to use C++ algorithms.
Post some code, or at least explain your logic.
A for loop sounds good, where is the detail.
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.
I forgot to add . and @ to the list of chars since it IS an e-mail addy.
umm, I haven't formulated a for loop yet, I'm just trying to write down some pseudo-code, and I can't get the logic to work.
the only thing I've come up with is if-else statements within the for loop, somethin like
for(int a=0; a<length; a++)
{
if(email[a] < 'a' || email[a] > 'z')
{
....
But, as you can probably tell, doing it for lower case, upper case, then numbers isn't going to work...
At the very basic, you would need to nest the loops. One to check for the char ( letter ) and
one to check the numbers ( int )
But post some code, give it a try yourself and if you are still stumped, peole will be able to help you
No need to nest the loops, although you might need to nest the ifs (not necessarily though).
Your pseudo code looks fine. If you can use isalpha that would make your job a little easier. Otherwise, if you know functions, making separate functions that check for lowercase, uppercase, digits and the two extra characters might be a good way to go to reduce repeated code and make things clearer. In fact, just doing that in your pseudo-code could help.
okay, this is what I come up with, but I still can't seem to get it to work right...
I came up with the bounds by looking at an ascii value key...
Code:for(int g = 0; g < length; g++) { if(email[g] > '.' && email[g] < 'z') { if(email[g] == '/') { return 0; } else if(email[g] > '9' && email[g] < '@') { return 0; } else if(email[g] > 'Z' && email[g] < 'a') { return 0; } } else { return 0; } }
>> I still can't seem to get it to work right.
What's going wrong with it? The logic there looks ok. Where's the rest of the code?
A way to avoid all those if statements is to create another char array (preferably a std::vector or std::string) with all the characters that are allowed. It's then a simple matter of traversing this array for each character in your string.
If it is found, you break from the loop and continue to the next character in your string. If it is not found, then I think your search is over. The email is not valid. You set some flag variable to false indicating the email is not valid and break of the loops.
If you are allowed to use the STL, then it all becomes much easier. With std::string and member functions like find_first_not_of()
Originally Posted by brewbuck:
Reimplementing a large system in another language to get a 25% performance boost is nonsense. It would be cheaper to just get a computer which is 25% faster.
Originally Posted by Daved
It makes everything come up invalid...
>> It makes everything come up invalid...
The logic there looks ok. Where's the rest of the code? What input do you give it?
nevermind, I got it working.
Thanks, guys
