Hilfe!? ;)

Masteroi · 03.06.2005, 17:43

Hallo erstmal... ich bin noch blutiger anfänger in php!
Bin gerade dabei ein kleines Gbook zu schreiben, und dachte ihr könnt mir vielleicht helfen:

PHP-Code:

  <?

 
@mysql_connect("$db_server","$db_user","$db_pw") or die

("Keine Verbindung moeglich");

mysql_select_db("$db_datenbank") or die

("Die Datenbank existiert nicht");

 
 
$result = mysql_query ("select Name from MyGuestbook") or die ("MySql Error!"); 

while ($row = mysql_fetch_array ($result)) { 

 
$datum=date("l dS of F Y h:i:s A");

 
 
switch ($action) {

    case "eintrag":

    echo'<form action="gaestebuch.php?action=eintragen" method="post"><input name="name">
<input name="email">
<input name="homepage">
<input name="text">
<input type="submit" value="Eintragen">';

    break;

    

    case "eintragen":

    $eintrag = "INSERT INTO MyGuestbook (Name, Text, EMail, Homepage, Datum) VALUES ('$name', '$text', '$email', '$homepage', '$datum')";

    $eintragen = mysql_query($eintrag);

    echo "
Der Eintrag wurde erfolgreich übernommen!
";

    echo '[url="gaestebuch.php"]Gaestebuch erneut anschauen[/url]';

    break;

 
    default:

?>

    [url="gaestebuch.php?action=eintrag"]Eintragen[/url]

 
    <table border="1" cellpadding="0" cellspacing="0" width="90%">

        

        <tr> 

        <td align="right"><? echo $row["Name"]; ?></td> 

        </tr>

 
    </table>

<?

}

}

?>

Wieso kommt bei diesem Code alles 3-fach?! Und wie kann ich dies ändern?
Hab das alles auf meinem ftp wer sich das mal anschauen will

http://philippdrehwitz.ph.funpic.de/...gaestebuch.php

Danke schonmal im Voraus!
MfG Philipp

mkl0815 · 03.06.2005, 17:50

weil du erst die datenbank abfragst und dann in der schleife in der du die einzelnen einträge ausgibst auch das formular mit ausgibst.

sortier am besten deinen code.

1. den link "eintragen" anzeigen

2. prüfen ob formulardaten übergeben wurden

2.1 wenn ja, dann daten eintragen
2.2 wenn nicht, dann weiter

3. db-einträge abfragen

4. einträge in einer schleife ausgeben

------------------------------------

register_globals ist on !!!! -> ausschalten, bzw. nicht verwenden
also $_POST['wasauchimmer'] und $_GET['wasauchsonst']

weitere fehlerquelle: eingegebene daten escapen, da sonst sql-injection möglich.

mario.

03.06.2005, 17:50

was machst Du da?!?

mkl0815 · 03.06.2005, 17:51

besser noch :

1. prüfen ob formulardaten übergeben wurden

1.1 wenn ja, dann daten eintragen
1.2 wenn nicht, dann den link "eintragen" anzeigen

3. db-einträge abfragen

4. einträge in einer schleife ausgeben

Masteroi · 03.06.2005, 17:54

Ich sag ja... blutiger anfänger

was meinst du mit daten escapen?!

imported_Ben · 03.06.2005, 17:54

Hi,
vielleicht schaust Du Dich hier mal um und arbeitest die Tutorials mal durch

Gästebuch auf Textdateien basierend
Dazu:

Textdateien: Lesen und Schreiben von Inhalten

MySQL-DB-basiertes Gästebuch

Bonusmaterial

Eingabeüberprüfung eines Formulars

Masteroi · 03.06.2005, 17:56

Danke ich denke dass hält mich erstmal ne woche fit

Gruß Philipp

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
Datenbanktabelle erstellen - brauche Hilfe	Raccoon	PHP Tipps 2008	1	14.06.2008 11:03
[Erledigt] Hilfe: Projektabreit Informatik und Technik bitte helft uns.		Off-Topic Diskussionen	31	05.04.2006 18:25
PHP <--> MySQL \| Brauche Hilfe	ahnungsloser	Datenbanken	18	16.02.2006 21:49
[Erledigt] HILFE HILFE HILFE mysqladmin.exe fehlt		Datenbanken	7	20.12.2005 14:00
[Erledigt] Hilfe mit encoded php-Files		PHP-Fortgeschrittene	9	02.11.2005 16:16
HILFE ! Brauche ganz dringend hilfe!		PHP Tipps 2005-2	2	10.10.2005 23:08
Script, welches PDF mit Hilfe von PDFlib generiert		Beitragsarchiv	3	02.09.2005 15:49
[Erledigt] Probleme mit Linkverfolgung !!! HILFE !!!		PHP-Fortgeschrittene	5	20.07.2005 18:24
[Erledigt] VARIABLEN WERDEN NICHT RICHTIG VERARBEITET!!!! HILFE		PHP Tipps 2005-2	6	02.07.2005 18:32
[Erledigt] Hilfe! If Tags werden Ignoriert		PHP Tipps 2005	10	16.05.2005 10:16
php Neuling braucht mal schnell hilfe !!		PHP Tipps 2005	5	05.03.2005 13:30
[Erledigt] HILFE		PHP Tipps 2004-2	1	16.12.2004 20:49
Hilfe!!! PHP + EXE-Dateien und Interface		Beitragsarchiv	4	16.12.2004 11:50
[Erledigt] Chat Server mit Datenbank realisieren - - - Hilfe benötigt		PHP Tipps 2004-2	3	14.12.2004 18:38
Suche -dringend- Hilfe zu einem Smily Mod! <- HiLFE - :-)		PHP Tipps 2004-2	0	07.11.2004 13:50

