Java Help

Question

welbyobeng 3 Junior Poster in Training

16 Years Ago

I am trying to create a java password code where the password is at least 6 characters Long, at least 1 UPPERCASE, & at least 1 lowercase & at least 1 Digit.Here is an example: Passw3

Can someone help me because it is not working. Here is the code

private static boolean isValid(String custNumber)
{
    boolean goodSoFar = true;
    int index = 0;

    // Test if string the correct length
    if (custNumber.length() !=6)
        goodSoFar = false;

    // Test if string has at least one UPPER case
    while (goodSoFar && index < 6)
    {
        if (!Character.isUpperCase(custNumber.charAt(index)))
        goodSoFar = false;
        index++;
    }

    //Test if string has at least one lower case

    while (goodSoFar && index < 6)
    {
        if (!Character.isLowerCase(custNumber.charAt(index)))
        goodSoFar = false;
        index++;
    }

    //Test if string has at least one Digit

    while (goodSoFar && index < 6)
    {
        if (!Character.isDigit(custNumber.charAt(index)))
        goodSoFar = false;
        index++;
    }

    return goodSoFar;

}
}

java

Edited 11 Years Ago by mike_2000_17 because: Fixed formatting

4 Contributors
12 Replies
102 Views
3 Days Discussion Span
Latest Post 16 Years Ago Latest Post by javaAddict

All 12 Replies

jwenting 1,889 duckman

16 Years Ago

if (custNumber.length() !=6)

Doesn't take strings longer than 6 characters (which are allowed) into account.

Everywhere else, take the actual length instead of 6 as the limit, and keep in mind that the index starts at 0, not 1.

Then think about what you're doing. You reject the string as soon as any character doesn't match what you're checking.
That's of course incorrect.
You need to count the number of characters matching your requirement and reject only if that number is too low (or better accept as soon as it's high enough).

jwenting 1,889 duckman

16 Years Ago

I already told you what to do.
Count the number of uppercase characters you encounter and as soon as it reaches the minimum number required set the flag to true (start with the flag at false) and break out of the loop.

jwenting 1,889 duckman

16 Years Ago

You do have to set the flag to false before each step.
If it fails one test, skip the rest as the entire string is invalid.
If it passes one test, reset it to false and try the next test.
That way it will be true only if it passes all the tests.

Reply to this topic

Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, networking, learning, and sharing knowledge.

welbyobeng 3 Junior Poster in Training · Answer 1 · 2008-04-07T00:38:01+00:00

if (custNumber.length() !=6)
Doesn't take strings longer than 6 characters (which are allowed) into account.
Everywhere else, take the actual length instead of 6 as the limit, and keep in mind that the index starts at 0, not 1.
Then think about what you're doing. You reject the string as soon as any character doesn't match what you're checking.
That's of course incorrect.
You need to count the number of characters matching your requirement and reject only if that number is too low (or better accept as soon as it's high enough).

Ok I change the length if the character is less than 6 it should be false. Now Can you please help me with just the Uppercase code and I think I can do the rest

private static boolean isValid(String custNumber)
{
boolean goodSoFar = true;
int index = 0;
// Test if string the correct length
if (custNumber.length() < 6)
goodSoFar = false;
// Test if string has at least one UPPER case
while (goodSoFar && index < 6)
{
if (!Character.isUpperCase(custNumber.charAt(index)))
goodSoFar = false;
index++;
}
//Test if string has at least one lower case
while (goodSoFar && index < 6)
{
if (!Character.isLowerCase(custNumber.charAt(index)))
goodSoFar = false;
index++;
}
//Test if string has at least one Digit
while (goodSoFar && index < 6)
{
if (!Character.isDigit(custNumber.charAt(index)))
goodSoFar = false;
index++;
}
return goodSoFar;
}
}

welbyobeng 3 Junior Poster in Training · Answer 2 · 2008-04-07T23:53:46+00:00

I already told you what to do.
Count the number of uppercase characters you encounter and as soon as it reaches the minimum number required set the flag to true (start with the flag at false) and break out of the loop.

Hi thank you for helping me but you have to understand me am a newbie in java. This is what I did and it is still not working

private static boolean isValid(String custNumber)
{
boolean goodSoFar = false;
int index = 0;
// Test if string the correct length
if (custNumber.length() < 6)
goodSoFar = false;
// Test if string has at least one UPPER case
while (goodSoFar && index >= 1)
{
if (!Character.isUpperCase(custNumber.charAt(index)))
goodSoFar = true;
index++;
}
//Test if string has at least one lower case
while (goodSoFar && index >= 1)
{
if (!Character.isLowerCase(custNumber.charAt(index)))
goodSoFar = true;
index++;
}
//Test if string has at least one Digit
while (goodSoFar && index >= 1)
{
if (!Character.isDigit(custNumber.charAt(index)))
goodSoFar = true;
index++;
}
return goodSoFar;
}
}

