Forum PHP.pl

Witam.

Mam problem z utworzeniem tabeli w php - niby przykład jest ok ale coś mi nie działa utworzenie tabeli ponieważ mam jakiś błąd w składni - możeci mi podpowiedzieć gdzie ?

Błąd :

1064: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')' at line 1

składnia polecenia :

[PHP] pobierz, plaintext 
$query ="user_number MEDIUMINT(10) DEFAULT '0' NOT NULL AUTO_INCREMENT,";
$query.="user_id VARCHAR(8) BINARY NOT NULL,";
$query.="user_name VARCHAR(30) NOT NULL,";
$query.="user_password VARCHAR(20) BINARY NOT NULL,";
$query.="user_country VARCHAR(50) NOT NULL,";
$query.="user_email VARCHAR(50) NOT NULL,";
$query.="user_profile TEXT NOT NULL,";
$query.="user_registre_date DATE DEFAULT '0000-00-00' NOT NULL,";
$query.="user_last_access_time TIMESTAMP(14),";
$query.="PRIMARY KEY(user_id),";
$query.="UNIQUE user_number(user_number)";
[PHP] pobierz, plaintext 

Liczę na waszą pomoc bo takie ponoć podstawowe i proste zapytanie a nie wiem co jest w nim źle

Jeżeli ktoś może mi jeszcze poradzić jak podzielić bazę danych odnośnie informacji o użytkowniku to też będę wdzięczny.

Docelowo chcę stworzyć 2 tabele dla użytkownika - w pierwszej mają być wszelkie informacje jakie użytkownik będzie podawał podczas rejestracji a w drugiej będą inne informacje powiązane z użytkownikiem i innymi danymi.

Ten post edytował Setesh 21.01.2011, 18:27:53

A gdzie zjadles początek, czyli
CREATE TABLE blabla
?

[PHP] pobierz, plaintext 
$dbname = "komoorkowo";
$user_tablename = "user";
 
$query ="user_number MEDIUMINT(10) DEFAULT '0' NOT NULL AUTO_INCREMENT,";
$query.="user_id VARCHAR(8) BINARY NOT NULL,";
$query.="user_name VARCHAR(30) NOT NULL,";
$query.="user_password VARCHAR(20) BINARY NOT NULL,";
$query.="user_country VARCHAR(50) NOT NULL,";
$query.="user_email VARCHAR(50) NOT NULL,";
$query.="user_profile TEXT NOT NULL,";
$query.="user_registre_date DATE DEFAULT '0000-00-00' NOT NULL,";
$query.="user_last_access_time TIMESTAMP(14),";
$query.="PRIMARY KEY(user_id),";
$query.="UNIQUE user_number(user_number)";
 
$link_id = db_connect();
if(!$link_id) die(sql_error());
 
if(!mysql_query("CREATE TABLE $user_tablename ($query)")) die(sql_error());  // JEST TUTAJ :)
  else echo "utworzono tabele";
[PHP] pobierz, plaintext 

// EDIT :

query -> $query // poprawiłem przy wywołaniu

teraz mam inny błąd

"1067: Invalid default value for 'user_number'"



Jakby co będę około 15:00 to zobaczę co jeszcze dopatrzę się ale nie wiem jaka inna wartość może być poza "0" dla auto inc kiedy nie może być też null

Ten post edytował Setesh 21.01.2011, 12:59:51

$query ="user_number MEDIUMINT(10) DEFAULT 0 NOT NULL AUTO_INCREMENT,";

zmieniłem i nadal że zła wartość

autoincrement nie ma domyslnej wartosci

$query ="user_number MEDIUMINT(10) NOT NULL AUTO_INCREMENT,";

no i gitara

Dzięki wielkie Zawsze to trochę się nauczyłem

Ten post edytował Setesh 21.01.2011, 13:06:55

1)

Co to za tag? Tag ma odzwierciedlac rodzaj problemu a nie Twoj stan wiedzy. popraw

2)
nie:MEDIUMINT(10)
a:MEDIUMINT
10 niczemu nie sluzy.

3)
Niby robisz autoincrement user_number, ale jako klucza głównego uzywasz user_id i to na dodatek tekstu. Czemu to niby ma sluzyc? Zazwyczaj kluczem głównym jest autoincrement

4)
user_password VARCHAR(20)
haslo trzymasz jawnie? Zle
Haslo trzymasz w hashu? To po co zmienna dlugosc pola?

[PHP] pobierz, plaintext 
    $dbname = "komoorkowo";
    $user_tablename = "user";
 
    $query.="user_id MEDIUMINT NOT NULL AUTO_INCREMENT,";
    $query.="user_name VARCHAR(30) NOT NULL,";
    $query.="user_password VARCHAR(20) BINARY NOT NULL,";
    $query.="user_country VARCHAR(50) NOT NULL,";
    $query.="user_email VARCHAR(50) NOT NULL,";
    $query.="user_profile TEXT NOT NULL,";
    $query.="user_registre_date DATE DEFAULT '0000-00-00' NOT NULL,";
    $query.="user_last_access_time TIMESTAMP(14),";
    $query.="PRIMARY KEY(user_id),";
    $query.="UNIQUE user_number(user_number)";
 
    $link_id = db_connect();
    if(!$link_id) die(sql_error());
 
    if(!mysql_query("CREATE TABLE $user_tablename ($query)")) die(sql_error());
    else echo "utworzono tabele";
[PHP] pobierz, plaintext 

Ok poprawiłem co się dało (mam nadzieję że dobrze) ale nie wiem jak z hasłem zrobić sad.gif Mógłbym prosić o gotowy przykład jak to zrobić aby było zahashowane ? Czytałem o opcji

[PHP] pobierz, plaintext 
password($user_password);
[PHP] pobierz, plaintext 

i że ponoć samo hash załatwia (a może coś mi się pomyliło )

Zmieniłem unikalny element na user_email i user_name (wydawało się najrozsądniejsze) :

[PHP] pobierz, plaintext 
 $query.="UNIQUE user_email(user_email, user_name)";
[PHP] pobierz, plaintext 

Obecne zapytanie wygląda tak :

[PHP] pobierz, plaintext 
    $query.="user_id MEDIUMINT NOT NULL AUTO_INCREMENT,";
    $query.="user_email VARCHAR(50) NOT NULL,";
    $query.="user_password CHAR BINARY NOT NULL,";
    $query.="user_name VARCHAR(30) NOT NULL,";
    $query.="user_country VARCHAR(50) NOT NULL,";
    $query.="user_profile TEXT NOT NULL,";
    $query.="user_registre_date DATE DEFAULT '0000-00-00' NOT NULL,";
    $query.="user_last_access_time TIMESTAMP(14),";
    $query.="PRIMARY KEY(user_id),";
    $query.="UNIQUE user_email(user_email, user_name)";
[PHP] pobierz, plaintext 

Zamieniłem user_password z

[PHP] pobierz, plaintext 
VARCHAR(20) na CHAR
[PHP] pobierz, plaintext 

(nie wiem czy dobrze).