[PostgreSQL][PHP] Problem z zapytaniem, projekt bazy danych Opcje

[Xenom]

Korzystam z bazy danych postgresql, ale używam poradników z mysql i w razie błedów zapytań staram się poszukać informacji w google.
W poniższym przypadku trafiłem na nadmiar informacji i tutaj poproszę o pomoc. Przedstawię najlepiej cały kod z zakomentowaniem linijek dających błąd, to będzie mówić samo za siebie i więcej niż moje amatorskie wyjaśnienie.
Jest to mój pierwszy projekt bazy danych/php i za krytyczne uwagi będę wdzięczny (varchary nie są skonfigurowane).

[PHP] pobierz, plaintext 
<?php
require "db_conf.php";
require "db_connect.php";
 
$create_users_table = pg_query('CREATE TABLE users (
	id_user BIGINT NOT NULL,                        //w tym wypadku nie SERIAL bo dla zwykłego usera bedzie korzystanie z konta facebook
	money FLOAT DEFAULT '0',
	name VARCHAR(20) DEFAULT 'none',
	mail VARCHAR(20) DEFAULT 'none',
	PRIMARY KEY (id_user)
	)');
 
$create_companies_table = pg_query('CREATE TABLE companies (
	login VARCHAR(20) NOT NULL,                //też nie SERIAL, tylko login na pr.key, poniewaz wiekszosc zapytan bedzie szukac po loginie
	company_name VARCHAR(20) NOT NULL,
	regon VARCHAR(20) NOT NULL,
	nip VARCHAR(20) NOT NULL,
	krs VARCHAR(20) DEFAULT 'none',
	director VARCHAR(20) NOT NULL,
	adress VARCHAR(20) NOT NULL,
	city VARCHAR(20) NOT NULL,
	state VARCHAR(20) NOT NULL,
	phone VARCHAR(20) NOT NULL,
	mail VARCHAR(20) NOT NULL,
	password VARCHAR(20) NOT NULL,
	money FLOAT DEFAULT '0',
	active BOOL DEFAULT '0',
	PRIMARY KEY (login)
	)');
 
$create_products_table = pg_query('CREATE TABLE products (
	id_product BIGINT SERIAL NOT NULL,     //bigint bo moze byc z czasem cala masa produktów/ jest cos takiego jak unsigned w cpp?
	login_seller VARCHAR(20) NOT NULL,
	normal_price FLOAT NOT NULL,
	promotion_price FLOAT NOT NULL,
	product_name VARCHAR(20) NOT NULL,
	description text NOT NULL,
	product_city VARCHAR(20) NOT NULL,
	start_date VARCHAR(20) NOT NULL,
	day_count INT NOT NULL,
	photo VARCHAR(20) NOT NULL,
	PRIMARY KEY (id_product),
	KEY (login_seller)                             //blad tutaj przy zapytaniu
	)');
 
$control_table = pg_query('CREATE TABLE control ( //ta tabela kotroluje jaki user, jaki produkt zakupil
	id_user BIGINT NOT NULL,
	id_product BIGINT NOT NULL,
	date VARCHAR(16) NOT NULL
	INDEX ('id_user','id_product')           //blad tutaj przy zapytaniu
	)');
?>
[PHP] pobierz, plaintext 

Ten post edytował Xenom 18.04.2012, 12:27:34

[bpskiba]

Po mojemu jest to bardzo zła konstrukcja.
1. Kwestia porządku: Jeżeli tworzysz tabelę users, to kluczem powinno być id_users. Dla tabeli products id_products
2. Dla kluczy głównych obowiązkowo autoincrement
3. Za pomysł uczynienia kluczem głównym pola VARCHAR w tabeli companies należy Cię wybatożyć
4. Pola na pieniądze nie float, a numeric lub decymal
5. Pole na datę nie typu Varchar, a np DATETIME
6. Brak powiązań między tabelami
7. Takie dane jak telefon lepiej przechowywać w polach INT
8. Za mało kluczy zewnętrznych wymuszających więzy integralności
9. Widzę pole start_date. Jeżeli masz zamiar określać odstępy między datami, to obowiązkowo typ Timestamp
10. NIP, regon to pola typu INT (pomyśl jak będzie wyglądać wyszukiwanie po numerze nip)

W ramach higieny języka polskiego zapytam czemu te nazwy tabel są po angielsku.
Na osłodę dodam, że to są typowe błędy, które popełniał każdy (ze mną włącznie) (IMG:style_emoticons/default/aarambo.gif)