[RESOLVED] PHP While....Loop - Help Needed

[modpluz]

hi guyz, i have this question like;

In ASP there is something like

Do While arg

some more codes...

Loop

what that does in ASP/VB is that it loops until the argument is false

and i thought it should be doing the same thing with PHP's while function

but it doesn't instead it loops forever.

take this code for example;

'Note that this is an ASP Code
strPost = "[EDIT]C:\Program Files\bpmp\htdocs\smat_3.jpg[/EDIT]"
Do While InStr(1, strPost, "[EDITED]", 1) > 0 AND InStr(1, strPost, "[/EDITED]", 1) > 0

'some more codes...

Loop

but if i try that in PHP using the While....Loop then it loops for ever

pls note the function InStr() function,this is used to check if a given string occurs in another string and also it returns the position where the occurence starts and i have a similar php function that does that.
here it is.

function InStr($string,$find,$CaseSensitive = false) {
        $i=0;
        while (strlen($string)>=$i) {
            unset($substring);
            if ($CaseSensitive) {
                $find=strtolower($find);
                $string=strtolower($string);
            }
            $substring=substr($string,$i,strlen($find));
            if ($substring==$find) return $i;
            $i++;
        }
        return -1;
    } 

[penagate]

Use strpos() and stripos() to do an Instr.

[modpluz]

it still does the same thing...it loops forever

[penagate]

You don't need to write your own InStr function.

[modpluz]

ok...am now using strpos() and it still does the same thing

[penagate]

I am sorry that I can't post as long a post as I would like to explain.
strpos() and stripos() are builtin PHP functions that do what InStr does.
strpos() is case-sensitive and stripos() is not.

You would write this:

InStr(1, strPost, "[EDITED]", 1)

like this:

strpos($strPost, '[EDITED]') 

- it's the same function.


So there is no need to write your own function as they are already built in.


edit: Thanks for the merge Rob

[modpluz]

found a way to do this, instead of using the strpos() function, i did it this way.
i checked the string for all matches of [EDITED] by using the preg_match_all() function after which i used For...Loop to loop through the string and change whatever i wish to change.

[visualAd]

Whats wrong with the strpos function? - it is a lot faster than pregh_match_all which actually, compiles, searches and stores internally the positon of every occurance of [EDITED] in the string.

[modpluz]

thanks...i don't know why it doesn't work for me.strpos() does get the position where the string [EDITED] started(i think).
but what if i have more than one [EDITED] in my string?