[PHP]Domyślne zdjęcie jeżeli brak odpowiedniego na serwerze Opcje

[eeagle]

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 

[nospor]

co przy większej ilości niż 1 robi problem

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

[SaMi]

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 

[nospor]

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

[eeagle]

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 .

[Turson]

glob

[nospor]

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

[Turson]

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

[SaMi]

@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 

Ten post edytował SaMi 15.12.2014, 09:30:16

[nospor]

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)){ ... }
?

[eeagle]

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.

[nospor]

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

[eeagle]

możesz mi to badziej wytłumaczyć?

[nospor]

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

[eeagle]

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 

[nospor]

robi to samo co poprzedni skrypt. Czyli ani kroczku do przodu

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....

[by_ikar]

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

[Pyton_000]

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

Ten post edytował Pyton_000 15.12.2014, 12:52:44

[nospor]

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

[by_ikar]

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.