Gästebuch mit $_GET

[Proex]

Hallo Community,

erstmal ein großes Lob an die Seite und den fleißigen Foren-Member ^^

Ich habe folgendes Problem. Ich möchte mein Gästebuch so konstruieren, dass mit if-Anweisungen die URL lesen und die entsprechenden befehle ausführen. Doch ich habe eingie Probleme.

1. $_GET hängt die endung index.php?seite=gaestebuch an die URL
2. Ich lese aus der Datenbank die Ergebnise aus und gebe sie mit der while-Schleife aus.
3. Forumlar wird included.
4. Beim absenden wir index.php?seite=gaestebuch&aktion=eintragen an die URL gesendet.
5. Echo-Ausgabe "Eintrag gespeichert".

Doch das Problem ist, dass die Ausgabe von 5. unter dem Formular angezeigt wird. Ich möchte aber das es allein in der Mitte steht und kein andere Inhalt mehr zu sehen ist.

Wie mach ich das???

Mein Gästebuch: DOWNLOADEN!!!

Mfg
Florian

[vt1816]

Hallo Florian und willkommen hier im Forum.


Bitte poste hier den Code mit den problematischen Stellen. Ein Dateidownload aus unbekannter Quelle ist PFUI!

[cortex]

Zitat von Proex

das Problem ist, dass die Ausgabe von 5. unter dem Formular angezeigt wird. Ich möchte aber das es allein in der Mitte steht und kein andere Inhalt mehr zu sehen ist.

mein denkanstoss geht in diese richtung: eva-prinzip.

cx

[Proex]

Ich benutzte ja das EVA-Prinzip doch ich möchte nicht extra für das Gästebuch noch eine Datei anlegen. Ist es nicht irgendwie möglich das so zu machen?

Code:

PHP-Code:

<?php
//******************************************************************************
//*                     Homepage programmiert von Florian                      *
//*                                                                            *
//*                              gaestebuch.php                                *
//*                                                                            *
//*                        Copyright © 2009 by Florian                         *
//******************************************************************************
include("/include/mysql.php");

$seite = $_GET["seite"];
$aktion = $_GET["aktion"];

mysql_connect($db_host,$db_user,$db_pass) or die ("Keine Verbindung möglich!!!");
mysql_select_db($db) or die ("Die Datenbank gibt es nicht!!!");

if($seite == "gaestebuch")
  {
  $ergebnis = mysql_query("SELECT * FROM gaestebuch ORDER BY id DESC");

  //Ergenis ausgeben
  while($row = mysql_fetch_object($ergebnis))
     {
     echo "<font color='red'><b>Name:</b></font> $row->name<br>";
     echo "<font color='red'><b>E-Mail:</b></font> $row->email<br>";
     echo "<font color='red'><b>Homepage:</b></font> $row->homepage<br>";
     echo nl2br("<font color='red'><b>Nachricht:</b></font>
     <br>$row->nachricht<br><hr>");
     }

  //Gaestebuch.html einbinden
  include("/include/gaestebuch.html");
  }

if($seite == "gaestebuch" AND $aktion == "eintragen")
  {
  $name = $_POST["name"];
  $email = $_POST["email"];
  $homepage = $_POST["homepage"];
  $nachricht = $_POST["nachricht"];

  if($homepage == "Ihre Homepage")
    {
    $homepage = "Keine Angabe";

    if($name == "" OR $email == "" OR $nachricht == "")
      {
      echo "Alle Pflichtfelder ausfüllen!!!<br>";
      echo "<a href='index.php?seite=gaestebuch'>Zurück zum Gästebuch!!!</a>";
      exit;
      }
    }

  //Gästebucheintrag speichern
  $eintragen = mysql_query("INSERT INTO gaestebuch (name, email, homepage, nachricht)
               VALUES ('$name', '$email', '$homepage', '$nachricht')");

  if($eintragen == true)
    {
    echo "Ihr Gästebucheintrag wurde gespeichert!!!<br>";
    echo "<a href='index.php?seite=gaestebuch'>Zurück zum Gästebuch!!!</a>";
    }
  else
    {
    echo "Ihr Gästebucheintrag wurde <u>NICHT</u> gespeichert!!!";
    echo "<a href='index.php?seite=gaestebuch'>Zurück zum Gästebuch!!!</a>";
    }
  }
mysql_close();
?>

[Indyk]

Hallo Proex,

du musst natürlich keine Datei anlegen, wenn du möchtest kannst du deine ganze Homepage in einer datei verwalten. Es ist eine Frage der Übersichtlichkeit und Organisation.

Was für eine Antwort erwartest du von unseren Helfern? "Nein so geht es nicht" oder "Doch klar so geht es auch" ? Wo ist dein konkretes Problem mit dem Code? Was scheitert, kommen Fehlermeldungen oder verhält sich das Script nicht wie gewünscht?

[urvater]

Du legst auf deiner Seite nur fest, was passieren soll, wenn XY erfüllt ist.

Somit ist bei if($seite == "gaestebuch") und if($seite == "gaestebuch" AND $aktion == "eintragen") die Bedingung erfüllt, dass $seite = geastebuch ist. Also wird auch in diesem Fall beides ausgeführt.
Du könntest aber in deinem 1. IF dein zweites IF einbauen.

PHP-Code:

if($seite == "gaestebuch") {
    if($aktion == "eintragen") {
        echo 'Mache das was gemacht werden soll, wenn $aktion = eintragen ist';
    } else {
        echo 'Mache das, was gemacht werden soll, wenn $aktion != eintragen';
    }
} 


[urvater]

Allerdings stellt sich mir die Frage wieso du für Aktion einen Get-Wert einsetzt? Ich meine das Formular muß doch zum Eintragen auch abgesendet werden. Da könnte man doch auch GET weglassen und auf POST prüfen.

und wo wir gerade bei den POST-Daten sind.

Hier solltest du ZWINGEND eine Prüfung VOR dem Speichern in die DB vornehmen.

Google: XSS oder auch SQL Injektion

[Proex]

Ich hatte auch als erstes die if-Verschachtelung genommen doch dann hab ich es so versucht.

Mein Wunsch wäre, das der echo-Befehl "Eintrag wurde gespeichert bzw. NICHT!!!" allein auf der Seite angezeigt wird. Das tut es aber!!! Es steht immer unterm Formular und ich weiß nicht wie ich das weg bekomme.

[urvater]

Zitat von Proex

Ich hatte auch als erstes die if-Verschachtelung genommen doch dann hab ich es so versucht.

Mein Wunsch wäre, das der echo-Befehl "Eintrag wurde gespeichert bzw. NICHT!!!" allein auf der Seite angezeigt wird. Das tut es aber!!! Es steht immer unterm Formular und ich weiß nicht wie ich das weg bekomme.

Indem du dein Skript so änderst, daß

Somit ist bei if($seite == "gaestebuch") und if($seite == "gaestebuch" AND $aktion == "eintragen") die Bedingung erfüllt, dass $seite = geastebuch ist.

nicht mehr gegeben ist.

EDIT: Bzw. das das Formular nur angezeigt wird, wenn $aktion != eintragen ist.

[Proex]

Gut das ist ein Weg aber wie realisiere ich den jetzt. Hab da grade keine klaren Vorstellungen. Das Formular soll ja verschwinden wenn $aktion == "eintragen" ist und nur der Text "Eintrag gespeichert" soll erscheinen sonst nichts ^^