Script trägt daten nicht in db ein

Ryckmaster · 9. Dezember 2008, 09:07

Habe nen code geschrieben der auch funzt.
abgesehen von der function daten_schreiben($p_verbindungskennung).
Diese soll alles in die DB eintragen was sie auch macht, abgesehen von der ausgewählten beschreibung.
Ich poste mal den kompletten code.

PHP-Quellcode

<?php
//überprüfen ob Administrator angemeldet ist
require "./session.inc.php";
?>
<html>
<head>
<title>Administrationsbereich</title>
</head>
<body><a href=logout.php>Logout</a>
<?php
function datenbank_verbinden()
{
$db_server="localhost";
$db_name="smuv";
$db_user="root";
$db_passwort="";
$db_verbindung=mysql_connect($db_server,$db_user,$db_passwort);
if( !$db_verbindung ) {
die("Der Server kann nicht erreicht werden.");
}
if( !mysql_select_db($db_name,$db_verbindung) ) {
die("Die Datenbank kann nicht angesprochen werden.");
}
return $db_verbindung;
}
function gib_beschreibung(&$p_beschreibungsnummer,&$p_beschreibung,$p_verbindungskennung)
{
$query="select beschreibungsnummer,beschreibung
from beschreibungen order by beschreibung";
$ergebnis=mysql_query($query,$p_verbindungskennung);
$i=0;
$p_beschreibungsnummer=array();
$p_beschreibung=array();
while($zeile=mysql_fetch_array($ergebnis))
{
$p_beschreibungsnummer[$i]=$zeile['beschreibungsnummer'];
$p_beschreibung[$i]=$zeile['beschreibung'];
$i++;
}
}
function formular_aufbauen($p_beschreibungsnummer,$p_beschreibung)
{
?>
<form action="<?$_SERVER['php_self']?>" method="get">
<table>
<tr><th>Schulungsanfang:</th> <th><input type="text" name="es"></th></tr>
<tr><th>Schulungsende:</th> <th><input type="text" name="se"></th></tr>
<tr><th>Plätze:</th> <th> <input type="text" name="p"></th></tr>
</table>
Wählen Sie bis eine beschreibung:
<?php
for($i=0;$i<count($p_beschreibungsnummer);$i++)
{?><input type="radio" name="beschreibung[]"
value="<?php echo $p_beschreibungsnummer[$i]?>"><?php
echo $p_beschreibung[$i]." ";
}
?>
<input type="submit" name="ausfuehren" value="Speichern">
</form>
<?php
}
//überprüfung der Eingaben
function check_eintraege()
{
if(empty($_GET['es']))
{
echo "Fehler! Sie haben keinen Anfangszeitraum festegelegt. ";
return false;
}
if(empty($_GET['se']))
{
echo"Fehler! Sie haben kein Schulungsende angegeben. ";
return false;
}
if(!isset($_GET['beschreibung']))
{
echo"Fehler! Sie haben keine Beschreibung gewählt. ";
return false;
}
if(empty($_GET['p']))
{
echo"Fehler! Sie haben nicht angegeben wieviel Plätze vorhanden sind. ";
return false;
}
return true;
}
function daten_schreiben($p_verbindungskennung)
{
//zugeordnete beschreibung abspeichern
$beschreibungsarray=array();
$beschreibungsarray=$_GET['beschreibung2'];
$i=0;
{
$query=" INSERT INTO
schulung (ErsterSchulungstag, Schulungsende, Beschreibung, Plaetze)
VALUES
('".$_GET['es']."','".$_GET['se']."','".$_GET['beschreibung2']."','".$_GET['p']."')";
$ergebnis=mysql_query($query,$p_verbindungskennung);
$i++;
}
echo "Die Daten wurden gespeichert. ";
mysql_close($p_verbindungskennung);
}
function daten_merken()
{
$verbindungskennung=datenbank_verbinden();
if(check_eintraege($verbindungskennung)
&&(check_p()))
{
daten_schreiben($verbindungskennung);
}
else
{
echo "Ihre Daten wurden nicht gespeichert! ";
echo "Bitte versuchen sie es nochmal!! ";
}
}
//eingabe von Schulungsdauer prüfen
function check_sd()
{
if (isset($_GET['sd']))
{
if (is_numeric($_GET['sd']))
{
return true;
}
else
{
echo "Bitte nur Zahlen als Schulungsdauer eingeben!";
return false;
}
}
}
function check_p()
{
if (isset($_GET['p'])) {
if (is_numeric($_GET['p'])) {
return true;
} else {
echo "Bitte geben sie die Plätze in Zahlen an!";
return false;
}
}
}
function formular()
{
$verbindungskennung=datenbank_verbinden();
gib_beschreibung($beschreibungsnummer,$beschreibung,$verbindungskennung);
formular_aufbauen($beschreibungsnummer,$beschreibung);
}
//Von hier werden die Funktionen formular() und
// daten_merken() aufgerufen
if (isset($_GET['ausfuehren']))
daten_merken();
formular();
?>
</body>
</html>

