[PHP] regexp - zamiana adresu URL na link Opcje

[wiewiorek]

Jeśli w tekście są adresy URL, np. http://onet.pl to chciałbym żeby były zamieniane na linki: <a href="http://onet.pl>onet.pl</a> i odpowiednie wyrażenie regularne mam:

[PHP] pobierz, plaintext 
 "https?://([-\w\.]+)+(:\d+)?(/([\w/_\.]*(\?\S+)?)?)?" 
[PHP] pobierz, plaintext 

No, ale to wyrażenie regularne zamienia mi na linki też adresy url wewnątrz, np. <img src="..." />, <a href="....">...</a>, <iframe src="..."></iframe> itp. - jak temu przeciwdziałać?

[Sephirus]

Kod

"[^\"]{1}https?://([-\w\.]+)+(:\d+)?(/([\w/_\.]*(\?\S+)?)?)?[^\"]{1}"

[wNogachSpisz]

Do parsowania adresów URL służy funkcja parse_url()

Nie wywarzaj otwartych drzwi.
Nie wynajduj koła na nowo.
Nie używaj wyrażeń regularnych do standardowych zadań.

Ten post edytował wNogachSpisz 7.11.2011, 22:18:42

[wiewiorek]

hm..... sprawdziłem wyrażenie

[PHP] pobierz, plaintext 
 [^\"]{1}https?://([-\w\.]+)+(:\d+)?(/([\w/_\.]*(\?\S+)?)?)?[^\"]{1} 
[PHP] pobierz, plaintext 

dla tekstu:
http://gazeta.pl aaaaaa http://onet.pl dd <a href="http://onet.pl">onet.pl</a> ddd http://wp.pl

Na stronie testera wyrażeń regularnych: http://regexpal.com/ i odnajduje linki prawidłowo poza sytuacją gdy link jest na samym początku tekstu?

Ten post edytował wiewiorek 8.11.2011, 07:23:08

[Sephirus]

Ok masz poprawke

[PHP] pobierz, plaintext 
"[^\"]*https?://([-\w\.]+)+(:\d+)?(/([\w/_\.]*(\?\S+)?)?)?[^\"]*"
[PHP] pobierz, plaintext 

@wNogachSpisz to może pokaż koledze jak rozwiązać jego problem przy użyciu wspomnianej przez Ciebie funkcji

[croc]

Do parsowania adresów URL służy funkcja parse_url()

Do parsowania tak. Do sprawdzania poprawności nie. parse_url używa się dla linków, które już są sprawdzone, bo ta funkcja jest zbyt tolerancyjna.

[wNogachSpisz]

Do parsowania tak. Do sprawdzania poprawności nie. parse_url używa się dla linków, które już są sprawdzone, bo ta funkcja jest zbyt tolerancyjna.

Wierutna bazdura, funkcja parse_url() może z powodzeniem służyć do walidacji adrsu URL.

Croc, widać że nie masz bladego pojęcia o niczym.
Proszę Cię, dla dobra innych formuwiczów, nie wypowiadaj się,
jeśli już musisz, to do swoich wypowiedzi załączaj klauzule: "mogę się mylić" lub "nie znam się ale wydaje mi się".
Bez tego wprowadzasz zgubną dezinformację.

[PHP] pobierz, plaintext 
<?php
 
foreach ( $link_list as $link ) {
 
	$plink = parse_url($link);
 
	if ( ! isset($plink['scheme']) OR ! in_array($plink['scheme'], array('http', 'https'))) {
		continue;
	}
	if ( ! isset($plink['host'])) {
		continue;
	}
 
	$href = "$plink['scheme']://{$plink['host']}";
	$link = $plink['host'];
}
 
?>
[PHP] pobierz, plaintext 

Ten post edytował wNogachSpisz 8.11.2011, 11:33:20

[wookieb]

@wNogachSpisz uspokój się z zarzutami i spójrz

[PHP] pobierz, plaintext 
print_r(parse_url('czy to jest url')); // [path] => czy to jest url
[PHP] pobierz, plaintext 

I co? Teraz ja powinienem na Ciebie wbijać?

[wNogachSpisz]

[PHP] pobierz, plaintext 
print_r(parse_url('czy to jest url')); // [path] => czy to jest url
[PHP] pobierz, plaintext 

Przeczytaj dokładnie mój code snip,
żeby walidować musisz dodpisać:

[PHP] pobierz, plaintext 
isset($parsed_url['scheme']);
[PHP] pobierz, plaintext 

Następnym razem przeczytaj kilka razy to co napisałem.

Ten post edytował wNogachSpisz 8.11.2011, 11:55:34

[wookieb]

Po co? Skoro jest filter_var?

[wNogachSpisz]

Po co? Skoro jest filter_var?

Jasne, jeśli chcesz tylko walidować URL, filter_var() jest dokładnie tym co potrzebujesz...

Natomiastu tutaj trzeba sparsować URL, do czego służy parse_url(), po drodze można bez konieczności używania innych funkcji sprawdzić czy URL jest poprawny..

Wywołanie jednej funkcji jest tańsze niż wywołanie dwóch.

Jedyny problem z filter_var(), to fakt że wprowadzono je późno, dopiero w wersji 5.2.0

Ten post edytował wNogachSpisz 8.11.2011, 11:57:49

[Sephirus]

Ok bo ja się pogubiłem - ludzie przecież cała ta dyskusja jest bez sensu... Autor chciał rozwiązania pozwalającego zamienić ciąg:

Kod

jakiś tekst jakiś tekst
jakiś tekst jakiś tekst
ttp://www.strona.pl
jakiś tekst jakiś tekst
jakiś tekst jakiś tekst
ttp://www.inna-strona.com
jakiś tekst jakiś tekst
jakiś tekst jakiś tekst

(sprzed adresów skasowałem H bo mi na linki zamieniało)

na

[HTML] pobierz, plaintext 
jakiś tekst jakiś tekst
jakiś tekst jakiś tekst
<a href="http://www.strona.pl">http://www.strona.pl</a>
jakiś tekst jakiś tekst
jakiś tekst jakiś tekst
<a href="http://www.inna-strona.com">http://www.inna-strona.com</a>
jakiś tekst jakiś tekst
jakiś tekst jakiś tekst
[HTML] pobierz, plaintext 

Nic nie było o żadnej validacji linków, itd...

Ten post edytował Sephirus 8.11.2011, 12:01:28

[by_ikar]

@wNogahSpisz nie chcę się spierać, ale parse_url parsuje linki, a te link trzeba mieć. Autorowi tematu chodzi o ciąg który może zawierać linki, wówczas parse_url IMO nie ma za bardzo do tego jest potrzebne. Jeden preg_match i odpowiednie wyrażenie.

@up ubiegłeś mnie

Ten post edytował by_ikar 8.11.2011, 12:02:38

[croc]

Nic nie było o żadnej validacji linków, itd...

No tak, ale żeby zamienić, trzeba najpierw znaleźć Można byle jak, np. przez wspomnianą funkcję parse_url, która przepuści większość nieprawidłowych linków, a można (większym kosztem) dobrym wyrażeniem regularnym.

[nospor]

No tak, ale żeby zamienić, trzeba najpierw znaleźć Można byle jak, np. przez wspomnianą funkcję parse_url,

Czyli że co? dla parse_url zapodasz cały tekst a on z niego wyłapie url? No raczej nie, do parse_url musisz przekazać już wybrane przez Ciebie url z tekstu a żeby to zrobić musisz tak czy siak użyć wyrażeń regularnych.

[wNogachSpisz]

W takim razie trzeba wcześniej załadować HTML do Dom:Document, co utworzy obiekt DOM,
następnie przeszukać bloki tekstowe w poszukiwaniu wyrazów rozpoczynających się na to "https?://".

Pozostaje problem zamiany konkretnego adresu url na element anachor to można w sensowny sposób osiągnąć tylko przy pomocy z preg_replace ;p
Lub jakiś inny sposób podziału na wyrazy..

Ten post edytował wNogachSpisz 8.11.2011, 12:14:40

[croc]

@nospor Masz rację - źle się wyraziłem.

@wnogachspisz - wiesz, że istnieją jeszcze inne protokoły?

Ten post edytował croc 8.11.2011, 12:10:17

[wNogachSpisz]

Dobrze gdyby autor wrzcił przykładowe dane do przeszukania..

@wnogachspisz - wiesz, że istnieją jeszcze inne protokoły?

Wiesz.
Jaki z tego morał?

Ten post edytował wNogachSpisz 8.11.2011, 12:12:43

[croc]

A po co Ci przykładowe dane? o0 Uzależniasz algorytm zamiany URL na linki od tego, jaką ma zawartość? Tzn. jakby miał inną, to byś dał inny algorytm?

[wNogachSpisz]

croc nie troluj, zaklinam cie.