preg_match array

Question

It says that it "expects parameter 1 to be string". So i guess you cant have an array in preg_match. But what should I use instead? Thanks

$config['forbidden_names'] = array('admin', 'moderator', 'hoster');


if (preg_match($config['forbidden_names'], $string)) echo "Forbidden name!";

edit: in_array maybe. lol. im stupid

Duane · Accepted Answer · 2010-07-15 18:10:50Z

1

Change your second line to:

if (in_array($string, $config['forbidden_names'])) echo "Forbidden name!";

preg_match uses a regular expression, you should stay away from it unless you really need to search through a regex string, because it is slow.

answered Jul 15, 2010 at 18:10

Duane

4,5091 gold badge27 silver badges27 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Artefacto · Accepted Answer · 2010-07-15 18:29:52Z

1

if (in_array($string, $config['forbidden_names']))
    echo "Forbidden name!";

Or if you must use regex:

if (preg_match('/^(?:admin|moderator|hoster)$/', $string))
    echo "Forbidden name!";

edited Jul 15, 2010 at 18:29

answered Jul 15, 2010 at 18:04

Artefacto

98.1k17 gold badges207 silver badges230 bronze badges

2 Comments

Gumbo Over a year ago

Your regular expression does match strings that either start with admin, have moderator at any position, or end with hoster.

Artefacto Over a year ago

@Gumbo Eh damn precedence. Fixed, but feel free to edit any time.

antpaw · Accepted Answer · 2010-07-15 18:05:15Z

0

try this

if (preg_match('/(admin|moderator|hoster)/i', $string)) echo "Forbidden name!";

btw i would rather add to your roles model/table a new column/settings var that manages stuff user is allowed to add. this looks insecure to me.

answered Jul 15, 2010 at 18:05

antpaw

16.1k11 gold badges63 silver badges94 bronze badges

2 Comments

Paulo Coghi Over a year ago

It's exactly what we don't want to do. This array is just an example.

antpaw Over a year ago

a very bad example then, bacause i can imagen a form where user could edit his roles and by only editing the dom he could assign himself a admin role. so not sure what he ment

Collectives™ on Stack Overflow

preg_match array

3 Answers 3

Comments

2 Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

2 Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related