Alles anzeigen

was mach ich falsch, ich steh auf den schlauch

lg

Broken Sword · 9. Dezember 2008, 09:32

Du hast aber ganz schön viel Vertrauen in deine Seitenbesucher, wenn du MySQL-Injections zulässt. Immer schön mit addslahes() absichern

@topic:

Quellcode

{[b]//<<<hier[/b]
$query=" INSERT INTO
schulung (ErsterSchulungstag, Schulungsende, Beschreibung, Plaetze)
VALUES
('".$_GET['es']."','".$_GET['se']."','".$_GET['beschreibung2']."','".$_GET['p']."')";
$ergebnis=mysql_query($query,$p_verbindungskennung);
$i++;
}

Ich würde sagen, dort fehlt etwas VOR der Klammer. Vermute mal, dort wurde eine Schleife erdacht, was dann auch dem $i++ einen Sinn geben würde.
Wenn du das bewusst weggemacht hast, um hier zu posten kann es nur noch ein SQL-Fehler sein, der aber bei dir nicht behandelt wird. Seh dir mal diese Funktion an: PHP: mysql_error - Manual
Normal benutzt man die Funktion mit OR (OR kann man bei Funktionen, die etwas zurückgeben können verwenden, um etwas bei false ausführen zu lassen.) und die.

Quellcode

mysql_query($sql) OR die('MySQL-Error#'.mysql_errno().': '.mysql_error());

Gruß
Broken Sword

Ryckmaster · 9. Dezember 2008, 10:17

also mysql error gibt nichts aus.
er trägt j soeweit auch alles ein, bis auf die beschreibung.
was meinst du denn mit der schleife, ich glaube das muss so, bin aber auch kein pro

wie denkst dir das denn, bzw. wie sollte das aussehen?

lg

Broken Sword · 9. Dezember 2008, 10:21

Warum sollte das so müssen?
Erzähl doch mal, warum du die Klammern dahin gesetzt hast? Vor allem, was es mit dem $i = 0 und $i++ aufsich hat.

Gruß
Broken Sword

Ryckmaster · 9. Dezember 2008, 10:53

mit $i++ soll er zählen und $i=0 sagt das er ab 0 beginnen soll.
die klammern habe ich gesetzt da ich dachte, ohne geht es nicht, da lag ich falsch =)

lg

raveshow · 9. Dezember 2008, 11:04

wenn ich das richtig erkannt habe ist die mysql abfrage einfach falsch... du hast nur "order by beschreibung" muss man ihm dabei aber nicht noch DESC bzw ASC angeben ?

PHP-Quellcode

$query="SELECT beschreibungsnummer, beschreibung FROM beschreibungen ORDER BY beschreibung DESC";

oder

PHP-Quellcode

$query="SELECT beschreibungsnummer, beschreibung FROM beschreibungen ORDER BY beschreibung ASC";

ansonsten hab ich bis jetzt nichts gefunden beim rüberschauen

mfg

Broken Sword · 9. Dezember 2008, 11:05

Aber warum setzt du denn nicht gleich $i auf 1? $i wird doch immer wieder auf 0 gesetzt und dann einmal um 1 aufgestockt. Wobei du gar nicht auf $i zugreifen kannst. Das ist nämlich keine globale Variable. Auch irritiert mich, dass du gleich danach die Verbindung zur Datenbank trennst, in der Funktion sie aber nicht erstellst. Sprich, du kannst die Funktion nur einmal benutzen, was dann wieder gegen eine Funktion sprechen würde.

Gruß
Broken Sword

Ryckmaster · 9. Dezember 2008, 14:12

also desc oder so brauch ich nicht eingeben. er sortiert die daten gleich alphabetisch an.
Ich habe den code abgeleitet von einem anderen der funktioniert.
ich kann den ja mal posten, vielleicht versteh ich das dort verwendete $i ja falsch.

PHP-Quellcode

