Probleme mit getimagesize und array

picco · 02.03.2010, 23:36

Ich habe hier eine Bildergalerie, die bisher ohne Probleme lief. Konkret handelt es sich hier um Wallpaper. Bisher waren die Wallpaper im Format 1024*768. Allerdings möchte ich nun auch Wallpaper in höheren Auflösungen anbieten.

Das Ganze ist in etwa so aufgebaut.

Verzeichnis mit Bildern auslesen
Bestimmen wie viele Bilder dargestellt werden sollen
Alles in ein Array packen
Mit einer for-Schleife das Array "splitten", damit x Bilder pro Seite dargestellt werden
mit getimagesize ein paar Infos anzeigen

So lief alles bisher einwandfrei.

Das Ganze habe ich nun erweitert, damit Links zu den Wallpapern in der höheren Auflösung angezeigt werden. Soweit klappt alles auch. Allerdings macht getimagesize bei den großen Bildern faxen und zeigt nichts an bzw. gibt keine Werte zurück.

Da ich ich nicht zu jedem Wallpaper eines in einer größeren Auflösung habe, habe es ich so gelöst.

PHP-Code:

      if (isset($bilder1280[$x])) {

        $info1280 = getimagesize($pfad1280.$bilder1280[$x]);

        $size1280 = round(filesize($pfad1280.$bilder1280[$x])/1024, 1);

        $laenge1280 = $info1280[0];

        $hoehe1280 = $info1280[1];

        }

Klappt auch soweit. Allerdings nur, wenn ich alle Bilder auf einer Seite darstelle. Teile ich das Array wieder auf 18 Bilder pro Seite, erhalte ich keine Rückgabewerte von getimagesize. Die Links zu den Bilder werden komischerweise aber dargestellt. Hat jemand eine Idee wo das Problem liegen könnte?

Arne Drews · 03.03.2010, 08:09

Zitat:

Teile ich das Array wieder auf 18 Bilder pro Seite, erhalte ich keine Rückgabewerte von getimagesize.

Wenn Du den Teil des Codes denn auch mal postest, könnten wir zumindest schauen, ob wir helfen können...

Andreas · 03.03.2010, 09:36

Also mir drängen sich gleich zwei Fragen auf, wenn ich deinen Post lese:

Warum liest du immer alle Bilder ein, wenn blos 18 dargestellt werden sollen?
Warum splittest du das Array mit einer for-schleife? (Stichwort: array_slice)

Ansonsten warte ich auch mal auf den anderen Code

Lollix · 03.03.2010, 17:24

Stecken alle Wallpapers im selebn Ordner?

Hast du mal überprüft, ob PHP die Bilder überhaupt finden kann?

Hast du außerdem mal mit print_r oder so überprüft, ob wirklich nichts im Array drinnen ist?

Aber wie Arne Drews und Andreas schon sagten, wäre es sicher leichter für alle sich in deinen Code zu versetzen.

picco · 03.03.2010, 22:35

Ein wenig Code wäre vielleicht wirklich sinnvoll...

PHP-Code:

  // Ordner und Array

    $pfad = "img/wallpaper/";

    $tpfad = $pfad."thumbs/";

    $bilder = array();

 
# ab hier größere Bilder, so ähnlich geht es dann mit anderen Auflösungen weiter

// Ordner und Array 1280x800

    $pfad1280 = "".$pfad."1280/";

    $bilder1280 = array();

 
// Ordner durchsuchen und in Array packen 1024

$handle = opendir($tpfad);

    while($file = readdir($handle)) {

           if(preg_match("/.(jpg|gif)$/i", $file)){

              $bilder[] = $file;

        }

    }

closedir($handle);

natsort($bilder);

$bilder = array_values($bilder);

$gesamt = count($bilder);

 
# Auch hier wird es wiederholt

// Ordner auslesen

    // Ordner durchsuchen und in Array packen 1280x800

    $handle1280 = opendir($pfad1280);

        while($file1280 = readdir($handle1280)) {

            if(preg_match("/.(jpg|gif)$/i", $file1280)){

                $bilder1280[] = $file1280;

            }

        }

    closedir($handle1280);

    natsort($bilder1280);

    $bilder1280 = array_values($bilder1280);

    $gesamt1280 = count($bilder1280);

