[andy74]

Hallo,

ich habe ein Problem mit meinem Script. Ich erhalte einen Parser-Fehler

Parse error: parse error, unexpected T_STRING in /srv/www/htdocs/web163/html/validate.php on line 14

Das Script sieht folgendermaßen aus:

PHP-Code:

<?
include ("config.inc.php");

$query = "SELECT * FROM tmp_nl_anmeldung WHERE id = '$id";
$rs = mysql_query($query,$conn);
if (mysql_num_rows($rs) > 0) {
    $res = mysql_fetch_array($rs);
    $email = $res['email'];
    $name = $res['name'];
    $handy = $res['handy'];
    $sql = "INSERT INTO 'nl_anmeldung' ('email','name','handy','time','ip') VALUES ('$email','$name','$handy','$time','$ip');
    mysql_query($sql,$conn);
    
    $query = "DELETE FROM tmp_nl_anmeldung WHERE id = '$id'";
    mysql_query($query,$conn);
    }
else {
echo 'Es trat ein Fehler auf. Ihre Anmeldedaten konnten nicht gefunden werden. Bitte melden Sie sich erneut an';
}
?>

Ich habe irgendwie keine Ahnung, was beim Delete-query falsch ist. Würde mich über Hilfe wie immer ganz doll freuen.

[PaterNoster]

Das Syntax-Highlighting zeigt's Dir schon. das abschließende Anführungszeichen bei $sql fehlt...

[andy74]

Danke, der Hinweis hat weitergeholfen.

Beim testen des Scripts erhalte ich jetzt aber einen Fehler, die Zeile 6 betreffend

Warning: mysql_num_rows( supplied argument is not a valid MySQL result resource in /srv/www/htdocs/web163/html/validate.php on line 6

Ich habe mich hier an ein Beispielscript aus dem Buch "Programmieren lernen in PHP5" gehalten und kann daher den Fehler nicht ganz nachvollziehen.

[bob]

Hi,

PHP-Code:

$query = "SELECT * FROM tmp_nl_anmeldung WHERE id = '$id";
$rs = mysql_query($query,$conn); 


Da ist schon was falsch .... ich denke, das müsste so aussehen:

PHP-Code:

$query = "SELECT * FROM tmp_nl_anmeldung WHERE id = "'.$id.'" ";
$rs = mysql_query($query,$conn); 


Gruss,
bob

[andy74]

Die Änderung bringt mir wiederum einen Parser-Fehler

Parser-Fehler: C:\Programme\xampp\htdocs\zend\validate.php Zeile 4 - syntax error, unexpected T_CONSTANT_ENCAPSED_STRING

[PaterNoster]

PHP-Code:

$query = "SELECT * FROM tmp_nl_anmeldung WHERE id = '".$id."'";
$rs = mysql_query($query,$conn); 


Auch in dem Fall ist das Syntax-Highlighting Dein Freund...

[andy74]

OK, Parser-Fehler ist wieder weg, ich habe nun aber wieder den alten Fehler in Zeile 6.

Kann das was damit zu tun haben, dass ich auf dem Webspace noch PHP4 laufen habe?

[PaterNoster]

Woher kommt denn überhaupt der Wert in $id? Außerdem: Guck Dir mal in Deinem ersten Posting das erste $query an, da liegt doch auch schon was im argen... vergleich mal mit meinem Posting...

[andy74]

Den übernehme ich aus einem Link

PHP-Code:

http://www.......de/validate.php?id=$time&code=$code 


[Ben]

So sollte es funktionieren:

PHP-Code:

<?php

    include_once('config.inc.php');

    $sql = 'SELECT 
                     * 
              FROM 
                     tmp_nl_anmeldung 
              WHERE 
                     id = ' . $id;

    $resource = mysql_query($sql) or exit(mysql_error());

    if(mysql_num_rows($resource) > 0) {
        $tmp= mysql_fetch_assoc($resource);
        $email = $tmp['email'];
        $name = $tmp['name'];
        $handy = $tmp['handy'];
        
        $sql = "INSERT INTO 
                      nl_anmeldung 
                  (
                     'email',
                     'name',
                     'handy',
                     'time',
                     'ip'
                  ) 
                  VALUES 
                  (
                     '" . $email ."',
                     '" . $name ."',
                     '" . $handy . "',
                     '" . $time . "',
                     '" . $ip . "'
                  )";

        mysql_query($sql) or exit(mysql_error());
    
        $sql= 'DELETE FROM 
                      tmp_nl_anmeldung 
                 WHERE 
                      id = ' . $id;

        mysql_query($sql) or exit(mysql_error());
    }
    else {
        echo 'Fehlermeldung';
    }

?>

Ungetestet.
Nur mal etwas formatiert .

[bob]

dann mach es mit $_GET['id']; ... :rolleyes: ...

ICh habe es so gelöst:

$sql = "DELETE FROM tabelle WHERE id='".$id."'";

und nein bei mir ist id keine get variable

[Jojo]

@OP
befolge ma Bens Ratschlag bis auf eins:
das "exit" nach mysql_query() änderst du bitten in "die", denn sonst wirds nicht funktionieren, nich Ben?

//edit: narf scheiß de-docu bei PHP. Man! der Bernd hat doch recht ^^

[andy74]

Mal noch eine Frage: Muss ich $id nicht definieren, wenn ich es aus einer url nehme?

Bekomme Fehlermeldung "Query was empty".

[Lars]

Ja.

PHP-Code:

$id = array_key_exists('id', $_GET) ? intval($id) : 0; 


Steckt in der URI eine "id", steht sie in $id, ansonsten 0. array_key_exists() sollte selbsterklärend sein.

[Jojo]

wenn du $id aus einer URL entnimmst, dann musst du natürlich erst mal den Wert in der URL in $id speichern. arrow: hier dazu ein kleiner "Aufkärungskurs" http://forum.developers-guide.net/sh...t=688#post6007)

um auch bei eingeschaltenem error_reporting() keinen Fehler zu bekommen, würd ich dir folgende zuweisung empfehlen:

PHP-Code:

$id =  (isset($_GET['id']) (int) $_GET['id'] : 0; 


//edit: @Lars: da du schon ma schneller warst: was is besser: intval() oder (int)?

[PaterNoster]

Wenn Du es mit

PHP-Code:

$id = $_GET['id']; 


abholst, nicht.

Auch wenn es jetzt zu Test-/Übungszwecken übertrieben sein dürfte, solltest Du im produktiven Betrieb die Angaben bzw Variablen, die per URL kommen, immer noch mal prüfen. Nix ist sicher, schon gar nicht Werte, die von außen kommen.

[andy74]

hm, bekomms nicht zum laufen.

Muss es in Zeile 12

PHP-Code:

    $resource = mysql_query($query) or die(mysql_error()); 


oder

PHP-Code:

    $resource = mysql_query($sql) or die(mysql_error()); 


heissen??

[Ben]

Ich habe $query mal in $sql umbenannt .. nicht verwirren lassen

EDIT:
Sorry .. natürlich .. hehe .. .. habsc bisschen rumgekuddelmuddelt .. sorry

[andy74]

Ich bekomm jetzt folgende Meldung "Unknown column 'id' in 'where clause'"

[Jojo]

Jo, hast du denn überhaupt ne Spalte "id" ?