Napisałem funckje do wyciagania adresow obrazow ze strony htm objetych w tagach <img src=""> oraz w <... background="" >

[PHP] pobierz, plaintext 
<?php
 
function getImages($string)
{
	$string=substr($string,strpos($string,'<body'));
	$pieces=explode(' ',$string);
	foreach ($pieces as $val)
	{
		$pos = strpos($val,'src="');
 
		if ($pos === 0)
		{
 
			$temp=substr(trim($val),5);
 
			$temp=substr($temp,0,strlen($temp)-1);
			if ((strpos($temp,'.jpg'))||(strpos($temp,'.gif'))||(strpos($temp,'.png')))
				$return[]=$temp;
		}
 
		$pos = strpos($val,'background="');
 
		if ($pos === 0)
		{
 
			$temp=substr(trim($val),12);
 
			$temp=substr($temp,0,strlen($temp)-1);
			if ((strpos($temp,'.jpg'))||(strpos($temp,'.gif'))||(strpos($temp,'.png')))
				$return[]=$temp;
		}
	}
	return $return;
}
 
?>
[PHP] pobierz, plaintext 

Funkcja dziala ale mysle ze mozna rozwiazac to wyrazeniami regularnymi - co moja mocna strona nie jest. Jesli ma ktos pomysl, lub dojzy blad w funkcji to prosze o komentarz.

Moja wersja (ale i tak można pewnie to zoptymalizować):

[PHP] pobierz, plaintext 
<?php
function getImages( $string )
{
   $output = array();
   if( preg_match_all( '#<.*(src|background)=('|")([^'"]+)('|").*>#i', $string, $out ) )
   {
	  $output = $out[ 3 ];
   }
   return $output;
}
?>
[PHP] pobierz, plaintext 

[ot] Skrypt nie ma sensu gdy trafisz na stronę zrobioną w xhtmlu ;] [/ot]

Zamiast zachlannnego operatora * uzyj *?. Czyli .* zamien na .*?.