// Ordner auslesen Ende

 
// Letztes Bild der Anzeige

    $end = $start + $maximal;

 
if($end > $gesamt)$end = $gesamt;

 
// Ausgabe Bilder

echo "   <div class=\"container\">\n";

    // aktuelle Seite mit 30 bildern ausgeben

    for($x = $start; $x < $end; $x++) {

    // Infos zu den Bildern

        $info = getimagesize($pfad.$bilder[$x]);

        $size = round(filesize($pfad.$bilder[$x])/1024, 1);

        $laenge = $info[0];

        $hoehe = $info[1];

 
# Wird wiederholt mit anderen Werten        

// Infos zu großen WPs

    // Infos zu den Bildern 1280x800

    if (isset($bilder1280[$x])) {

        $info1280 = getimagesize($pfad1280.$bilder1280[$x]);

        $size1280 = round(filesize($pfad1280.$bilder1280[$x])/1024, 1);

        $laenge1280 = $info1280[0];

        $hoehe1280 = $info1280[1];

        }

// Infos zu großen WPs Ende

 
# Zum testen habe ich die Ausgabe der Thumbnails entfernt. Es werden nur die Pfade dargestellt

// Ausgabe    

    echo "    <div class=\"image-wp\">\n";

    echo "        <div class=\"image-box-wp\">\n";

    echo "            <a href=\"".$pfad.$bilder[$x]."\" target=\"_blank\">".$tpfad.$bilder[$x]."</a><br>\n";

    echo "            <a href=\"".$pfad.$bilder[$x]."\" target=\"_blank\">$laenge x $hoehe</a> || $size KB\n";

    

    // Erweiterung Wallpaper

        if (file_exists($pfad1280.$bilder[$x])) echo "\t\t\t<br /><a href=\"".$pfad1280.$bilder[$x]."\" target=\"_blank\">$laenge1280 x $hoehe1280</a> || $size1280 KB\n";

    // Erweiterung Wallpaper Ende

    

    echo "        </div>\n";

    echo "    </div>\n";

    }        

echo "   </div>\n";

// Ausgabe Bilder Ende

 
// Navigation etc... hier nicht relevant

So sieht es aus. Verbesserungsvorschläge für den Code nehme ich gerne an, da ich weiß, dass es nicht perfekt ist

nikosch · 03.03.2010, 22:43

