Enrico: Fotogalerie >>> Menü funktioniert / Anzeige Bilder nicht

Hallo,

So, nun habe ich es endlich geschafft, eine übersichtliche Navigation durch die unterschiedlichen Ordner unserer Fotogalerie am Bildschirm anzuzeigen.

Was allerdings noch nicht funktioniert, ist die Anzeige der Bilder der jeweiligen Ordner.

Desweiteren befürchte ich, dass der Code auch noch verbesserungswürdig sein dürfte.

Hier der Code:

<?php

$Verzeichnis        = $_GET ["Verzeichnis"];

$Standardpfad       = "pic/pic/klein";
  $ArrayVerzeichnisse = array ();

if (! isset ($Verzeichnis) || $Verzeichnis == "")
    $Verzeichnis = $Standardpfad;

echo $Verzeichnis . "<br><br>";

$VerzeichnisZeiger = @opendir ($Verzeichnis);

while ($Element = @readdir ($VerzeichnisZeiger))
  {
    if ($Element != "." && $Element != "..")
    {
      if (is_dir ("$Verzeichnis/$Element"))
        echo "<a href='PHP.GalerieAuswahl.php?Verzeichnis=" . $Verzeichnis . "/" . $Element ."'>" . $Element . "<br>";
      else
      {
        if (is_file ($Element))
          echo "<img src='" . $Verzeichnis . "/" . $Element . "'>";
      }
    }
  }

?>

Was stimmt an meinem Code nicht ?

Vielen Dank im voraus.
Gruss, Enrico

  1. Guten Morgen,

    <?php

    $Verzeichnis        = $_GET ["Verzeichnis"];

    $Standardpfad       = "pic/pic/klein";
      $ArrayVerzeichnisse = array ();

    bitte mal überprüfen, ob Du mit der Variablen überhaupt

    arbeitest, sonst, kann das auch raus (problemunabängiger

    Tip)

    if (! isset ($Verzeichnis) || $Verzeichnis == "")
        $Verzeichnis = $Standardpfad;

    echo $Verzeichnis . "<p>";

    $VerzeichnisZeiger = @opendir ($Verzeichnis);

    while ($Element = @readdir ($VerzeichnisZeiger))
      {
        if ($Element != "." && $Element != "..")
        {
          if (is_dir ("$Verzeichnis/$Element"))
            echo "<a href='PHP.GalerieAuswahl.php?Verzeichnis=" . $Verzeichnis . "/" . $Element ."'>" . $Element . "<br>";

    echo "<a href='PHP.GalerieAuswahl.php?Verzeichnis=" . urlencode($Verzeichnis . "/" . $Element )."'>" . $Element . "</a><br>";

    mit is_dir() trennst Du Datein von Verzeichnisen

    else
          {
            if (is_file ($Element))

    if (is_file("$Verzeichnis/$Element"))

    allerdings ist das doch unnötig

    was soll es denn anderen als eine Datei sein ;)

    interessanter abzufragen, wäre doch, ob die

    Datei tatsächlich ein Bild ist ;)

    echo "<img src='" . $Verzeichnis . "/" . $Element . "' alt=''>";

    }
        }
      }

    closedir($VerzeichnisZeiger);

    ?>

    Was stimmt an meinem Code nicht ?

    Passiert mit auch immer ;)

    Gruß aus Berlin!
    eddi

    --
    Manchmal trifft es einen doch ganz unverhofft t86591:
    > '..."Vorläufig abgebrochen" ist ungefähr so sinnvoll formuliert, wie "einstweilig erschossen" oder "temporär verbrannt"...'
    Ich danke Sven für diese Erkenntnis - Gott, was habe ich gelacht ;)
    1. Guten Morgen,

      <?php

      $Verzeichnis        = urldecode($_GET ["Verzeichnis"]);

      oh je; bin noch nicht ganz wach...

      Gruß aus Berlin!
      eddi

      --
      Manchmal trifft es einen doch ganz unverhofft t86591:
      > '..."Vorläufig abgebrochen" ist ungefähr so sinnvoll formuliert, wie "einstweilig erschossen" oder "temporär verbrannt"...'
      Ich danke Sven für diese Erkenntnis - Gott, was habe ich gelacht ;)
      1. Moin!

        $Verzeichnis        = urldecode($_GET ["Verzeichnis"]);

        oh je; bin noch nicht ganz wach...

        Stimmt, denn: Nein, urldecode ist nicht notwendig. Das passiert auf dem Weg von Browser zum Server automatisch! Das ist doch der Witz daran!

        - Sven Rautenberg

        1. Moin Moin!

          $Verzeichnis        = urldecode($_GET ["Verzeichnis"]);

          oh je; bin noch nicht ganz wach...

          Stimmt, denn: Nein, urldecode ist nicht notwendig. Das passiert auf dem Weg von Browser zum Server automatisch! Das ist doch der Witz daran!

          Danke :)

          Gruß aus Berlin!
          eddi

          --
          Manchmal trifft es einen doch ganz unverhofft t86591:
          > '..."Vorläufig abgebrochen" ist ungefähr so sinnvoll formuliert, wie "einstweilig erschossen" oder "temporär verbrannt"...'
          Ich danke Sven für diese Erkenntnis - Gott, was habe ich gelacht ;)