Baza Danyc czy poprawnie zrobiona? Opcje

[sej87]

hejka czy mógłby ktos luknąc czy to jest poprawnie napisane
z gory wielkie THX

[SQL] pobierz, plaintext 
DROP TABLE Wiadomosc;
DROP TABLE O_user;
DROP TABLE Zainteresowania;
DROP TABLE USERS;
DROP TABLE Plec;
DROP TABLE Wyksztalcenie;
DROP TABLE Wojewodztwa;
 
 
CREATE TABLE Wojewodztwa (
id_wojewodztwa     NUMBER          NOT NULL CONSTRAINT pk_wojewodztwa PRIMARY KEY,
wojewodztwo        VARCHAR2(20)    NOT NULL
);
 
CREATE TABLE Wyksztalcenie (
id_wykształcenia   NUMBER          NOT NULL CONSTRAINT pk_Wyksztalcenie PRIMARY KEY,
wyksztalcenie      VARCHAR2(20)    NOT NULL
);
 
CREATE TABLE Plec (
id_plci            NUMBER          NOT NULL CONSTRAINT pk_Plec PRIMARY KEY,
plec               VARCHAR2(15)    NOT NULL
);
 
CREATE TABLE USERS (
user_id            NUMBER          NOT NULL CONSTRAINT pk_USERS PRIMARY KEY,
user_login VARCHAR2(10)    NOT NULL ,
user_haslo         VARCHAR2(10)    NOT NULL ,
user_imie          VARCHAR2(15)    NOT NULL ,
user_plec          NUMBER          NOT NULL ,
user_rocznik       NUMBER(4)       NOT NULL ,
user_wyksztalcenie NUMBER          NOT NULL ,
user_wojewodztwo   NUMBER          NOT NULL ,
CONSTRAINT fk_user_wojewodztwo    FOREIGN KEY (user_wojewodztwo)   REFERENCES Wojewodztwa (id_wojewodztwa),
CONSTRAINT fk_user_wyksztalcenie  FOREIGN KEY (user_wyksztalcenie) REFERENCES Wyksztalcenie (id_wyksztalcenia),
CONSTRAINT fk_user_plec           FOREIGN KEY (user_plec)          REFERENCES Plec (id_plci)
);
 
CREATE TABLE O_user (
id_o_user          NUMBER          NOT NULL CONSTRAINT pk_O_user PRIMARY KEY,
id_user            NUMBER          NOT NULL ,
strona_www         VARCHAR(20)     NOT NULL ,
zodiak             VARCHAR2(30)    NOT NULL ,
wzrost             NUMBER(3)       NOT NULL ,
kol_wlos           VARCHAR2(20)    NOT NULL ,
o_sobie            VARCHAR2 (200),
CONSTRAINT fk_id_user FOREIGN KEY (id_user) REFERENCES USERS (user_id)
);
 
CREATE TABLE Zainteresowania (
id_zaint           NUMBER          NOT NULL CONSTRAINT pk_Zainteresowania PRIMARY KEY,
user_id            NUMBER          NOT NULL ,
hobby              VARCHAR2(30)    NOT NULL ,
ul_rodz_film       VARCHAR2(30)    NOT NULL ,
ul_rodz_muzyki     VARCHAR2(30)    NOT NULL ,
CONSTRAINT fk_user_id FOREIGN KEY (user_id) REFERENCES USERS (user_id)
);
 
CREATE TABLE Wiadomosc (
wiad_id           NUMBER           NOT NULL CONSTRAINT pk_Wiadomosc PRIMARY KEY,
wiad_tresc        VARCHAR2(200),
wiad_do           NUMBER ,
wiad_od           NUMBER ,
wiad_przeczytane  VARCHAR2(20),
wiad_data         VARCHAR2(10),
wiad_temat        VARCHAR2(30),
wiad_czyj         VARCHAR2(20),
CONSTRAINT fk_wiad_do FOREIGN KEY (wiad_do) REFERENCES USERS (user_id),
CONSTRAINT fk_wiad_od FOREIGN KEY (wiad_od) REFERENCES USERS (user_id)
);
[SQL] pobierz, plaintext 

Ten post edytował sej87 10.06.2008, 21:04:27

[szumigt]

Czort wie do czego Ty chesz używać tę bazę ale to jakby na drugim planie.
Dlaczego w tabeli wiadomość pole przeczytane jest typu varchar jeszcze 20 a może być przecież boolean. Tutaj wystarczy przecież tak lub nie?? Następne. Co to jest pole wiad_czyj?? Z nagłówka widząc szukałbym tam wartości czyja jest zamieszczona wiadomość. Jeśli tak to nie bardzo wiem po co jest pole wiad_od, notabene całkiem sensowne.
Reszta tak na szybko chyba ok ...

Ten post edytował szumigt 26.06.2008, 12:38:02

[nospor]

troche stary ten temat... no, moze nie stary, ale napewno nie pierwszej mlodosci

No ale i ja dorzuce pare groszy:
plec VARCHAR2(15) - przeciez tu tez wystaryczy jeden bajt na wpisanie 0 - obojniak, 1 - kobieta, 2 - mezczyzna
wiad_data VARCHAR2(10), - skoro to jest data, to powinno to byc pole typu datowego. Ulatwisz sobie zycie przy wszelkiego rodzaju wyszukiwaniach