Forum PHP.pl > [PHP] Funkcja

Pomoc - Szukaj - Użytkownicy - Kalendarz

artekp999

28.08.2012, 10:57:14

Stwierdziłem że aby zoptymalizować mój kod muszę poużywać funkcji. Mam problem z przypisywaniem zmiennej, jeżeli zamiast zmiennej $video_zdjecie używam echo wszystko działa ale ja potrzebuje żeby zapisywało to do zmiennej.

[PHP] pobierz, plaintext 
// Początek kodu zaraz po <body>
function video_czy_zdjecie($obrazek_spr)
{
    $formaty_graficzne = array("jpg", "jpeg", "png", "gif", "tiff", "bmp", "svg");
     $format = explode(".", $obrazek_spr);
     $rozszerzenie = strtolower($format[1]);
 
     if (in_array($rozszerzenie, $formaty_graficzne)) {
     $video_zdjecie = '<img src="/zdjecia/'.$obrazek_spr.'"/>';
      }
      else
      $video_zdjecie = '<iframe width="600" height="300" src="https://www.youtube.com/embed/'.$obrazek_spr.'" frameborder="0" allowfullscreen></iframe>';      
}
 
// kilkadziesiąt linijek dalej
video_czy_zdjecie($rekord[obrazek]);
echo $video_zdjecie;
[PHP] pobierz, plaintext

rocktech.pl

28.08.2012, 11:34:38

Witam.

Funkcje powinny zwracać wartość a nie ją wyrzucać na ekran.

[PHP] pobierz, plaintext 
function video_czy_zdjecie( $obrazek_spr ) {
 
return $video_zdjecie;
}
 
echo video_czy_zdjecie ( $obrazek_spr );
[PHP] pobierz, plaintext

Zerknij też na http://php.net/manual/en/functions.anonymous.php.

pamil

28.08.2012, 14:54:51

Twoja funkcja sparwdza ciąg między pierwszą kropką a drugą. Czyli "szablon.jpg.avi" w twoim skrypcie ma rozszerzenie jpg.
Po drugie, twoja funkcja video_czy_zdjecie() powinna zwracać 'video' lub 'zdjecie', sama konstrukcja lingwistyczna pytania "Viedo czy zdjęcie?" nakazuje albo odpowiedź "Video!", albo "Zdjęcie".
Ty za to chcesz zwrócić kod HTML który wyświetla dany zasób.

artekp999

28.08.2012, 21:39:04

Ok czyli muszę to inaczej rozwiązać

pamil
1. Z fragmentu tego kodu do dobrze wnioskujesz. Ale w połączeniu z całym kodem którego tu nie umieściłem działa dobrze, ponieważ wcześniej w formularzu inaczej sprawdza rozszerzenie. A z tym video_czy_zdjecie to było pisane na szybko aby znaleźć dobre rozwiązanie a później odpowiednio przerobić.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.