<html>
<head><title>Schwarzes Webbrett</title></head>
<body>
<?php
error_reporting (E_ALL | E_STRICT);
ini_set ('display_errors' , 1);
/**
*baut eine verbindung zur datenbank auf und gibt die
*verbindung an die aufgerufene verbindung zurück
*/
function datenbank_verbinden()
{
$db_server="localhost";
$db_name="webbrett";
$db_user="root";
$db_passwort="";
$db_verbindung=mysql_connect($db_server,$db_user,$db_passwort);
if(!$db_verbindung)
die("Der Server kann nicht erreicht werden.");
if(!mysql_select_db($db_name,$db_verbindung))
die("Die Datenbak kann nicht angesprochen werden.");
return $db_verbindung;
}
function gib_rubriken(&$p_rubriknummer,&$p_rubrikbezeichnung,$p_verbindungskennung)
{
$query="select rubriknummer,rubrikbezeichnung
from rubrik order by rubrikbezeichnung";
$ergebnis=mysql_query($query,$p_verbindungskennung);
$i=0;
$p_rubriknummer=array();
$p_rubrikbezeichnung=array();
while($zeile=mysql_fetch_array($ergebnis))
{
$p_rubriknummer[$i]=$zeile['rubriknummer'];
$p_rubrikbezeichnung[$i]=$zeile['rubrikbezeichnung'];
$i++;
}
}
function formular_aufbauen($p_rubriknummer,$p_rubrikbezeichnung)
{
?>
<form action="<?$_SERVER['php_self']?>" method="get">
Nickame: <input type="text" name="nickname">
E-Mail: <input type="text" name="email">
Anzeigentext: <input type="text" name="anzeigentext" size=40 maxleght=255>
Wählen Sie bis zu drei Rubriken:
<input type="submit" name="ausfuehren" value="Registrieren">
</form>
<?php
}
function check_nickname($p_verbindungskennung)
{
$query=sprintf("select nickname from inserent
where nickname='%s'
and email!='%s'",
$_GET['nickname'],
$_GET['email']);
$ergebnis=mysql_query($query,$p_verbindungskennung);
if(mysql_num_rows($ergebnis)!=0)
{
echo "Fehler! Der Name ist leider schon vergeben.
Waehlen Sie bitte einen anderen Namen oder
die passende E-Mail-Adresse. ";
return false;
}
return true;
}
function check_email($p_verbindungskennung)
{$query=sprintf("select email from inserent
where nickname!='%s'
and email='%s'",
$_GET['nickname'],
$_GET['email']);
$ergebnis=mysql_query($query,$p_verbindungskennung);
if(mysql_num_rows($ergebnis)!=0)
{
echo "Fehler! Die E-Mail Adresse ist leider schon vergeben.
Waehlen Sie bitte eine andere oder
die passenden Namen. ";
return false;
}
return true;}
function check_eintraege()
{
if(empty($_GET['nickname']))
{
echo "Fehler! Sie haben keinen Namen angegeben. ";
return false;
}
if(empty($_GET['email']))
{
echo"Fehler! Sie haben keine E-Mail-Adresse angegeben. ";
return false;
}
if(empty($_GET['anzeigentext']))
{
echo"Fehler! Sie haben keinen Anzeigentext angegeben. ";
return false;
}
if(!isset($_GET['rubrik']))
{
echo"Fehler! Sie haben keine Rubrik ausgewählt. ";
return false;
}
if(count($_GET['rubrik'])>3)
{
echo"Fehler! Sie können nur maximal 3 Rubriken wählen. ";
return false;
}
return true;
}
function schreib_inserat($p_verbindungskennung)
{
$query=sprintf("select inserentennummer from inserent
where nickname='%s'
and email='%s'",
$_GET['nickname'],
$_GET['email']);
$ergebnis=mysql_query($query,$p_verbindungskennung);
if(mysql_num_rows($ergebnis)!=0)
{
//Inserent ist schon in der Datenbank, Inserentennummer merken
$zeile=mysql_fetch_array($ergebnis);
$inserentennummer=$zeile['inserentennummer'];
}
else
{
//neuen Inserenten anlegen und inserentennummer merken
$query=sprintf("insert into inserent
set nickname='%s',
email='%s'",
$_GET['nickname'],
$_GET['email']);
$ergebnis=mysql_query($query,$p_verbindungskennung);
$inserentennummer=mysql_insert_id();
}
//Anzeigentext abspeichern und anzeigennummer merken
$query=sprintf("insert into anzeige
set anzeigentext='%s',
anzeigendatum=curdate(),
inserentennummer=$inserentennummer",
$_GET['anzeigentext']);
$ergebnis=mysql_query($query,$p_verbindungskennung);
$anzeigennummer=mysql_insert_id();
//zugeordnete Rubriken abspeichern
$rubrikarray=array();
$rubrikarray=$_GET['rubrik'];
$i=0;
while($i<count($rubrikarray))
{
$query="insert into veroeffentlichen
set rubriknummer=$rubrikarray[$i],
anzeigennummer=$anzeigennummer";
$ergebnis=mysql_query($query,$p_verbindungskennung);
$i++;
}
echo "Ihre Anzeige wurde entgegengenommen. ";
mysql_close($p_verbindungskennung);
}
function anzeige_eintragen()
{
$verbindungskennung=datenbank_verbinden();
if(check_nickname($verbindungskennung)
&&check_email($verbindungskennung)
&&check_eintraege())
schreib_inserat($verbindungskennung);
}
function formular()
{
$verbindungskennung=datenbank_verbinden();
gib_rubriken($rubriknummer,$rubrikbezeichnung,$verbindungskennung);
formular_aufbauen($rubriknummer,$rubrikbezeichnung);
}
//Beginn
//Von hier werden die Funktionen anzeige_eintragen() und
//formular() aufgerufen
echo "Anzeige Aufgeben ";
echo "<a href=\"./index.php\">Zurueck</a> ";
if (isset($_GET['ausfuehren']))
anzeige_eintragen();
formular();
?>
<a href="index.php">Zurück</a>
</body>
</html>

