Need Some Help With Preg_match

[madcitysw]

What i am trying to do is get any tags that don't have numbers in the id and price, so I can report an error:

[lock id="3" price="45"]heh[/lock] = This is OK

[lock id="3sa" price="45"]heh[/lock] = This is bad!

preg_match("#\[lock id=([\"']|"|&\#039;|&\#39;)(\d+)([\"']|"|&\#039;|&\#39;) price=([\"']|"|&\#039;|&\#39;)(\d+)([\"']|"|&\#039;|&\#39;)\](.+?)\[/lock\]#is", $pclass->post['post'])

However, I had try replacing the \d+ to like \D+ and it don't work very well because the char must be at the begining of the id or price for it to return true, I want it to return true if it found any non-decial number in the id and price. What am I doing wrong.

[barkerja]

if (preg_match("#\[lock id=([\"']|"|&\#039;|&\#39;)(\d+)([\"']|"|&\#039;|&\#39;) price=([\"']|"|&\#039;|&\#39;)(\d+)([\"']|"|&\#039;|&\#39;)\](.+?)\[/lock\]#is", '[lock id="3" price="45"]heh[/lock]')) {
	echo 'good';
}
else {
	echo 'bad';
}

Basically I just copied what you have and it works fine for me.

[madcitysw]

NONO, I want it to return true when it finds bad values for the id and price. I need it to be the other way heh.

[barkerja]

Ah okay then add a ! to the begining of the condition. Simple as that..

if (!preg_match(......

[madcitysw]

Um... I really don't think that would work for me, you see it suppose to be checking if there are errors in the tags, it would return true if it didn't find any tags if I used the !, that common sense.

[barkerja]

I think I am confused because you're wanting exactly the opposite of what it's doing now. Well, if you do as I suggested then it's just that. It will in fact return true if it find that string but that's a good thing. Because there are obviously errors in the string if it matches.

if (!preg_match("#\[lock id=([\"']|"|&\#039;|&\#39;)(\d+)([\"']|"|&\#039;|&\#39;) price=([\"']|"|&\#039;|&\#39;)(\d+)([\"']|"|&\#039;|&\#39;)\](.+?)\[/lock\]#is", '[lock id="as3" price="45"]heh[/lock]')) {
	echo 'true contains char';
}
else {
	echo 'false doesnt contain char';
}

That's going to echo true because of the as3. Take the as out and it will be false. Isn't this what you want or am I way off?

[madcitysw]

OK i tested it and found bad news, it works if there only one tag, but if there more than one tag, it fails to find the second tag, this is what i mean.

[lock id="222" price="2"]hehe[/lock] - Reports OK
[lock id="222a" price="2"]hehe[/lock] - Reports OK

but if the bad tag was alone like this
[lock id="222a" price="2"]hehe[/lock] - Reports BAD

I also tryed using the preg_match_all function and the same result.

I have solve this with the following

        if ($tag_count = preg_match_all("#\[lock id=([\"']|"|&\#039;|&\#39;)(.+?)([\"']|"|&\#039;|&\#39;) price=([\"']|"|&\#039;|&\#39;)(.+?)([\"']|"|&\#039;|&\#39;)\](.+?)\[/lock\]#is", $pclass->post['post'], $tag_matches)) {
          for ($i = 0; $i <= $tag_count - 1; $i++) {
            if (!is_numeric($tag_matches[2][$i]) or !is_numeric($tag_matches[5][$i])) {
            $pclass->class->obj['post_errors'] = "cant_post_purchase_lock_problem";
            $pclass->show_form();
            }
          }
        }