03.06.2005, 17:43
Masteroi Erfahrener Benutzer Registriert seit: 03.06.2005 Beiträge: 151	Hilfe!? ;) Hallo erstmal... ich bin noch blutiger anfänger in php! Bin gerade dabei ein kleines Gbook zu schreiben, und dachte ihr könnt mir vielleicht helfen: PHP-Code: <? @mysql_connect("$db_server","$db_user","$db_pw") or die ("Keine Verbindung moeglich"); mysql_select_db("$db_datenbank") or die ("Die Datenbank existiert nicht"); $result = mysql_query ("select Name from MyGuestbook") or die ("MySql Error!"); while ($row = mysql_fetch_array ($result)) { $datum=date("l dS of F Y h:i:s A"); switch ($action) { case "eintrag": echo'<form action="gaestebuch.php?action=eintragen" method="post"><input name="name"> <input name="email"> <input name="homepage"> <input name="text"> <input type="submit" value="Eintragen">'; break; case "eintragen": $eintrag = "INSERT INTO MyGuestbook (Name, Text, EMail, Homepage, Datum) VALUES ('$name', '$text', '$email', '$homepage', '$datum')"; $eintragen = mysql_query($eintrag); echo " Der Eintrag wurde erfolgreich übernommen! "; echo '[url="gaestebuch.php"]Gaestebuch erneut anschauen[/url]'; break; default: ?> [url="gaestebuch.php?action=eintrag"]Eintragen[/url] <table border="1" cellpadding="0" cellspacing="0" width="90%"> <tr> <td align="right"><? echo $row["Name"]; ?></td> </tr> </table> <? } } ?> Wieso kommt bei diesem Code alles 3-fach?! Und wie kann ich dies ändern? Hab das alles auf meinem ftp wer sich das mal anschauen will http://philippdrehwitz.ph.funpic.de/...gaestebuch.php Danke schonmal im Voraus! MfG Philipp


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

03.06.2005, 17:50
mkl0815 Erfahrener Benutzer Registriert seit: 10.09.2004 Beiträge: 339	weil du erst die datenbank abfragst und dann in der schleife in der du die einzelnen einträge ausgibst auch das formular mit ausgibst. sortier am besten deinen code. 1. den link "eintragen" anzeigen 2. prüfen ob formulardaten übergeben wurden 2.1 wenn ja, dann daten eintragen 2.2 wenn nicht, dann weiter 3. db-einträge abfragen 4. einträge in einer schleife ausgeben ------------------------------------ register_globals ist on !!!! -> ausschalten, bzw. nicht verwenden also $_POST['wasauchimmer'] und $_GET['wasauchsonst'] weitere fehlerquelle: eingegebene daten escapen, da sonst sql-injection möglich. mario. __________________ Problems with Windows : REBOOT Problems with Linux : BE ROOT

03.06.2005, 17:51
mkl0815 Erfahrener Benutzer Registriert seit: 10.09.2004 Beiträge: 339	besser noch : 1. prüfen ob formulardaten übergeben wurden 1.1 wenn ja, dann daten eintragen 1.2 wenn nicht, dann den link "eintragen" anzeigen 3. db-einträge abfragen 4. einträge in einer schleife ausgeben __________________ Problems with Windows : REBOOT Problems with Linux : BE ROOT

03.06.2005, 17:54
Masteroi Erfahrener Benutzer Registriert seit: 03.06.2005 Beiträge: 151	Ich sag ja... blutiger anfänger was meinst du mit daten escapen?!

03.06.2005, 17:54
imported_Ben Erfahrener Benutzer Registriert seit: 18.09.2003 Beiträge: 13.598 PHP-Kenntnisse: Fortgeschritten	Hi, vielleicht schaust Du Dich hier mal um und arbeitest die Tutorials mal durch Gästebuch auf Textdateien basierend Dazu: Textdateien: Lesen und Schreiben von Inhalten MySQL-DB-basiertes Gästebuch Bonusmaterial Eingabeüberprüfung eines Formulars __________________ Webentwickler-Blog - Buchbesprechung PHP Design Patterns - SQL-Antipatterns

03.06.2005, 17:56
Masteroi Erfahrener Benutzer Registriert seit: 03.06.2005 Beiträge: 151	Danke ich denke dass hält mich erstmal ne woche fit Gruß Philipp __________________ Nein ich bin nicht die Signatur... ich putz hier nur!