Daten werden nicht gespeichert... @ tutorials.de: Tutorials, Forum & Hilfe

chpa · 03.08.04 17:54

Hi alle zusammen.

Ich habe ein kleines Problem. Ich hab ein Formular und der Inhalt soll in eine Datenbank eingetragen werden.

Das Formular seht Ihre hier: Formular

Und hier das Script, das alles abspeichern soll.

PHP-Code:


<?

  include("dbconnect.php");

  ?>

<HTML>

<HEAD>

<TITLE>Kontrolle</TITLE>

<style type="text/css"><!--

    .rand{

     border:1px white solid;

}

.cssButton{

border-right: #FFFFFF 2px inset;

border-top: #FFFFFF 2px outset;

font-weight:Normal;

font-size: 10pt; 

border-left: #FFFFFF 2px outset;

color: #FFFFFF ;

border-bottom: #FFFFFF 2px inset;

font-family: Arial, Helvetica, sans-serif;

background-color: #035787; 

}

-->

</style>



  </HEAD>

  <BODY bgcolor="#035787" style="scrollbar-face-color:#035787; scrollbar-track-color:#035787; scrollbar-base-color:#035787; scrollbar-arrow-color:white;">



 <FONT face="arial" size="2" color="white"><center>Die Daten werden gesichert.<p>Bitte warten Sie einen Moment. Sie werden automatisch zum Wochenplan weitergeleitet.<p><IMG src=buttons/loader.gif><p>

 <meta http-equiv="refresh" content="3; url=center.php">

 <?

 $table = "$kalenderwoche$tag$nummer";



 $knowdebeka = $HTTP_POST_VARS["knowdebeka"];

 $verein = $HTTP_POST_VARS["verein"];

 $gesetz = $HTTP_POST_VARS["gesetz"];

 $beruf = $HTTP_POST_VARS["beruf"];

 $uberschusse = $HTTP_POST_VARS["uberschusse"];

 $kostenlos = $HTTP_POST_VARS["kostenlos"];

 $knowdebekabe = $HTTP_POST_VARS["knowdebekabe"];

 $vereinbe = $HTTP_POST_VARS["vereinbe"];

 $gesetzbe = $HTTP_POST_VARS["gesetzbe"];

 $berufbe = $HTTP_POST_VARS["berufbe"];

 $uberschussebe = $HTTP_POST_VARS["uberschussebe"];

 $kostenlosbe = $HTTP_POST_VARS["kostenlosbe"];

 $geschlecht = $HTTP_POST_VARS["geschlecht"];

 $name = $HTTP_POST_VARS["name"];

 $adresse = $HTTP_POST_VARS["adresse"];

 $plz = $HTTP_POST_VARS["plz"];

 $ort = $HTTP_POST_VARS["ort"];

 $telenummer = $HTTP_POST_VARS["telenummer"];

 $kalenderwoche = $HTTP_POST_VARS["kalenderwoche"];

 $tag = $HTTP_POST_VARS["tag"];

 $nummer = $HTTP_POST_VARS["nummer"];

 $uhr = $HTTP_POST_VARS["uhr"];

 $wegbe = $HTTP_POST_VARS["wegbe"];



 $eintrag = "INSERT INTO $table (name, adresse, plz, wohnort, telefon, kalenderwoche, tag, nummer, uhr, wegbe, knowdebeka, verein, gesetz, beruf, uberschusse, kostenlos, knowdebekabe, vereinbe, gesetzbe, berufbe, uberschussebe, kostenlosbe, geschlecht) VALUES ('$name', '$adresse', '$plz', '$ort', '$telenummer', '$kalenderwoche', '$tag', '$nummer', '$uhr', '$wegbe', '$knowdebeka', '$verein', '$gesetz', '$beruf', '$uberschusse', '$kostenlos', '$knowdebekabe', '$vereinbe', '$gesetzbe', '$berufbe', '$uberschussebe', '$kostenlosbe', '$geschlecht')";



 $eintragen = mysql_query($eintrag);

 ?>



</BODY>

</HTML>

Was hab ich da falsch gemacht?

Sven Petruschke · 03.08.04 18:05

Mal mit mysql_error() prüfen, ob ein Fehler ausgegeben wird. Wenn ja, die SQL-Anweisung ausgeben lassen und prüfen.

snuu

chpa · 03.08.04 18:16

Hallo nochmal

hab ich vergessen zu schreiben. So eine Error-Anzeige hab ich schon drin. Ich bekomme keine Fehlermeldung.

Sebastianus · 03.08.04 18:45

Hola! Mal zwei Sachen von mir:

1. Es ist zu empfehlen Variablen in SQL-Statements stets mit '".$variable."' einzubinden, da es so schonmal nicht möglich ist SQL-Befehle per Variable zu übergeben und auch die Typen keine Probleme machen.

2. $eintragen = mysql_query($eintrag); dürfte falsch sein
=> versuche es ohne das $eintragen
mysql_query($eintrag); müsste richtig sein

Sven Petruschke · 03.08.04 18:59

Original geschrieben von Sebastianus
Hola! Mal zwei Sachen von mir:

