Thread: A simple program again!

Code:

int main() {


    char password[20];
    int i = 0, tru1, tru2, tru3;


    printf("Enter your password (Must have 1 uppercase, lowercase and and a digit!): \n");
    scanf(" %s", password);


    do {
            tru1 = 0; tru2 = 0; tru3 = 0;
        if (isupper(password[i])) {
            tru1 = 1;
        } else if (islower(password[i])) {
            tru2 = 1;
        } else if (isdigit(password[i])) {
            tru3 = 1;
        } else if (tru1 == 1 && tru2 == 1 && tru3 == 1) {
            printf("Congratulations you did it right your password is: %s", password);
            break;
        } else {
            printf("Something went wrong!");
            break;
        }
        password[i++];
    } while (i <= 20);
    return 0;
}

It keeps on printing something went wrong? What's wrong with my computer logic :O XD

Originally Posted by Ktulu

You're setting the values of tru1/2/3 to zero each time the loop cycles so your good boy message will never be shown. Also, entering a passwords which exceeds 20 characters can lead to really bad things in your program.

Oh yeah, now it works Lol. Why bad? I don't get it.

Originally Posted by Salem

Get a large jug of water and pour it into a smaller jug of water.
How do you decide when to stop pouring to avoid your feet getting wet?

> char password[20];
Now consider how scanf would know to stop at 20 characters.

Ohh so I see, I should make it bigger then XDDD Ahahaa, oh yeah. I should also think of something like that when using scanf. Thanks! :-D

Hi all...

Am I missing somethiing or have I been lead up the garden path.... thought the functions printf and scanf Both require the #include <stdio.h> header....? which is not shown in the code, so why would it work at all....?

John