Filter na img z formularza... Opcje

[Ania-678]

Witam serdecznie.

Chcę zrobić "filtrowanko" na linki wkładane pomiędzy znaczniki bbcode (jak to jest na wielu skryptach for), które są wyciągane z przychodzącego (z formularza) do skryptu tekstu. Wymodziłam takie coś:

[PHP] pobierz, plaintext 
<?php
$str = $_POST['test'];
preg_match_all('#[img](.*?)[/img]#si', $str, $img);
$i = 0;
foreach ($img as $val)
{
	$val[$i] = preg_replace('#[img](.*?)[/img]#si', '1', $val[$i]);
	if(getimagesize($val[$i]))
	{
		echo '<br />OK<br />';
	}
	$i++;
}
 
?>
[PHP] pobierz, plaintext 

Jednak jest pewne ALE...

...jeśli w tekście jest więcej niż 2x [ img] [ /img] to sprawdza tylko dwa razy (2 razy echo'uje się...).

Dlaczego tak się dzieje - nie wiem, już kombinowałam na różne sposoby ale nic z tego nie wychodzi Tak więc przychodzę z tym problemem na te forum

[rzymek01]

bardzo proszę:

[PHP] pobierz, plaintext 
<?php
$str = $_POST['test'];
 
preg_match_all('#[img](.*?)[/img]#si', $str, $img);
 
$val = &$img[1];
for ($i = 0, $count = count($val); $i < $count; ++$i)
{
  if (getimagesize($val[$i]))
  {
	  echo '<br />OK<br />';
  }
}
 
?>
[PHP] pobierz, plaintext 

zgodnie z manualem jesli nie ma ustawionej zadnej flagi to preg_match_all przyjmuje flagę PREG_PATTERN_ORDER
zgodnie z którą w indeksie [0] tablicy $img masz wyniki wraz z tagami [img][/img]
a w indeksie [1] tylko to co kazałaś zapamiętać, więc w naszym przypadku chodzi o indeks [1]

i teraz juz wszystko co chciałaś znaleźć masz w $img[1] i nie potrzeba już dalej szukać w pętli,
teraz można już spokojnie wykonać wszystkie operacje na wynikach

Pozdrawiam!

[Ania-678]

Działa idealnie, dziękuję bardzo.

Tylko pozostaje mały problemik, a mianowicie próba sprawdzenia Nieistniejącego pliku to wywala:

Warning: getimagesize(http://localhost/test/2.jpg)[function.getimagesize]: failed to open stream: HTTP request failed! HTTP/1.1 404 Not Found in D:Program FilesWebServhttpd\testindex.php on line 35

i nie bardzo potrafię się tego pozbyć

Ten post edytował Ania-678 21.07.2008, 12:21:20

[rzymek01]

daj znak małpy @ przed nazwą funkcji

Pozdrawiam!

[Shili]

daj znak małpy @ przed nazwą funkcji

To nie jest dobry pomysł, błędy należy eliminować, a nie maskować.

Dodaj

[PHP] pobierz, plaintext 
<?php
if file_exists($val[$i]))
?>
[PHP] pobierz, plaintext 

tam, gdzie to potrzebne.

[rzymek01]

i po co?
i potem jeszcze getimagesize?

fukcja getimagesize po prostu wysyła nam powiadomienie, że takiego pliku nie ma

a co jesli serwer nie ma włączonych wraperów url? kolejny błąd

Ten post edytował rzymek01 22.07.2008, 12:42:12

[Shili]

A co tutaj robi getimagesize? Sprawdza czy plik istnieje i wyświetla komunikat, w przeciwnym przypadku warning.

O wyłączonych wrapperach programista raczej wie na wstępie, o tym czy plik wpisany przez użytkownika istnieje niekoniecznie i należy wręcz zakładać (zgodnie z zasadą ograniczonego zaufania do użytkowników), że jednak część z nich istniała nie będzie. I tym należy się odpowiednio zająć, a nie problem maskować

[rzymek01]

jesli chodzi o pliki - masz rację, ale tu chodzi czy ten plik to obrazek, a nie czy plik który wpisał user istnieje