PHP Fehler

DerSaboteur · 11/27/2011, 14:14

Hey

Ich finde meinen eigenen Fehler hier nicht... Ich bin wirklich blind heute ô.ô
Hoffe einer von euch kann mir helfen?

PHP Code:


			
<?php



$verbindung = mysql_connect("localhost", "Benutzername", "Passwort")

or die ("Fehler beim Einloggen in MySql");



mysql_select_db("DatenbankName")

or die ("Verbindung zur Datenbank ist nicht erfolgreich..");



$Vorname = $_POST["Vorname"];

$Name = $_POST["Name"];

$SName = $_POST["SName"];

$Homepage = $_POST["Homepage"];

$Info = $_POST["Info"];

$BTeam = $_POST["BTeam"];

$BLagg = $_POST["BLagg"];

$BHomepage = $_POST["BHomepage"];



if($Vorname == "" or $Name == "" or $SName == "" or $Homepage == "" or $Info == "" or $BTeam == "" or $BLagg == "" or $BHomepage == "")

{

    echo "Bitte fuelle die Felder aus!";

}

else

{

    $eintrag = "INSERT INTO bewertung

    (Vorname, Name, SName, Homepage, Info, BTeam, BLagg, BHomepage)



    VALUES

    ('$Vorname', '$Name', '$SName', '$Homepage', '$Info', '$BTeam', '$BLagg', '$BHomapage')";



    $eintragen = mysql_query($eintrag);

    if ($eintragen == true)

    {

        echo "Deine Bewertung wurde eingetragen!"

    }

    else

    {

        echo "Fehler im System. Konnte nicht gespeichert werden..."

    }

}



mysql_close($verbindung);



?>

ich bekomme immer diese Fehlermeldung..

PHP Code:


			
Parse error: syntax error, unexpected '}', expecting ',' or ';' in /var/www/web127218/html/Bewertung/Bewertung.php on line 34

Ich kann mein kopf gegen die Wand klatschen wenn es ein einfacher Fehler ist.....

Mfg DerSaboteur

Fratyr · 11/27/2011, 14:55

Wie wärs wenn du hinter deine echos mal ; packst?

~~.exTremez™~~ · 11/27/2011, 14:56

Bei Zeile 33 fehlt am ende ein ;

Code:

echo "Deine Bewertung wurde eingetragen!"[COLOR="Red"][B][U];[/U][/B][/COLOR]

Bei Zeile 37 das gleiche

Code:

echo "Fehler im System. Konnte nicht gespeichert werden..."[B][COLOR="Red"][U];[/U][/COLOR][/B]

EDIT://

Zeile 28(Du hast BHomapage anstatt BHomepage geschrieben ^^):

Code:

    ('$Vorname', '$Name', '$SName', '$Homepage', '$Info', '$BTeam', '$BLagg', '[B][COLOR="Red"][U]$BHomepage[/U][/COLOR][/B]')";

Whoknowsit · 11/27/2011, 15:01

Hinter deinen echo-Anweisungen fehlt das Semikolon

Ich würde es übrigens in etwa so machen:

PHP Code:


			
<?php 
$verbindung = mysql_connect("localhost", "Benutzername", "Passwort") or die ("Fehler beim Einloggen in MySql"); 
mysql_select_db("DatenbankName") or die ("Verbindung zur Datenbank ist nicht erfolgreich.."); 

function cleanUp($data) {
    $data = stripslashes($data);
    $data = strip_tags($data);
    $data = mysql_real_escape_string($data);
    return $data;
}

$_POST = array_map('cleanUp', $_POST);

$Vorname = $_POST['Vorname']; 
$Name = $_POST['Name']; 
$SName = $_POST['SName']; 
$Homepage = $_POST['Homepage']; 
$Info = $_POST['Info']; 
$BTeam = $_POST['BTeam']; 
$BLagg = $_POST['BLagg']; 
$BHomepage = $_POST['BHomepage']; 

if($Vorname == ""|| $Name == ""|| $SName == ""|| $Homepage == ""|| $Info == ""|| $BTeam == ""|| $BLagg == ""|| $BHomepage == "") { 
    echo "Bitte fuelle die Felder aus!"; 
} 
else { 
    $eintrag = "INSERT INTO bewertung (Vorname, Name, SName, Homepage, Info, BTeam, BLagg, BHomepage) VALUES ('".$Vorname."', '".$Name."', '".$SName."', '".$Homepage."', '".$Info."', '".$BTeam."', '".$BLagg."', '".$BHomepage."')"; 

    $eintragen = mysql_query($eintrag); 
    
    if (mysql_affected_rows() >= 1) { 
        echo "Deine Bewertung wurde eingetragen!";
    } 
    else { 
        echo "Fehler im System. Konnte nicht gespeichert werden...";
    } 
}