PHP-Code:

  if (isset($bilder1280[$x])) {

ist Quatsch, denn bei fehlenden Bildern wird der Schlüssel weiter inkrementell vergeben und Normal und Großversion haben dann andere Schlüssel.

picco · 03.03.2010, 22:48

Was wäre denn eine bessere Variante? Die Wallpaper sind in verschiedenen Ordnern, aber die gleichen Bilder haben auch den gleichen Dateinamen.

nikosch · 03.03.2010, 22:53

file_exists + pfad + gemeinsamer Dateiname

picco · 04.03.2010, 01:15

Habe ich das nicht so gemacht?

Andreas · 04.03.2010, 09:29

Schau dir vielleicht wirklich mal die Funktionen/Statements hier an:

glob (in Verbindung mit dem Flag: GLOB_NOSORT)
array_slice
foreach

Damit solltest du ohne "isset" und ohne "file_exists" auskommen

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
[Erledigt] Problem bei update mit array feldern	fulltilt	PHP Tipps 2010	6	13.02.2010 00:59
probleme mit array	rodger	PHP Tipps 2009	3	05.10.2009 00:39
Kategoriedaten rekursiv in Array bringen	Schiedsrichter	PHP Tipps 2008	2	10.12.2007 15:43
Probleme beim Array schreiben	heohni	PHP Tipps 2006	8	21.06.2006 12:58
Array Probleme	Robstar	PHP Tipps 2006	12	31.05.2006 01:39
ARRAY: alle möglichen Kombinationen	Tentacle	PHP Tipps 2006	13	12.05.2006 00:18
Array Formatierung		PHP Tipps 2006	3	17.01.2006 19:12
Menü mit Unterpunkten	supertramp	Beitragsarchiv	7	18.10.2005 22:40
(schnellere) Funktion zum Zusammenfassen von CSS		PHP-Fortgeschrittene	21	08.08.2005 16:47
Probleme bei getimagesize		PHP Tipps 2005	4	16.05.2005 11:40
[Erledigt] Mehrdimensionales Array in eindimensionales Array umwandeln		PHP-Fortgeschrittene	3	03.01.2005 22:31
Abfrage mit id aus anderer Tabelle	suter	PHP Tipps 2004-2	15	16.12.2004 14:25
Probleme mit Array!		PHP Tipps 2004	8	14.07.2004 21:34
Probleme mit Form / mehrdimensionales Array		PHP Tipps 2004	3	05.07.2004 16:55

02.03.2010, 23:36
picco Erfahrener Benutzer Registriert seit: 25.08.2004 Beiträge: 128	Probleme mit getimagesize und array Ich habe hier eine Bildergalerie, die bisher ohne Probleme lief. Konkret handelt es sich hier um Wallpaper. Bisher waren die Wallpaper im Format 1024*768. Allerdings möchte ich nun auch Wallpaper in höheren Auflösungen anbieten. Das Ganze ist in etwa so aufgebaut. Verzeichnis mit Bildern auslesen Bestimmen wie viele Bilder dargestellt werden sollen Alles in ein Array packen Mit einer for-Schleife das Array "splitten", damit x Bilder pro Seite dargestellt werden mit getimagesize ein paar Infos anzeigen So lief alles bisher einwandfrei. Das Ganze habe ich nun erweitert, damit Links zu den Wallpapern in der höheren Auflösung angezeigt werden. Soweit klappt alles auch. Allerdings macht getimagesize bei den großen Bildern faxen und zeigt nichts an bzw. gibt keine Werte zurück. Da ich ich nicht zu jedem Wallpaper eines in einer größeren Auflösung habe, habe es ich so gelöst. PHP-Code: `if (isset($bilder1280[$x])) { $info1280 = getimagesize($pfad1280.$bilder1280[$x]); $size1280 = round(filesize($pfad1280.$bilder1280[$x])/1024, 1); $laenge1280 = $info1280[0]; $hoehe1280 = $info1280[1]; }` Klappt auch soweit. Allerdings nur, wenn ich alle Bilder auf einer Seite darstelle. Teile ich das Array wieder auf 18 Bilder pro Seite, erhalte ich keine Rückgabewerte von getimagesize. Die Links zu den Bilder werden komischerweise aber dargestellt. Hat jemand eine Idee wo das Problem liegen könnte?


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

03.03.2010, 09:36
Andreas Erfahrener Benutzer Registriert seit: 22.04.2005 Beiträge: 391	Also mir drängen sich gleich zwei Fragen auf, wenn ich deinen Post lese: Warum liest du immer alle Bilder ein, wenn blos 18 dargestellt werden sollen? Warum splittest du das Array mit einer for-schleife? (Stichwort: array_slice) Ansonsten warte ich auch mal auf den anderen Code

03.03.2010, 17:24
Lollix Erfahrener Benutzer Registriert seit: 25.04.2009 Beiträge: 201 PHP-Kenntnisse: Anfänger	Stecken alle Wallpapers im selebn Ordner? Hast du mal überprüft, ob PHP die Bilder überhaupt finden kann? Hast du außerdem mal mit print_r oder so überprüft, ob wirklich nichts im Array drinnen ist? Aber wie Arne Drews und Andreas schon sagten, wäre es sicher leichter für alle sich in deinen Code zu versetzen. __________________ Privatsphäre war gestern, denn: Bundestrojaner 2.0: Die neue Version ist da.

03.03.2010, 22:35
picco Erfahrener Benutzer Registriert seit: 25.08.2004 Beiträge: 128	Ein wenig Code wäre vielleicht wirklich sinnvoll... PHP-Code: // Ordner und Array $pfad = "img/wallpaper/"; $tpfad = $pfad."thumbs/"; $bilder = array(); # ab hier größere Bilder, so ähnlich geht es dann mit anderen Auflösungen weiter // Ordner und Array 1280x800 $pfad1280 = "".$pfad."1280/"; $bilder1280 = array(); // Ordner durchsuchen und in Array packen 1024 $handle = opendir($tpfad); while($file = readdir($handle)) { if(preg_match("/.(jpg\|gif)$/i", $file)){ $bilder[] = $file; } } closedir($handle); natsort($bilder); $bilder = array_values($bilder); $gesamt = count($bilder); # Auch hier wird es wiederholt // Ordner auslesen // Ordner durchsuchen und in Array packen 1280x800 $handle1280 = opendir($pfad1280); while($file1280 = readdir($handle1280)) { if(preg_match("/.(jpg\|gif)$/i", $file1280)){ $bilder1280[] = $file1280; } } closedir($handle1280); natsort($bilder1280); $bilder1280 = array_values($bilder1280); $gesamt1280 = count($bilder1280); // Ordner auslesen Ende // Letztes Bild der Anzeige $end = $start + $maximal; if($end > $gesamt)$end = $gesamt; // Ausgabe Bilder echo " <div class=\"container\">\n"; // aktuelle Seite mit 30 bildern ausgeben for($x = $start; $x < $end; $x++) { // Infos zu den Bildern $info = getimagesize($pfad.$bilder[$x]); $size = round(filesize($pfad.$bilder[$x])/1024, 1); $laenge = $info[0]; $hoehe = $info[1]; # Wird wiederholt mit anderen Werten // Infos zu großen WPs // Infos zu den Bildern 1280x800 if (isset($bilder1280[$x])) { $info1280 = getimagesize($pfad1280.$bilder1280[$x]); $size1280 = round(filesize($pfad1280.$bilder1280[$x])/1024, 1); $laenge1280 = $info1280[0]; $hoehe1280 = $info1280[1]; } // Infos zu großen WPs Ende # Zum testen habe ich die Ausgabe der Thumbnails entfernt. Es werden nur die Pfade dargestellt // Ausgabe echo " <div class=\"image-wp\">\n"; echo " <div class=\"image-box-wp\">\n"; echo " <a href=\"".$pfad.$bilder[$x]."\" target=\"_blank\">".$tpfad.$bilder[$x]."</a><br>\n"; echo " <a href=\"".$pfad.$bilder[$x]."\" target=\"_blank\">$laenge x $hoehe</a> \|\| $size KB\n"; // Erweiterung Wallpaper if (file_exists($pfad1280.$bilder[$x])) echo "\t\t\t<br /><a href=\"".$pfad1280.$bilder[$x]."\" target=\"_blank\">$laenge1280 x $hoehe1280</a> \|\| $size1280 KB\n"; // Erweiterung Wallpaper Ende echo " </div>\n"; echo " </div>\n"; } echo " </div>\n"; // Ausgabe Bilder Ende // Navigation etc... hier nicht relevant So sieht es aus. Verbesserungsvorschläge für den Code nehme ich gerne an, da ich weiß, dass es nicht perfekt ist

03.03.2010, 22:43
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.990 PHP-Kenntnisse: Fortgeschritten	PHP-Code: `if (isset($bilder1280[$x])) {` ist Quatsch, denn bei fehlenden Bildern wird der Schlüssel weiter inkrementell vergeben und Normal und Großversion haben dann andere Schlüssel. __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

03.03.2010, 22:48
picco Erfahrener Benutzer Registriert seit: 25.08.2004 Beiträge: 128	Was wäre denn eine bessere Variante? Die Wallpaper sind in verschiedenen Ordnern, aber die gleichen Bilder haben auch den gleichen Dateinamen.

03.03.2010, 22:53
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.990 PHP-Kenntnisse: Fortgeschritten	file_exists + pfad + gemeinsamer Dateiname __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

04.03.2010, 01:15
picco Erfahrener Benutzer Registriert seit: 25.08.2004 Beiträge: 128	Habe ich das nicht so gemacht?

04.03.2010, 09:29
Andreas Erfahrener Benutzer Registriert seit: 22.04.2005 Beiträge: 391	Schau dir vielleicht wirklich mal die Funktionen/Statements hier an: glob (in Verbindung mit dem Flag: GLOB_NOSORT) array_slice foreach Damit solltest du ohne "isset" und ohne "file_exists" auskommen