outputting result with count

gotissues68

I'm using the following code:

<img src>
<img src>
<img src>
<img src>
<img src>
<?
$thisFile = file_get_contents($_SERVER[SCRIPT_FILENAME]);
$assigned_to[] = preg_match_all("/<img src>/", $thisFile, $matches);

$count=count($assigned_to);
for ($i=0; $i<$count; $i++) {
echo "$assigned_to[$i]";
}
?>

Count it outputting 6 rather then 5 and I can't seem to figure out why. I'm simply trying to count the number of occurances of an html tag. I'm not able to get it to de-increment by 1 in order to show the proper number of tags matched. Any ideas would be appreciated.

Thanks

Shrike

Because count() starts counting from 1 rather than 0.

Weedpacket

Nah, I think it's just because there are six occurrences of the string "<img src>" in the file....

gotissues68

There should only be 5 occurences, yet it outputs 6, unless the the occurence thats used within preg_match is seen as an occurence itself from within the script, is that what might be doing it?

tsinka

Hi,

I modified your script a little bit and it worked on my server:

<?PHP
// to test I use a simple string
//$thisFile = file_get_contents($_SERVER[SCRIPT_FILENAME]);
$thisFile = "<img src>aaa<img src>aaa<img src>aaa<img src>aaa<img src>aaa";
$count = preg_match_all("/<img src>/", $thisFile, $matches);
print_r($matches);
for ($i=0; $i<$count; $i++) {
echo $matches[0][$i]."<br>\n";
}
?>

Thomas

gotissues68

thanks tsinka, i tried your method and it didn't seem to work for some reason bleh. But I did find a way around it that works using my original code. Its an ugly ass hack that'll need to figured out properly at some point in time.

<img src>
<img src>
<img src>
<img src>
// 4 img src tags
<?
$thisFile = file_get_contents($_SERVER[SCRIPT_FILENAME]);
$assigned_to[] = preg_match_all("/<img src>/", $thisFile, $matches);

$count=count($assigned_to);
for ($i=0; $i<$count; $i++) {
$blah = $assigned_to[0] - 1;
echo "$blah";
}
?>

Weedpacket

Originally posted by gotissues68
There should only be 5 occurences, yet it outputs 6, unless the the occurence thats used within preg_match is seen as an occurence itself from within the script, is that what might be doing it?

That is what's doing it - no reason for that one to be skipped. For ugly hacks, try

preg_match_all("/<"."img src>/",.....

Or, if your tags are going to be at the start of the line

preg_match_all("/^<img src>/m",.....