1. Es ist zu empfehlen Variablen in SQL-Statements stets mit '".$variable."' einzubinden, da es so schonmal nicht möglich ist SQL-Befehle per Variable zu übergeben und auch die Typen keine Probleme machen.

2. $eintragen = mysql_query($eintrag); dürfte falsch sein
=> versuche es ohne das $eintragen
mysql_query($eintrag); müsste richtig sein

Ich muss Dir mal wiederprechen.

1.) Wenn Du die Variable in diesem Format einbindest, dann kannst Du immernoch Code einschleusen. Ausserdem ist die generelle Verwendung der Singlequotes um den einzufügenden Wert falsch. Nummerische Werte beispielsweise, werden nicht in Singlequotes eingefasst. Um sich gegen SQL-injections zu schützen, sollte man mysql_escape_string() für Strings verwenden, intval() für Ganzzahlen und doubleval() für Fließkommazahlen.

2.) Die Funktion mysql_query() gibt TRUE bei Erfolg oder FALSE im Fehlerfall zurück. Also ist diese Schreibweise auch korrekt.

snuu

Sebastianus · 03.08.04 19:05

Fein - hab ich auch wieder etwas gelernt! Ein Schüler kann selten besser sein als sein Meister - mir wurde es mal anders gesagt, aber nun werde ich dieses Wissen weiter in die Welt tragen - denn snuu schätze ich kompetenter ein als meinen Meister damals

Was mich jedoch wundert ist der Name der Tabelle! Ist wirklich sichergestellt, dass stest eine solche Tabelle existiert - denn ein Tebellenname mit Kalenderwoche Tag und Nummer klingt in meinen Augen schon sehr dynamisch

Ansonsten sehe ich auch keinen direkten Fehler da drin Außer, dass evtl. noch ein Tippfeld in den Spaltenbezeichnungen der Tabelle ist o.ä.!

chpa · 04.08.04 11:30

Hallo

also die Tabelle wird ausgewählt aus der kalenderwoche in der der Termin stattfindet, aus dem wochentag und aus terminnummer.

chpa · 04.08.04 12:07

Muss ich vielleicht die Verbindung zur Datenbank erst wieder beenden

Kann das sein?

Achso und die MySQL Spalten hab ich alle so eingestellt:

z.B.: Name: "text" "100"

ist das überhaupt richtig?

Ben Ben · 04.08.04 12:16

Was heisst richtig... es ist irgendwo in der Mitte... aber ich würde dennoch soachen wie PLZ eher als INT speichern, Text als Varchar (es sei denn du willst nur ein Zeichen, dann char) sofern dieser nicht mehr als 255 Zeichen haben wird, sonst TEXT.

@snuu: Aber auch beim Schützen gegen Injections ist es schwachsinn, alles vorher in eine extra variable zu Packen. Das kostet a) Speicher (wenn auch nicht viel, aber dennoch eine Verschwendung), b) Schade um die Tipparbeit und c) können mysql_escape_string etc direkt im Statement gemacht werden...

EDIT: @snuu: sry verlesen...

Sven Petruschke · 04.08.04 12:40

@Ben Ben: Da gebe ich Dir vollkommen Recht! Ich habe auch nicht behauptet, dass er jede übergebene Variable nochmals in eine extra Variable verpacken soll. Ich sprach nur davon, dass die generelle Verwendung von Singlequotes falsch ist und dass man die Funktionen mysql_escape_string(), intval() und doubleval() verwenden sollte.

Bitte nochmal nachlesen ...

chpa · 04.08.04 19:45

Hallo,

1. Also es funktioniert jetzt.

Die Datenbank hat nur die Tabellenauswahl mit den 3 Variablen ($kalenderwoche$tag$nummer) nicht angenommen.

Hat jemand vielleicht eine andere Idee, wie ich jetzt durch das Formular die Tabelle auswählen könnte?

Mfg

CHPA

Sven Petruschke · 04.08.04 19:49

Du kannst doch nicht je Kalenderwoche, Tag und Nummer eine neue Tabelle verwenden. Nimm eine einzige Tabelle und füge dieser die Felder Kalenderwoche, Tag und Nummer hinzu. Somit kannst Du später über die WHERE-Bedingung im SQL-Statement die Daten einschränken.

snuu

tut orials.de	tut verbinden	tut dir helfen	tut kooperieren
©2000-2012 tutorials.de Alle Rechte vorbehalten	tut informieren » Impressum » Nutzungsregeln » Netiquette » Team » Redaktion	» Tutorials » Videotutorials » Showroom » Events & Contests » Chat » RSS-Feed » Blogs	» der-webdesigner.net » DesignerInAction » Design Nation » Directorforum » Flashhilfe » PHP-Resource » PHPwelt » PSD-Tutorials

Themen-Optionen

Anzeige

Ähnliche Themen

Warum werden die Daten nicht korrekt in der MySql-Db gespeichert? [UTF-8]

Typo3 Einsteiger. Wo werden Daten gespeichert ?

Daten werden nicht gespeichert

Daten werden in Array nicht richtig gespeichert

Daten werden nicht in der Datenbank gespeichert

[PHP][Snippet] Array zu XML konvertieren

PHP: LDAP / Active Directory abfragen

E-Mail:

Abonnieren Abbestellen