Alles anzeigen

Da dieser Code fast das gleiche macht wie den den ich jetzt neu schreibe, habe ich mir gedacht das ich nur die Varibeln umschreiben muss und alles überflüssige raus...
hmm scheint nicht zu klappen.
Seht ihr was?

lg

hab den code jetzt so geändert, das er die beschreibungsnummer eingibt.
wenn er jetzt die Beschreibungen eintragen würde, wäre mir lieber.

PHP-Quellcode

[code=php]
<?php
//überprüfen ob Administrator angemeldet ist
require "./session.inc.php";
?>
<html>
<head>
<title>Administrationsbereich</title>
</head>
<body><a href=logout.php>Logout</a>
<?php
error_reporting(E_ALL);
function datenbank_verbinden()
{
$db_server="localhost";
$db_name="smuv";
$db_user="root";
$db_passwort="";
$db_verbindung=mysql_connect($db_server,$db_user,$db_passwort);
if( !$db_verbindung ) {
die("Der Server kann nicht erreicht werden.");
}
if( !mysql_select_db($db_name,$db_verbindung) ) {
die("Die Datenbank kann nicht angesprochen werden.");
}
return $db_verbindung;
}
function gib_beschreibung(&$p_beschreibungsnummer,&$p_beschreibung,$p_verbindungskennung)
{
$query="select beschreibungsnummer,beschreibung
from beschreibungen order by beschreibung";
$ergebnis=mysql_query($query,$p_verbindungskennung);
$i=0;
$p_beschreibungsnummer=array();
$p_beschreibung=array();
while($zeile=mysql_fetch_array($ergebnis))
{
$p_beschreibungsnummer[$i]=$zeile['beschreibungsnummer'];
$p_beschreibung[$i]=$zeile['beschreibung'];
$i++;
}
}
function formular_aufbauen($p_beschreibungsnummer,$p_beschreibung)
{
?>
<form action="<?php $_SERVER['PHP_SELF'] ?>" method="get">
<table>
<tr><th>Schulungsanfang:</th> <th><input type="text" name="es"></th></tr>
<tr><th>Schulungsende:</th> <th><input type="text" name="se"></th></tr>
<tr><th>Plätze:</th> <th> <input type="text" name="p"></th></tr>
</table>
Wählen Sie bis eine beschreibung:
<?php
for($i=0;$i<count($p_beschreibungsnummer);$i++)
{?><input type="radio" name="beschreibung[]"
value="<?php echo $p_beschreibungsnummer[$i]?>"><?php
echo $p_beschreibung[$i]." ";
}
?>
<input type="submit" name="ausfuehren" value="Speichern">
</form>
<?php
}
//überprüfung der Eingaben
function check_eintraege()
{
if(empty($_GET['es']))
{
echo "Fehler! Sie haben keinen Anfangszeitraum festegelegt. ";
return false;
}
if(empty($_GET['se']))
{
echo"Fehler! Sie haben kein Schulungsende angegeben. ";
return false;
}
if(!isset($_GET['beschreibung']))
{
echo"Fehler! Sie haben keine Beschreibung gewählt. ";
return false;
}
if(empty($_GET['p']))
{
echo"Fehler! Sie haben nicht angegeben wieviel Plätze vorhanden sind. ";
return false;
}
return true;
}
function daten_schreiben($p_verbindungskennung)
{
//zugeordnete beschreibung abspeichern
$beschreibungsarray=array();
$beschreibungsarray=$_GET['beschreibung'];
$i=0;
while($i<count($beschreibungsarray))
{
$query="insert into schulung
set Beschreibung='$beschreibungsarray[$i]',
ErsterSchulungstag='$_GET[es]',
Schulungsende='$_GET[se]',
Plaetze='$_GET[p]';";
$ergebnis=mysql_query($query,$p_verbindungskennung) OR die('MySQL-Error#'.mysql_errno().': '.mysql_error());
$i++;
}
echo "Die Daten wurden gespeichert. ";
echo "ausgegebene Variabeln ";
echo "query=$query ";
echo "Beschreibungsarray=$beschreibungsarray ";
echo "Ergebnis=$ergebnis ";
}
function daten_merken()
{
$verbindungskennung=datenbank_verbinden();
if(check_eintraege($verbindungskennung)
&&(check_p()))
{
daten_schreiben($verbindungskennung);
}
else
{
echo "Ihre Daten wurden nicht gespeichert! ";
echo "Bitte versuchen sie es nochmal!! ";
}
}
//eingabe von Schulungsdauer prüfen
function check_sd()
{
if (isset($_GET['sd']))
{
if (is_numeric($_GET['sd']))
{
return true;
}
else
{
echo "Bitte nur Zahlen als Schulungsdauer eingeben!";
return false;
}
}
}
function check_p()
{
if (isset($_GET['p'])) {
if (is_numeric($_GET['p'])) {
return true;
} else {
echo "Bitte geben sie die Plätze in Zahlen an!";
return false;
}
}
}
function formular()
{
$verbindungskennung=datenbank_verbinden();
gib_beschreibung($beschreibungsnummer,$beschreibung,$verbindungskennung);
formular_aufbauen($beschreibungsnummer,$beschreibung);
}
//Von hier werden die Funktionen formular() und
// daten_merken() aufgerufen
if (isset($_GET['ausfuehren']))
daten_merken();
formular();
?>
</body>
</html>

