Poszukuję pomocy w jaki sposób sprawdzić i zamienić wszystkie pliki jpg (zdjęcia) na domyślny jeżeli brakuje go na serwerze.
Poniżej kod który się sprawdza ale tylko podaniu konkretnego pliku, co przy większej ilości niż 1 robi problem

[PHP] pobierz, plaintext 
<?php
	         $plik = "pliki/plik_xs.jpg"; 
		 $domyslny = pliki/nophoto_xs.jpg";
		 $test = file_exists($plik); 
if (!$test) 
    {
    echo "<img src=". $domyslny ." /> "; 
    }
    else
    {
    echo "<img src=". $plik ." /> ";
    }
 
?>
[PHP] pobierz, plaintext 

To napisz jaki problem bo ja osobiscie nie widze zadnego problemu nie liczac faktu, ze masz spierniczone cudzysłowia

Jeśli zmienna $plik jest pusta, to i tak katalog utrzymuje status true, ponieważ file_exists zwraca true dla plików i katalogów.

Można użyć is_file() w połączeniu z nim:

[PHP] pobierz, plaintext 
    if(file_exists($plik) && is_file($plik)){ ... }
[PHP] pobierz, plaintext 

@Sami nie dosc ze to nie mazwiazku ze sprawą, to jeszcze piszesz nieprawdziwe rzeczy.... file_exists dla pustej zmiennej nie zwraca true....

Cudzysłów zaginął przy przepisywaniu .
Może pytanie zadałem nie tak.
Problem polega na tym że skrypt ten sprawdza konkretny plik. W tym wypadku plik_xs.jpg.
Nie bardzo umiem sobie poradzić aby sprawdzał każdy plik np. z rozszerzeniem jpg .

glob

Turson po co glob..przeciez glob zwraca pliki ktore istnieją a jego interesują tez pliki, ktore nie istnieją.

@eeagle no ale gdzies te liste plikow masz przeciez zapisaną no nie? No to w czym problem?
zamiast $plik = "pliki/plik_xs.jpg"; podstawiaj kolejny plik z listy

No fakt, że zamysł jest inny no to lista plików musi gdzieś być

@nospor masz rację to był skrót myślowy chodziło mi o poniższe sytuacje.

[PHP] pobierz, plaintext 
if(file_exists('images/zdjecie.jpg')) echo 'true';
else echo 'false';
// zwraca true
if(file_exists('images/')) echo 'true';
else echo 'false';
//zwraca true
if(file_exists('imag')) echo 'true';
else echo 'false';
//zwraca false
[PHP] pobierz, plaintext 

Skrot myslowy? Wyraznie napisales o pustej zmiennej.
Poza tym po co wyjezdzasz z katalogami, skoro user wyraźnie napisal, że sprawdza pliki. Ale czasami plik nie istnieje. Ani razu nie bylo mowy o sprawdzaniu katalogow i problemow z nimi związanymi
Tak wiec nie ma sensu bawic sie w
if(file_exists($plik) && is_file($plik)){ ... }
skoro zawsze sprawdzane są pliki
A nawet jesli byloby inaczej, to po co tak? Czemu nie lepiej od razu:
if(is_file($plik)){ ... }
?

Listy to nie ma , ale to nie problem. Mi nie chodzi tylko o sprawdzenie czy istnieje ale jeszcze umieszczenie (nophoto.jgp) w to samo miejsce ( w kodzie html) gdzie powinien być brakujący plik.

No to przeciez to masz juz zrobione....

Od biedy jak chcesz miec ładnie, to załaduj to w funkcje, jako parametr funkcji przekazuj sciezke do pliku i po sprawie

możesz mi to badziej wytłumaczyć?

Czy wiesz co to są funkcje?
Jesli wiesz, to kod co masz masz wlozyc w funkcję.
Jak nie wiec co to są funkcje, to wpierw doczytaj.

http://php.net/manual/en/functions.arguments.php
http://php.net/manual/en/functions.returning-values.php

Nie jestem mistrzem w funkcjach wymyśliłem taką :

[PHP] pobierz, plaintext 
function sprawdzenie ($plik = "pliki/plik_xs.jpg" , $domyslny = "pliki/nophoto_xs.jpg") {
																$test = file_exists($plik);
												  				if (!$test) 
																		{
																				echo "<img src=". $domyslny ." /> "; 
																		}
																else
																		{
																				echo "<img src=". $plik ." /> ";
																		}
 
 
															}
 
sprawdzenie();
[PHP] pobierz, plaintext 

robi to samo co poprzedni skrypt. Czyli ani kroczku do przodu
Poniżej kod html w jaki sposób wyświetlam pliki - w tym miejscu chciałbym aby mi sprawdzało czy plik jest czy nie i wrzucanie odpowiednio nophoto_xl , nophoto_xs a jeżeli jest to zostawić tak jak jest.

[HTML] pobierz, plaintext 
 
<div class="main-image">
		<div class="outer">
			<span class="inset"><a href="pliki/plik_xl.jpg"><img src="pliki/plik_xs.jpg" alt="" /></a></span>
		</div>
</div>
[HTML] pobierz, plaintext 

No toc ci mowilem, ze masz tam wstawiac potem nazwy plikow jakie chcesz miec... no same sie nie wstawią....

sprawdzenie('jakisplik');
sprawdzenie('jakisplik2');
sprawdzenie('jakispli3');
itd....

Tylko po co do tego w ogóle angażować php, jak zwyczajnie można zrobić tło dla tego diva z klasą outer, czy tam spana z klasą inset ? Kiedyś ktoś miał podobny problem: http://forum.php.pl/index.php?s=&showt...st&p=920180

Czarny scenariusz... Wyświetlasz listę 10 tyś obrazeczków. Pech chciał że 3/4 nie ma
Po co robić bezsensowne 10001 requestów skoro można tylko 3001

Bo z tamtego tematu wynika, ze by_ikar zaoszczedzil na tym 10000 file_exists

PHP działa synchronicznie, więc sprawdzenie czy działa 10 tysięcy plików, będzie dużo wolniejsze i zasobożerne niż w przypadku sprawdzenia tego przez webserwer. Wolniejsze, bo w przypadku rozłożenia tego na background, połączenia do sprawdzenia czy obrazki istnieją wykonywane są asynchronicznie (po kilka na raz, w zależności od limitu połączeń przeglądarki), więc nie czekasz na skrypt a na szybkość odpowiedzi przez webserwer który będzie dla statycznego contentu kilka razy szybszy niż PHP. Dodatkowo, tworzysz sobie jakiś loader obrazków w javascript, który będzie pobierać background dopiero kiedy user dojedzie scrollem do obrazka (lazy loading).

Się śmiejecie, a zobaczcie jak to ma zrobione allegro, czy jakieś inne większe strony, które nie trzymają obrazków w obrębie tego samego webserwera.. Powodzenia z file_exists do jakiegoś cdn'a.