[RESOLVED] Problem with password checker

[viraj]

I got a small program which take the password of user and checks its strength. That is if password consist of Uppercase alphabets, Lowercase alphabets, digits and symbols. Am not getting the expected result. for example if i put password as raj, i get

Digit 2
Uppercase 1 instead of Lowercase 3

Digit 2
Uppercase 1
Lowercase 0
Symbol 0 

											<?php
						if($_POST['submit'] == "Check Password Strength")
						{
						$password = $_POST['password'];
						$pwd = strlen($password);

					$arrPwdLen 	= array();
					// Inserting password in array
					for($ik = 0; $ik < $pwd; $ik++)
						{
							array_push($arrPwdLen, $ik);
						}

					// Retrieving elements of array and checking to which category they belong:
					// Uppercase alphabets, Lowercase alphabets, Digits or Special symbols

					$LcaseAlpha		= 0;
					$UcaseAlpha		= 0;
					$Digit					= 0;
					$Symbol				= 0;

					for($i=0; $i<$pwd;$i++)
						{
							if(($arrPwdLen[$i]  >= '&#65;') && ($arrPwdLen[$i]  <= '&#90;'))
								{
									$UcaseAlpha++;																// Increment if element is uppercase alphabet	
								}
							 else if(($arrPwdLen[$i]  >= '&#97;') && ($arrPwdLen[$i]  <=  '&#122;'))
								{
									$LcaseAlpha++;																// Increment if element is lowercase alphabet	
								}
							 else if(($arrPwdLen[$i]  >= "&#32;" ) && ($arrPwdLen[$i]  <= "&#47;") || ($arrPwdLen[$i]  >= "&#58;" ) && ($arrPwdLen[$i]  <= "&#63;") || ($arrPwdLen[$i]  >= "&#91;" ) && ($arrPwdLen[$i]  <= "&#96;") || ($arrPwdLen[$i]  >= "&#123;" ) && ($arrPwdLen[$i]  <= "&#126;"))
								{
									$Symbol++;																	// Increment if element is a special symbol
								}
							else 
								{
									$Digit++;																		// Increment if element is digit																	
								}

						}

					echo "Digit ".$Digit . "<br>";
					echo "Uppercase ".$UcaseAlpha ."<br>";
					echo "Lowercase ".$LcaseAlpha ."<br>";
					echo "Symbol " . $Symbol;
					}					
					?>

[dagon]

I would foreach loop through the string then use is_numeric() and ctype lower() and ctype upper()

[rulian]

Try This, returns an array with properties,
just careful, cause "Symbols" includes spaces
TrimSymbols is symbols skipping spaces

<?PHP
$string ="This Is My String 0 392 / %449 blah"; 

class protoString {
/// Rulian Class
	function getProperties($string)
	{
	$strlen=strlen($string); 
		$retVar["UpperCase"]=$this -> rexCount($string, "/[A-Z]/"); 
		$retVar["LowerCase"]=$this ->rexCount($string, "/[a-z]/"); 
		$retVar["Digits"]=$this ->rexCount($string, "/[0-9]/"); 
		$retVar["Symbols"]=$this ->rexCount($string, "/[^a-zA-Z0-9]/"); 
		$retVar["TrimSymbols"]=$this ->rexCount($string, "/[^a-zA-Z0-9 ]/"); 
		return $retVar; 
	}

function rexCount($string, $regex)
{
if (preg_match_all($regex, $string, $matches, PREG_SET_ORDER)) return count($matches); 
else return false;

}

}

$protoStr=new protoString; 
$stringProperties=$protoStr -> getProperties($string); 

echo "<pre>"; 
print_r($stringProperties); 
echo "</pre>"; 

?>