Proszę o ocenę bazy danych, czy jest wystarczająca jeżeli chodzi o funkcjonalność sklepową i czy nie ma żadnych błędów logicznych. Czy takie założenie bazy danych sklepu jest poprawne?

Proszę o wstępne oceny, bo i tak nie wiecie zbyt dużo...

LINK

Hej

Kilka uwag:

• po co dwie tabele dla danych klientów?? Moja propozycja:

  • [SQL] pobierz, plaintext 
    TABLE klienci (
    id int(10) NOT NULL AUTO_INCREMENT,
    -- podstawowe dane konta --
    login char(25) collate utf8_polish_ci NOT NULL,
    haslo char(50) collate utf8_polish_ci NOT NULL,
    email char(25) collate utf8_polish_ci NOT NULL,
    pytanie char(50) collate utf8_polish_ci NOT NULL,
    odpowiedz char(50) collate utf8_polish_ci NOT NULL,
    ustawienia int(10) NOT NULL, -- id --
    typklientaid int(10) NOT NULL,
    -- dane statystyczne --
    ostatnie_logowanie char(50) collate utf8_polish_ci NOT NULL,
    ostatnie_ip char(50) collate utf8_polish_ci NOT NULL,
    ile_kupil int(10) collate utf8_polish_ci NOT NULL,
    ile_zamowil int(10) collate utf8_polish_ci NOT NULL,
    -- dane o kliencie jako osoba --
    imie char(25) collate utf8_polish_ci NOT NULL,
    nazwisko char(25) collate utf8_polish_ci NOT NULL,
    -- kontakt --
    numer_tel char(15) collate utf8_polish_ci NOT NULL,
    numer_gg char(15) collate utf8_polish_ci NOT NULL,
    skype char(20) collate utf8_polish_ci NOT NULL,
    PRIMARY KEY  (`id`),
    UNIQUE KEY (`login`)
    ) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=1;
     
    TABLE typklienta
    id
    typklienta
     
    TABLE adresy
    id int(10) NOT NULL
    klientid int(10) NOT NULL
    typadresuid int(10) NOT NULL
      krajid int(10) NOT NULL collate utf8_polish_ci,
      województwoid char(25) collate utf8_polish_ci NOT NULL,
      miasto char(25) collate utf8_polish_ci NOT NULL,
      kod_pocztowy char(10) collate utf8_polish_ci NOT NULL,
      ulica char(25) collate utf8_polish_ci NOT NULL,
      numer_m_d char(10) collate utf8_polish_ci NOT NULL,
     
    TABLE typadresu
    id
    typadresu
     
    TABLE kraj
    id
    kraj
    [SQL] pobierz, plaintext 

Wydaje mi się, iż takie atomowe wydzielenie danych pozwoli zachować łatwą możliwość rozbudowy systemu. Ponadto łatwiej będzie zbudować interfejs użytkownika w postaci selektów...

Idąc dalej tą drogą można jeszcze wydzielić tabele: typtelefonu(id, typ), telefon(id, typid, nr, klientid), komunikator(id, typid, nr, klientid), typkomunikatora (id, typ).

ps. pisałem pseudo kod, ale wierzę iż będziesz w stanie zrobić z tego pełnoprawny mysql...

Pozdrawiam

forum oceny sluzy innym ocenom. TEmat przenoszę.

a co ty wszędzie walisz char? Czy adres email ma zawsze dlugość 25 znaków? Poczytaj o typie varchar a nastepnie zastanow sie jaka jest roznica miedzy char a varchar.

"ostatnie_logowanie char(50)"
a od kiedy to data ostatniego logowania to tekst? Data to data. typ DATETIME

"ostatnie_ip char(50)"
Widziales kiedys IP skladające sie z 50 znaków?

"ile_kupil int"
Widziales kiedys klienta co kupił kilka miliardów sztuk czegos?
i na dodatek kupi liczbę ujemną? poczytaj o unsigned

"kod_pocztowy char(10)"
Wiesz jak wyglada kod pocztowy?

"na_ktorya_adres_przes char(20) collate utf8_polish_ci NOT NULL, -- wartosc: firmowy, konataktowy --"
poczytaj o typie ENUM