Alles anzeigen

wer sagt mir wie es geht?
ich fummel bis dahin weiter.

lg

Broken Sword · 9. Dezember 2008, 14:52

Ich gebe dir mal einen Tipp. Bring erstmal das original-Script zum Laufen und erst dann nimmst du Stück für Stück etwas weg, was du nicht brauchst. So weißt du gleich, was wo fehlt

Gruß
Broken Sword

P.S.
Im Original hat $i eine Bedeutung und es gibt eine while Schleife vor der Klammer (das nur zur Selbstbestätigung für mich :D)

Ryckmaster · 9. Dezember 2008, 14:58

guck hin, es geht auch ohne, nur der sql befehl war falsch..
sebstbestätigung für mich

lg

Broken Sword · 9. Dezember 2008, 15:02

Ryckmaster schrieb:

guck hin, es geht auch ohne, nur der sql befehl war falsch..

Nun lass die, die dir mühsam geholfen haben nicht dumm sterben. Ein zwei Sätze dürften doch nicht zu viel verlangt sein -.-

Wohin?

Ohne was denn?

Wenn der SQL-Befehl fehlerhaft war, dann hättest du das mit mysql_error eigentlich herausfinden müssen, was war denn falsch?

Gruß
Broken Sword

P.S.
Dafür, dass du den Code nicht verstehst, bist du dir ziemlich selbstsicher.

Ryckmaster · 9. Dezember 2008, 15:18

{
$query=" INSERT INTO
schulung (ErsterSchulungstag, Schulungsende, Beschreibung, Plaetze)
VALUES
('".$_GET['es']."','".$_GET['se']."','".$_GET['beschreibung2']."','".$_GET['p']."')";
$ergebnis=mysql_query($query,$p_verbindungskennung);
$i++;
}

das war ja der alte...
der richtige muss so aussehen:

$query="insert into schulung
set Beschreibung='$beschreibungsarray[$i]',
ErsterSchulungstag='$_GET[es]',
Schulungsende='$_GET[se]',
Plaetze='$_GET[p]';";

so also wie gehsagt soweit geht es. aber er soll ja die Beschreibungen eintragen und nicht dessen nummer....
das sollte ja eingentlich mit $p_beschreibung[$i] ausgebne aber wenn ich diese eingebe trägt er wieder nichts ein..

ne idee?

PHP-Quellcode

Quellcode

Quellcode

PHP-Quellcode

PHP-Quellcode

PHP-Quellcode

PHP-Quellcode

Teilen