Witam ponownie.

Otóż chciałbym zrobić sobie wyrażenie regularne wychwytujące wszystkie łącza (hrefy), ponieważ chce sprawdzac jakie łącza dają użytkownicy w komentarzach na mojej stronie i dodawać je do bazy, robiłem już ileś matchów jednak im bardziej starałem się poprawić, tym bardziej go niszczyłem i już kompletnie zaczął mi się mózg lansować.

$preg = preg_match_all('/<a.*href\s*=(?:\s*|\s*"|\s*\')([^\s>]+)(?:\s*?"|\s*?\'|\s*|\s*>)/i', $text, $url_list, PREG_PATTERN_ORDER);

Proszę o pomoc.

// EDIT

do moderatorow, musiałem usunąć BBCode, ponieważ forum mi docinało regexa

jakieś pomysły?

http://www.google.pl/search?hl=pl&q=UR...+Google&lr=

?

mózg może się lasować.. lansują się gwiazdy w telewizji

[PHP] pobierz, plaintext 
<?php
$match = preg_match_all('/<a.*?href="(.*?)".*?>.*?</a>/i', $text, $url_list);
?>
[PHP] pobierz, plaintext 

em1X mialem podobnego regexa na samym początku, jednak co jeśli łącza będą takie:

<a href='costam'>łącze</a>
<a href=costam>łącze</a>

możesz zrobić trzy oddzielne wyrażenia dla każdego przypadku.

em1X: ale po co dla każdego przypadku osobno?

[PHP] pobierz, plaintext 
<?php
$match = preg_match_all('/<a.*?href=('|"|)(.*?)('|"|)>.*?</a>/i', $href, $url_list );
?>
[PHP] pobierz, plaintext 

to rozwala linki z pojedynczymi i podwójnymi "uszami" i te bez też

jeden kij, btw Twój regexp nie złapałby linka w postaci: <a href=http://onet.pl title="cos"> więc małą modyfikację wypadałoby dodać

[PHP] pobierz, plaintext 
<?php
$match = preg_match_all('/<a.*?href=('|"|)(.*?)('|"|).*?>.*?</a>/i', $href, $url_list );
[PHP] pobierz, plaintext 

faktycznie, z rozpędu zabrakło jednego .*?. dzięki za poprawkę

jeszcze mały problem, bo już chce rozdać plusy, ale żeby do końca

regex em1X wyłapuje także takie przypadki:

http://php.sum1.pl/?kat=9&p=7" onFocus=blur(); class=link

nie wiem dokladnie czemu, bo na koncu linka jest cudzyslow a on pobiera dalej.

Próbowałem coś takiego:

'/<a.*?href=(?:\'|"|)(^"\'\s)*?(?:\'|"|)>.*?<\/a>/i'

ale mi wtedy nic nie pobiera...

Wrzucałem już tu kiedyś podobne wyrażenie. Powinno łapać wszystkie adresy:

[PHP] pobierz, plaintext 
<?php
 
	$text = 'aaa <a alt="aaa" href="http://aaa.pl/aaa.html" title="aaa">aaa</a> aaa
			 bbb <a alt='bbb' href='http://bbb.pl/bbb.html' title='bbb'>bbb</a> bbb
			 ccc <a alt=ccc href=http://ccc.pl/ccc.html title=ccc>ccc</a> ccc';
		 
			 
	preg_match_all( '#<as[^>]*hrefs*=s*("|')?(.*?)(?(1)1.*?>|(?:s.*?>|>))(.*?)</a>#si', $text, $matches );
 
	echo '<pre>';
	print_r( $matches );
	echo '</pre>';
 
?>
[PHP] pobierz, plaintext