[RESOLVED] preg_match_all help matching values between <li></li>

[skip]

Hi all I have some html and I am trying to grab some specific data from it using preg_match_all

so my data looks like this: (marked with data wanted)

<h3>Title value 1</h3> <!--// Want text only -->
<div><b>Some text</b>£10.00 </div> <!--// Want numbers only -->
<ul>
<li>1000 minutes to same network</li>
<li>3000 minutes</li> <!--// Want numbers only -->
<li>300 mins to other network</li>
<li>6000 texts</li> <!--// Want numbers only -->
<li>Unlimited web data</li> <!--// Want value before web data only -->
</ul>

<h3>Title value 2</h3> 
<div><b>Some text</b>£10.00</div>
<ul>
<li>5000 minutes</li>
<li>2000 texts</li>
<li>500Mb web data</li>
</ul>

<h3>Title value 3</h3> 
<div><b>Some text</b>£10.00</div>
<ul>
<li>6000 minutes</li>
<li>600 mins to other network</li>
<li>3000 texts</li>
<li>1Gb web data</li>
</ul>

I have written this code but it doesn't quite work - it seems to repeat the data in <li> with and without the html tags

preg_match_all('/<h3>(.*?)<\/h3>/si',$termsarray[$it],$tariffname);

preg_match_all('/<b>Handset Price<\/b>(.*?)<\/div>/si',$termsarray[$it],$price);

preg_match_all('/<li>(.*)minutes<\/li>/si',$termsarray[$it],$mins);

preg_match_all('/<li>(.*)texts<\/li>/si',$termsarray[$it],$txts);

preg_match_all('/<li>(.*)Mobile Web<\/li>/si',$termsarray[$it],$idata);

[Weedpacket]

skip wrote:

I have written this code but it doesn't quite work - it seems to repeat the data in <li> with and without the html tags

Yes. As the manual page for [man]preg_match_all[/man] says:

Orders results so that $matches[0] is an array of full pattern matches, $matches[1] is an array of strings matched by the first parenthesized subpattern, and so on.

Meanwhile, PHP already has an HTML parser as part of the [man]DOM[/man] extension.

[skip]

yep I can see that I call the results like this:

for($i=0;sizeof($mins[1]);$i++){
echo  $mins[1][$i];
echo '<br />';
}

my question is how do I just get the data I need?

Cheers

[NogDog]

Wrap the part of the pattern you want to actually capture in parentheses (so that it is a sub-pattern), then access it from the resulting array via the index for that sub-pattern. (Sometimes it's easiest to print_r() or var_dump() the result array to figure out which index to use if you have multiple and/or nested sub-patterns.)

[skip]

I do have them wrapped in parentheses:

preg_match_all('/<h3>(.*?)<\/h3>/si',$termsarray[$it],$tariffname);

preg_match_all('/<b>Handset Price<\/b>(.*?)<\/div>/si',$termsarray[$it],$price);

preg_match_all('/<li>(.*)minutes<\/li>/si',$termsarray[$it],$mins);

preg_match_all('/<li>(.*)texts<\/li>/si',$termsarray[$it],$txts);

preg_match_all('/<li>(.*)Mobile Web<\/li>/si',$termsarray[$it],$idata);  

could you provide an example?

Cheers

[Weedpacket]

So what exactly are you getting, and what's wrong with it? Could you provide an example, in other words?

[skip]

I figured it out thanks

removing /si from the preg_match_all fixed it 🙂