[mysql] Duplicate entry 'xxx' for key 1, Przy rejestracji... Opcje

[yamamoto]

Witam serdecznie,

Przy rejestracji na forum phpbb by Przemo (a właściwie stronie połączonej z tym forum) wyskakuje mi błąd następującej treści: Duplicate entry '328' for key 1. Nie mam zielonego pojęcia o co może chodzić, proszę o pomoc. Rejestracja dostępna pod adresem:
http://merengues.pl/register.php

Z góry dziękuję.

[yamamoto]

Spokojnie. ;]

[PHP] pobierz, plaintext 
<?php
function register()
{
  global $adres_strony, $mail_register_temat, $mail_register_tresc;
  
  if($_GET['action']=='dodaj')
  {
    
    if(empty($_POST['login']))
    {
      komunikat_blad('Pole login pozostało puste');
    }
    
    elseif(strlen($_POST['login'])<4)
    {
      komunikat_blad('Login musi mieć więcej niż trzy znaki');
    }
    
    elseif(empty($_POST['email']))
    {
      komunikat_blad('Pole e-mail pozostało puste');
    }
    
    elseif(!eregi('^[a-zA-z0-9_-.]+@[a-zA-z0-9-]+.[a-zA-z0-9-.]+$', $_POST['email']))
    {
      komunikat_blad('Adres e-mail jest niepoprawny');
    }
    
    elseif(empty($_POST['haslo']))
    {
      komunikat_blad('Pole hasło pozostało puste');
    }
    
    elseif(empty($_POST['rehaslo']))
    {
      komunikat_blad('Pole powtórz hasło pozostało puste');
    }
    
    elseif($_POST['haslo'] != $_POST['rehaslo'])
    {
      komunikat_blad('Hasła nie są takie same');
    }
    
    elseif(strlen($_POST['haslo'])<4)
    {
      komunikat_blad('Hasło musić mieć więcej niż 3 trzy znaki');
    }
    
    elseif(empty($_POST['token']))
    {
      komunikat_blad('Pole kod z obrazka pozostało puste');
    }
  
    elseif($_POST['token'] != $_SESSION['token'])
    {
      komunikat_blad('Źle przepisany kod z obrazka');
    }
  
    
    else
    {
      $rows_login = sql_rows("SELECT username FROM phpbb_users WHERE username='".$_POST['login']."'");
      $rows_email = sql_rows("SELECT user_email FROM phpbb_users WHERE user_email='".$_POST['email']."'");
      $rows_users = sql_rows("SELECT * FROM phpbb_users");
      
      if($rows_login>0)
      {
        komunikat_blad('Ten login jest już zajęty!');
      }
      
      elseif($rows_email>0)
      {
        komunikat_blad('Ten e-mail jest już zajęty!');
      }
      
      else
      {
        $login = addslashes($_POST['login']);
        $email = addslashes($_POST['email']);
        $haslo = addslashes($_POST['haslo']);
        $user_id = $rows_users+1;
        
        $sql = "INSERT INTO phpbb_users SET  username='".$login."',
                                             user_id='".$user_id."',
                                             user_active='0', 
                                             user_password='".md5($haslo)."', 
                                             kod='".md5($login)."',
                                             user_lang='polish',
                                             user_regdate = '".time()."', 
                                             user_email='".$email."'";
        if($query = mysql_query($sql) or die(mysql_error()))
        {
          @phpmailer($mail_register_temat, $mail_register_tresc, $email, $login);
          komunikat_finish_action('Rejestracja została zakończona, pozostaje tylko teraz aktywować konto. Na podany e-mail zostały wysłane dalsze instrukcje.', 'register.php');
        }
      }
    }
  }
  
  elseif($_GET['action']=="activation")
  {
    $sql = "UPDATE phpbb_users SET user_active='1' WHERE kod='".$_GET['code']."'";
        if($query = mysql_query($sql) or die(mysql_error()))
        {
          komunikat_finish_action('Konto zostało aktywowane!', 'index.php');
        }
  }
  
  include('module_theme/register.temp.php');
}
 
 
 
?>
[PHP] pobierz, plaintext 

Ten post edytował yamamoto 1.02.2009, 14:09:39

[piaseq]

Bez struktury tabeli z bazy danych nie jestem na 100% pewny, ale myślę, że chodzi o zmienną $user_id. Określasz ją zliczając liczbę użytkowników z tabeli phpbb_users i dodając 1. Jest to błąd ponieważ zapytanie

[SQL] pobierz, plaintext 
SELECT * FROM phpbb_users
[SQL] pobierz, plaintext 

zwraca liczbę użytkowników, a nie największe id. Widzę dwa wyjścia z tej sytuacji:
- sprawdź czy kolumna zawierająca id w bazie danych nie jest przypadkiem auto increment, wtedy przy odpowiedniej modyfikacji zapytania baza "sama ustali" jaki id należy wpisać
- możesz też zmienić zapytanie tak aby pobierało maksymalny numer id i dodawać do niego 1.

Taka mała uwaga, do kodu php używaj tagów [php] zamiast [code], łatwiej się czyta (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)