Kan deze code korter/compacter?

[lano]

Hallo Forum lezers/helpers,

Kan onderstaande code simpeler of kleiner gemaakt worden om bijvoorbeeld maar een foreach te gebruiken. Mocht het kunnen? Hoe pas ik dan twee foreaches aan naar een foreach?

<?php

$string = file_get_contents('mijnwebsite.nl');
preg_match_all('/href=\"(.*?)\"/i', $string, $matches);
$result = array_unique($matches);

preg_match_all('/wrapped\" title=\"(.*)\">(.*?)<\/span>/i', $string, $titel);
$titels = array_unique($titel);

foreach($result as $match){
foreach ($titels as $naam){
for ($x = 0; $x < count($match); $x++){
echo '<a '.$match[$x].'>'.$naam[$x].'</a>';
echo '<br>';
}
}
}
?>

 

[wampier]

Als je bij je preg_match all kiest voor :

preg_match_all('/wrapped\" title=\"(.*)\">(.*?)<\/span>/i', $string, $titel,PREG_SET_ORDER);

staan je matches mogelijk beter georganiseerd (geen idee wat je terugkrijgt dus moet je even testen). Overigens is 2x foreach geen ramp

 

[lano]

Bedankt voor de uitleg Wampier.