DerSaboteur · 11/27/2011, 15:21

Quote:
Originally Posted by .exTremez™
Bei Zeile 33 fehlt am ende ein ;
Code:
echo "Deine Bewertung wurde eingetragen!"[COLOR="Red"][B][U];[/U][/B][/COLOR]
Bei Zeile 37 das gleiche
Code:
echo "Fehler im System. Konnte nicht gespeichert werden..."[B][COLOR="Red"][U];[/U][/COLOR][/B]
EDIT://

Zeile 28(Du hast BHomapage anstatt BHomepage geschrieben ^^):
Code:
    ('$Vorname', '$Name', '$SName', '$Homepage', '$Info', '$BTeam', '$BLagg', '[B][COLOR="Red"][U]$BHomepage[/U][/COLOR][/B]')";

Danke. werds gleich umändern

Quote:

Originally Posted by demologin

Der 1€ Sonntag! Alles nur für 1€ Heute bis 17:00 uhr bestellen und ein beliebiges Produkt für nur 1€ im Ersten Monat bekommen.

dankeschoen für deinen hilfreichen Post <3
echt nett danke

Quote:

Originally Posted by Whoknowsit

Hinter deinen echo-Anweisungen fehlt das Semikolon

Ich würde es übrigens in etwa so machen:

PHP Code:

<?php $verbindung = mysql_connect("localhost", "Benutzername", "Passwort") or die ("Fehler beim Einloggen in MySql"); mysql_select_db("DatenbankName") or die ("Verbindung zur Datenbank ist nicht erfolgreich.."); function cleanUp($data) { $data = stripslashes($data); $data = strip_tags($data); $data = mysql_real_escape_string($data); return $data; } $_POST = array_map('cleanUp', $_POST); $Vorname = $_POST['Vorname']; $Name = $_POST['Name']; $SName = $_POST['SName']; $Homepage = $_POST['Homepage']; $Info = $_POST['Info']; $BTeam = $_POST['BTeam']; $BLagg = $_POST['BLagg']; $BHomepage = $_POST['BHomepage']; if($Vorname == ""|| $Name == ""|| $SName == ""|| $Homepage == ""|| $Info == ""|| $BTeam == ""|| $BLagg == ""|| $BHomepage == "") { echo "Bitte fuelle die Felder aus!"; } else { $eintrag = "INSERT INTO bewertung (Vorname, Name, SName, Homepage, Info, BTeam, BLagg, BHomepage) VALUES ('".$Vorname."', '".$Name."', '".$SName."', '".$Homepage."', '".$Info."', '".$BTeam."', '".$BLagg."', '".$BHomepage."')"; $eintragen = mysql_query($eintrag); if (mysql_affected_rows() >= 1) { echo "Deine Bewertung wurde eingetragen!"; } else { echo "Fehler im System. Konnte nicht gespeichert werden..."; } }

Was ist an dein .php besser? ich bin anfänger und weiß noch net alles

Whoknowsit · 11/27/2011, 15:33

Sagen wir so: Wenn du wirklich noch Anfänger bist, gewöhn dir am Besten direkt einen halbwegs ordentlichen Codingstil an. An meinem Beispiel (ist ungetestet) ist u.A. "besser", dass Formulareingaben (meiner Meinung nach) bestmöglichst gefiltert werden, bevor sie in die Datenbank geschrieben werden. Es sollte dadurch nicht mehr möglich sein, Schadcode einzuschleusen (Stichwort SQL-Injection und auch XSS).

Das geht auch alles noch "schöner", aber für den Anfang reichts ^^

Im Übrigen ist das reine Abfragen der Rückgabe von mysql_query nicht unbedingt aussagekräftig nzw. ich persönlich verlasse mich ungerne darauf. Daher habe ich mysql_affected_rows verwendet.

An für sich ist alles bis auf die cleanUp()-Funktion, die ich eingebaut habe, Geschmackssache. Soll jeder machen, wie er meint