[Ben]

Nochmal von vorne.
Zeig uns mal die Tabellenstruktur .. am besten ein Strukturexport aus phpMyAdmin heraus.

[andy74]

eeemmmm, moment. Gebt mir mal ein bissl Zeit. Ich glaube, ich habe durch die Änderung von Variablen hier was durcheinander gebracht

[andy74]

So, habe hier doch etwas durcheinander gebracht. Das Script sieht bei mir jetzt wie folgt aus:

PHP-Code:

<?php

    include_once('config.inc.php');
    $sql = 'SELECT 
                     * 
              FROM 
                     tmp_nl_anmeldung 
              WHERE 
                     code = ' . $code;

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

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

        mysql_query($sql) or die(mysql_error());
    
        $sql= 'DELETE FROM 
                      tmp_nl_anmeldung 
                 WHERE 
                      code = ' . $code;

        mysql_query($sql) or die(mysql_error());
        echo 'Sie wurden erfolgreich registriert';
    }
    else {
        echo 'Fehlermeldung';
    }

?>

In der tmp_nl_anmeldung habe ich folgende Spalten: email, name, handy, time, code, ip
In der nl_anmeldung habe ich dann noch email, name, handy, time, ip.

Ich erhalte jetzt nach Umstellung folgenden Fehler:

Unknown column 'YffWz1CieNYXQlGp8XlM' in 'where clause'

wobei ich das Script mit folgendem Link aufrufe:

http://www......php?code=YffWz1CieNYXQlGp8XlM

[Lars]

PHP-Code:

$sql = "
         [...]
         WHERE
             code = '" . $code . "'"; 


[andy74]

Es erscheint jetzt erst mal keine Fehlermeldung mehr.

Allerdings passt es irgendwie immer noch nicht. Wenn ich einen Datensatz in der tmp_nl_anmeldung habe und dann auf den link klicke, ignoriert er die if-Schleife, setzt also mysql_num_rows auf 0 und geht zu else

[WarrenFaith]

Check mal ob dein $code mit dem in der DB übereinstimmt?! Ich hatte mal den fehler, dass die DB was abgeschnitten hat, weil die Länge des Feldes überschritten wurde.
Ist allerdings nur eine Vermutung ins blaue

[andy74]

hab eine Länge von 30 Zeichen - funktioniert trotzdem nicht.

[Ben]

Zitat von Ben

Nochmal von vorne.
Zeig uns mal die Tabellenstruktur .. am besten ein Strukturexport aus phpMyAdmin heraus.

[andy74]

Hier die Strukturen:

-- Table structure for table `tmp_nl_anmeldung`
--

CREATE TABLE `tmp_nl_anmeldung` (
`email` text NOT NULL,
`name` text,
`handy` text,
`time` text NOT NULL,
`code` text NOT NULL,
`ip` text NOT NULL
) ENGINE=MyISAM DEFAULT CHARSET=latin1;

-- Table structure for table `nl_anmeldung`
--

CREATE TABLE `nl_anmeldung` (
`email` text NOT NULL,
`name` text,
`handy` text,
`time` text NOT NULL,
`ip` text NOT NULL
) ENGINE=MyISAM DEFAULT CHARSET=latin1;

Noch eine Idee: Habe ich überhaupt eine Verbindung zur Datenbank?

[bob]

Noch eine Idee: Habe ich überhaupt eine Verbindung zur Datenbank?

LOOOL.

Verbinden kannst du dich mittels mysql_connect

http://de.php.net/manual/de/function.mysql-connect.php


Gruss,
bob

[andy74]

Zitat von bob

LOOOL.

Verbinden kannst du dich mittels mysql_connect

http://de.php.net/manual/de/function.mysql-connect.php


Gruss,
bob

Nee, das wars doch nicht, hatte mich geirrt.

Ach, noch eine Feststellung: Wenn ich gleich am Anfang des Scripts $code definiere und danach per echo ausgebe, erscheint nix. Die Variable scheint also nicht übernommen zu werden.

[Jojo]

Joah
du füllst die Variable $code ja nirgends!
Lars, PaterNoster und ich haben dir hintereinander geschrieben, was du machen musst.
Grüße,
Jojo

[andy74]

Variable habe ich jetzt mittels

PHP-Code:

$code = $_GET['code']; 


gefüllt.
Ergebnis ist, dass ich die Variable angezeigt bekomme, jedoch dann auch gleich den Sprung zu else {}

[Jojo]

okey,
bist du dir sicher, dass überhaupt eine Zeile besteht, die bei code das drinne stehen hat?
Teste deinen Code doch mal im PHPMyAdmin-Interface und schau, ob überhaupt was ausgewählt wird.
Warum sind bei dir eigentlich alle Spalten vom "text"-Type?
Normalerweise nimmt man für abschätzbare, nicht zu lange Einträge VARCHAR.
Noch was: Lass dir mal bitte die zusammengestellte sql-query ausgeben und poste sie mal hier (hab da so ne Vermutung).

Grüße,
Jojo

[andy74]

So, mit Jojo´s Hilfe hab ich es dann noch hinbekommen. Es waren hauptsächlich Probleme mit den SQL-Anweisungen. Ich möchte hier noch einmal das Script veröffentlichen, so, wie es funktioniert.

PHP-Code:

<?php
  $dbhost   = '***';
  $dblogin  = '***';
  $dbpasswd = '***';
  $dbname   = '***';

  mysql_connect($dbhost, $dblogin, $dbpasswd) or die(mysql_error());
  mysql_select_db($dbname) or die(mysql_error());

$code = $_GET['code'];   

    $sql = "SELECT 
                     * 
              FROM 
                     tmp_nl_anmeldung 
              WHERE 
                     code = '" .$code. "'";

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

        mysql_query($sql) or die(mysql_error());
    
        $sql= "DELETE FROM 
                      tmp_nl_anmeldung 
                 WHERE 
                      code = '" .$code. "'";

        mysql_query($sql) or die(mysql_error());
        echo 'Sie wurden erfolgreich registriert';
    }
    else {
        echo 'Fehlermeldung';
    }

?>

Gleichzeitig möchte ich mich für Eure Ausdauer bedanken, so dass ich hier wieder jede Menge lernen konnte.

[Lars]

Die Definierung von $code ist anfällig für SQL-Injection.

PHP-Code:

$code = array_key_exists('code', $_GET) ? mysql_real_escape_string($_GET['code']) : ''; 


[dejan_spasic]

Nur so mal am Rande. Ich war mal neugirig und wollte wissen, welche der beiden Methoden, um ein Schlüssel eines Feldes zu überprüfen, Perfomanter ist.

PHP-Code:

$code = isset($_GET['code']);//....
//oder
$code = array_key_exists('code', $_GET);//... 


Mal davon abgesehen das ich ein Fauler Typ bin und ich aus dem Gründen die isset Method bevorzuge, ist diese Methode auch um einiges Perfomanter. Um ca. 40% sogar.

Getestet mit PHP 5.0.4.

[Lars]

Steht in $_GET['code] NULL, gibt isset() false zurück, array_key_exists() afaik true.

[dejan_spasic]

Da gebe ich dir Recht. Aber bei $_GET und $_POST sind die übergebenen Werte eine Zeichenkette.

[Lars]

Da hast du Recht.
Die Performance halte ich bei maximal 10 Aufrufen pro Sckript allerdings für zweirangig