welbyobeng 3 Junior Poster in Training · Answer 3 · 2008-04-08T01:52:38+00:00

This is what I have done so far

private static boolean isValid(String custNumber)
{
boolean goodSoFar = false;
int index = 0;
// Test if string the correct length
if (custNumber.length() >= 6)
goodSoFar = true;
// Test if string has at least one UPPER case
while (goodSoFar && index >= 1)
{
if (!Character.isUpperCase(custNumber.charAt(index)))
goodSoFar = true;
index++;
}
//Test if string has at least one lower case
while (goodSoFar && index >= 1)
{
if (!Character.isLowerCase(custNumber.charAt(index)))
goodSoFar = true;
index++;
}
//Test if string has at least one Digit
while (goodSoFar && index >= 1)
{
if (!Character.isDigit(custNumber.charAt(index)))
goodSoFar = true;
index++;
}
return goodSoFar;
}
}

new_2_java -3 Junior Poster · Answer 4 · 2008-04-08T01:53:10+00:00

You should change your while loop as follow: As well, you need to re-initialize your index to zero, each time you are verifying.

int index = 0;
while (goodSoFar && index < 6) {
     // your checking here ...
}

index = 0;
while (goodSoFar && index < 6) {
     // your checking here ...
}

welbyobeng 3 Junior Poster in Training · Answer 5 · 2008-04-08T02:06:12+00:00

I dont understand can you please it in the code above in one example for me to see it

new_2_java -3 Junior Poster · Answer 6 · 2008-04-08T19:30:45+00:00

private static boolean isValid(String custNumber) {
   boolean goodSoFar = false;
   int index = 0;

   // Test if string the correct length
   if (custNumber.length() >= 6) {
      goodSoFar = true;
   }

   // Test if string has at least one UPPER case
   while (goodSoFar && index < 6) {
      if (!Character.isUpperCase(custNumber.charAt(index))) {
         goodSoFar = true;
         index++;
      }
   }

   //Test if string has at least one lower case

   index = 0
   while (goodSoFar && index < 6) {
      if (!Character.isLowerCase(custNumber.charAt(index))) {
         goodSoFar = true;
         index++;
      }
   }

    // .. and so on
}

welbyobeng 3 Junior Poster in Training · Answer 7 · 2008-04-09T21:07:41+00:00

I have been on this the whole day but it is not working! I am trying to create a java password code where the password is at least 6 characters Long, at least 1 UPPERCASE, & at least 1 lowercase & at least 1 Digit.Here is an example: Passw3

private static boolean isValid(String custNumber) {
boolean goodSoFar = true;
int index = 0;
// Test if string the correct length
if (custNumber.length() < 6) {
goodSoFar = false;
}
// Test if string has at least one UPPER case
while (goodSoFar && index < 1) {
if (!Character.isUpperCase(custNumber.charAt(index))) {
goodSoFar = false;
index++;
}
}
//Test if string has at least one lower case
index = 0;
while (goodSoFar && index < 1) {
if (!Character.isLowerCase(custNumber.charAt(index))) {
goodSoFar = false;
index++;
}
}
//Test if string has at least one Digit
index = 0;
while (goodSoFar && index < 1) {
if (!Character.isDigit(custNumber.charAt(index))) {
goodSoFar = false;
index++;
}
}
return goodSoFar;
}
}

new_2_java -3 Junior Poster · Answer 8 · 2008-04-09T23:28:48+00:00

Try this. This should work 100%

private static boolean isValid(String custNumber) {
	boolean goodSoFar = true;
	boolean isOK = false;

	// Test if string the correct length
	if (custNumber.length() < 6) {
		goodSoFar = false;
	}

	if (goodSoFar) {
		// Test if string has at least one UPPER case		
		for (int i=0;i < custNumber.length() && !isOK;i++ ) {
			if (Character.isUpperCase(custNumber.charAt(i))) {
				isOK = true;
			}
		}

		if (!isOK)
			goodSoFar = false;
		isOK = false;
		for (int i=0;i < custNumber.length() && !isOK;i++ ) {
			if (Character.isLowerCase(custNumber.charAt(i))) {
				isOK = true;
			}
		}
		if (!isOK)
			goodSoFar = false;
		isOK = false;
		for (int i=0;i < custNumber.length() && !isOK;i++ ) {
			if (Character.isDigit(custNumber.charAt(i))) {
				isOK = true;
			}
		}
		if(!isOK)
			goodSoFar = false;
	}
	return goodSoFar;

}

javaAddict 900 Nearly a Senior Poster Team Colleague Featured Poster · Answer 9 · 2008-04-10T14:45:23+00:00

It would be if you use one for-loop instead of 3. You could have 3 boolean variables inside the for-loop taking values true or false depending on the current character. And at the end check if all of them are true

Java Help

Recommended Answers Collapse Answers

All 12 Replies

Recommended Answers