"CREATE TABLE obserwowane_produkty (
id int(10) NOT NULL auto_increment,
produkt int(10) NOT NULL,
klient int(10) NOT NULL,
PRIMARY KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=1;
"
Zainteresuj się typem InnoDB i relacjami w bazie danych

"cena char(10)"
cena to naprawde tekst?


DAlej mi się już nie chce podawać bo wszystko na jedno kopyto.
ocena: 2-

Czyli lepiej w niektórych miejscach dawać varchar?

CHAR służy do przechowywania danych o stałej długości (np. hasło zakodowane MD5), natomiast w VARCHAR pakujesz dane, których długości nie znasz (imię, nazwisko).

To klientów tak załatwiłem

[SQL] pobierz, plaintext 
CREATE TABLE klient (
id int(10) NOT NULL AUTO_INCREMENT,
idklienci  int(10) NOT NULL,
idustawienia int(10) NOT NULL, -- id --
idtypklienta int(10) NOT NULL, -- id --
idadresy char(25) collate utf8_polish_ci NOT NULL, -- id adrsów --
idfirma int(10) NOT NULL, -- 0 gdy nie firma! --
PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=1;
 
TABLE klienci (
id int(10) NOT NULL AUTO_INCREMENT,
-- podstawowe dane konta --
login char(25) collate utf8_polish_ci NOT NULL,
haslo char(50) collate utf8_polish_ci NOT NULL,
email varchar collate utf8_polish_ci NOT NULL,
pytanie varchar collate utf8_polish_ci NOT NULL,
odpowiedz varchar collate utf8_polish_ci NOT NULL,
-- dane statystyczne --
ostatnie_logowanie char(50) collate utf8_polish_ci NOT NULL,
ostatnie_ip char(50) collate utf8_polish_ci NOT NULL,
ile_kupil int(10) collate utf8_polish_ci NOT NULL,
ile_zamowil int(10) collate utf8_polish_ci NOT NULL,
-- dane o kliencie jako osoba --
imie varchar collate utf8_polish_ci NOT NULL,
nazwisko varchar collate utf8_polish_ci NOT NULL,
-- kontakt --
numer_tel char(12) collate utf8_polish_ci NOT NULL,
numer_gg char(10) collate utf8_polish_ci NOT NULL,
skype varchar collate utf8_polish_ci NOT NULL,
PRIMARY KEY  (`id`),
UNIQUE KEY (`login`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=1;
 
CREATE TABLE typklienta (
id int(10) NOT NULL AUTO_INCREMENT,
typklienta char(10) collate utf8_polish_ci NOT NULL, -- detaliczni, hurtowi -- 
PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=1;
 
CREATE TABLE adresy (
id int(10) NOT NULL AUTO_INCREMENT,
typadresu int(10) NOT NULL, -- firma, nie firma --
kraj int(10) NOT NULL, -- id --
województwo varchar collate utf8_polish_ci NOT NULL,
miasto varchar collate utf8_polish_ci NOT NULL,
kod_pocztowy char(5) collate utf8_polish_ci NOT NULL,
ulica varchar collate utf8_polish_ci NOT NULL,
numer_m_d char(10) collate utf8_polish_ci NOT NULL,
PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=1;
 
CREATE TABLE firma (
id int(10) NOT NULL AUTO_INCREMENT,
nip varchar collate utf8_polish_ci NOT NULL,
regon varchar collate utf8_polish_ci NOT NULL,
PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=1;
 
CREATE TABLE panstwa (
id int(10) NOT NULL AUTO_INCREMENT,
nazwa varchar collate utf8_polish_ci NOT NULL,
PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=1;
[SQL] pobierz, plaintext 

To chyba DATETIME powinno być raczej



Nie chcesz mieć polskich liter w nazwach tabel/pól. Uwierz mi.


Poczytaj o funkcjach INET_ATON i INET_NTOA i zmień to pole na INTEGER UNSIGNED


A nie INT przypadkiem?



Akurat te numery mają stałą ilość znaków, więc CHAR w odpowiednim rozmiarze.



Może lepiej NULL gdy nie firma?



Ogólnie: VARCHAR wymaga podania maksymalnego rozmiaru pola.