Password Assignment

[sharingan_gv]

I'm supposed to make a program that lets you type in a password that has to be atleast 8 characters, with at least 2 digits.

Not sure if this is right, but I haven't been able to test it because i keep getting a syntax error before both of the "else" part of the program.

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <string.h>
#include <ctype.h>


int main()
{
    char x[30];
    int i=0; 
    int numdig=0, letters=0, ch;
    
    printf("Enter Password:");
    scanf("%s", x);
    x[strlen(x)]= '\n';
    
    while(gets(x)!= NULL)
    {
                
     
      if(isdigit(x[i]))
      numdig++;
      i++;
      
      else if((islower(x[i])) || (isupper(x[i])))     
       letters++;
       i++;             
           
      else
           break;         
                    
                    }
                    
                    
        if ((strlen(x)>= 8) &&(numdig>=2) )
        printf("Valid Password\n");    
        
        else
        printf("Invalid Password");
        
    while ((ch = getchar()) != '\n' && ch != EOF);
    getchar();
    return 0;
    
    }

[AndrewHunter]

Well lets clean that code up and look at it:

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <string.h>
#include <ctype.h>


int main()
{
    char x[30];
    int i=0; 
    int numdig=0, letters=0, ch;
    
    printf("Enter Password:");
    scanf("%s", x);
    x[strlen(x)]= '\n'; //for a c string this should be '\0'

    while(gets(x)!= NULL) //what are you doing here? Maybe a simple for loop would suffice (for int i=0;i<strlen(x);i++)
    {
		if(isdigit(x[i])){ //<------you forgot all your brackets
			numdig++;
			i++;
		}
		else if((islower(x[i])) || (isupper(x[i]))){ //isalpha() would work here    
			letters++;
			i++;             
		}  
		else
           break;         
     }
     if ((strlen(x)>= 8) &&(numdig>=2) )
	printf("Valid Password\n");    
     else
	printf("Invalid Password");
        
    while ((ch = getchar()) != '\n' && ch != EOF);
    getchar();
    return 0;
    
}

[sharingan_gv]

Thanks again ! I thought about the while loop and I just forgot about the for loop.

And i didn't have to add the i++ in all my other if -if else statements.

Off to do my last one!

[ObjectWithBrain]

How would you make it read the password without displaying what you typed on the screen?

[Matticus]

There's an example of that here, though it uses non-standard functions.

[AndrewHunter]

The only way to do that is with non-standard functions.