code wont enter into loop. thanks in advance for your help

im writing a password check program.
i want it to give "a valid password" if the rules are satisfied and "not a valid password" if they are not, along with the violated rules. but it wont enter into the first loop if i put for example "cmps200" which is a correct password it would type "not a valid password" any suggestions(if it helps the rules are length>6 and <15, at least a digit and at least a number and doesnt end with 99 after a sequence of alphabetical letters)
if(((len > 6) && (len < 15)) && ((charCount > 0)&& (intCount> 0)) && ((charCount > 0) &&((S.charAt(len-1)) != '9' && (S.charAt(len-2) != '9')))) //all rules satisfied
{
if ((charCount > 0) && (intCount> 0))
{
if ((charCount > 0) && ((S.charAt(len-1) != '9') && (S.charAt(len-2) != '9')))
{
System.out.println("a valid password");
}
}
}

else
{
System.out.println("not a valid password");

if ((len < 6) || (len > 15))
{
System.out.println("rule 1 violated");
}
i posted up to rule one.

your if part is confusing ..

You post a code .Then only we can understand what charCount,intCount and how you are manipulating to come to this if etc we will know.

It's highly impossible to give solution without knowing head and foot.

charCount is used to count if there are any characters and intCount to count if there are any integers
(used to check that at least there is 1 integer and character) and !9 since i dnt want the last two digits to be nine if it is proceeded by an alphabetical sequence.
thanks

You are not getting what Iam saying...Iam able to get your requirement.

But,my question is you just post your code for getting charCount and intCount.I want to see to correct you.

Also use code tags.
The lack of formatting in that code makes it hard to see the flow.

[QUOTE=farahm;158354]

You're overcomplicating matters (you have included too many redundant tests and too many parentheses). This should do it:

Code:

---

if (len > 6 && len < 15 && charCount > 0 && intCount > 0 && (S.charAt(len-1) != '9' || S.charAt(len-2) != '9')) //all rules satisfied
  System.out.println("a valid password");
else
  System.out.println("not a valid password");

---

kind regards,

Jos

i figured it out
what was wrong with the code is that between each condition/rule i should have put || not &&(or not and).

Quote:

---

Originally Posted by farahm

i figured it out
what was wrong with the code is that between each condition/rule i should have put || not &&(or not and).

---

It's just boolean logic: let E be the condition that the last character is a '9', let F be the condition that the second to last character is a '9'. So:

Code:

---

E: S.charAt(len-1) == '9'
F: S.charAt(len-2) == '9'

---

You don't want both of them to be true (then the String S ends with 99), in other words: !(E&&F) == !E || !F; this translates to Java as:

Code:

---

S.charAt(len-1) != '9' || S.charAt(len-2) != '9'

---

kind regards,

Jos