PHP strlen question

Question

Ok I am checking that a string is at least 4 characters long and 25 or less characters short

I tried to use strlen like this

$userNameSignupLength = strlen($userNameSignup);

else if($userNameSignupLength<4 && $userNameSignupLength>25) {

            $userNameSignupError = "Must be between 4 to 25 characters long";

        }

but it doesn't work... what did I do wrong?

You might want to look at http://www.php.net/manual/en/function.mb-strlen.php , strlen counts multibyte characters like UTF-8 incorrectly. — , Mar 22 '10 at 14:29
@p4blo, no - 25 is ok, so he wants to check `> 25` for ones which are too long. — nickf, Mar 22 '10 at 14:29

score 8 · Accepted Answer · answered Mar 22 '10 at 14:26

8

Using strlen is correct to check the length of a string (in bytes). But a number cannot be both smaller than 4 and greater than 25 at the same time. Use || instead:

if ($userNameSignupLength < 4 || $userNameSignupLength > 25)

Now the condition is fulfilled if the number is either smaller than 4 or greater than 25.

answered Mar 22 '10 at 14:26

Gumbo

643,351
109
780
844

thank you very much lol I do that all the time mess up my ands and ors >< one day I will get it lol ^.^ – MrEnder Mar 22 '10 at 14:30

score 2 · Answer 2 · answered Mar 22 '10 at 14:26

2

Change the && to ||

else if ($userNameSignupLength<4 || $userNameSignupLength>25)

answered Mar 22 '10 at 14:26

Kevin

13,153
11
60
87

score 2 · Answer 3 · answered Mar 22 '10 at 14:28

2

I think you want an OR there:

else if($userNameSignupLength < 4 || $userNameSignupLength > 25) {

Like Gumbo said, the length cannot possibly be both less than 4 AND greater than 25. && means and.

answered Mar 22 '10 at 14:28

Andy Shellam

15,403
1
27
41

score 2 · Answer 4 · answered Jan 20 '12 at 14:28

2

and get rid of the 'else' in front of the 'if' keyword!

answered Jan 20 '12 at 14:28

Chris Walsh

3,423
2
42
62

score -1 · Answer 5 · answered Sep 04 '15 at 14:34

With your code, it is evident that you are trying to validate a text field. You have not shared html of this code. Anyways , from my expertise i will say that you should not use &&. You should use ||. Also you should not directly use else if. You should use if for this code.

PHP strlen question

5 Answers5

Linked

Related