Function not being called?

[sakonpure6]

Hi, for some reason my program wont call my Basic_Requirements function, all it does is quit after the first function call. Why?

#include <iostream>
#include <string>

using namespace std;

string uper[26]={"A","B","C","D","E","F","G","H","I","J","K","L","M","N","O","P","Q","R","S","T","U","V","W","X","Y","Z"};
string lower[26]={"a","b","c","d","e","f","g","h","i","j","k","l","m","n","o","p","q","r","s","t","u","v","w","x","y","z"};
string num[10]={"0","1","2","3","4","5","6","7","8","9"};

string y;
string x;
int u,l,n,s=0;

void Length(int total){
	int a;
	a=x.length();
    total=total * 4;

	while(a<8){
	cout<<"Enter a longer password: ";
	cin>>x;
	a=x.length();
	}
}

void check_3_req(){

	int c=x.length();
	int a=x.length();

	while(a!=0){
		y=x.substr(a,1);
		//loop to check if there are any upper,lower,numbers for the char y.
		for(int counter=0;counter!=c;counter++){

		if(y.compare(uper[counter])==0)
			u++;
		else
		if(y.compare(lower[counter])==0)
			l++;
		else
		if(y.compare(num[counter])==0)
			n++;
		else
			s++;

		}
		
		a--;

	}
	
}

void Basic_Requirements(int total){
	
	//int a=0;
	//Not using?!/
	//a=x.length();
	
    check_3_req();

	if (u<1 && l<1 && n<1 || u<1 && l<1 && s<1 || u<1 && n<1 && s<1 || l<1 && n<1 && s<1){
	cout<<"Please enter another password that meets requirements: ";
	cin>>x;
	}
	

	while(u<1 && l<1 && n<1 || u<1 && l<1 && s<1 || u<1 && n<1 && s<1 || l<1 && n<1 && s<1){
	check_3_req();
	cin>>x;
	}


}

int main(){
	int total=0;
	
	cout<<"Enter a password: ";
		cin>>x;

		Length(total);
		
		Basic_Requirements(total);
}

[giblit]

There are functions to check if they are lower case , uppercase , or decimal-digits try these. islower( CHAR ); isupper( CHAR ); isdigit( CHAR );
They should be in the cctype header. Also you should not use global variables.
And as far as your problem goes you are not actually asigning a value to total if you want to do that either do total = Length( total ) and return the value of total return( total ); or in your Length function declaration make total a reference void Length( int &total )

[keskiverto]

What does "quit" mean?

Not this? http://www.cplusplus.com/forum/beginner/1988/

[Daleth]

I am guessing it is your conditions. Group the related conditions together in parentheses.
For example:

if (
   (u<1 && l<1 && n<1) || 
   (u<1 && l<1 && s<1) || 
   (u<1 && n<1 && s<1) || 
   (l<1 && n<1 && s<1)
){

Or better yet, make a function that returns a bool, so you don't have to keep retyping these lists.