Baza SQL Opcje

[sej87]

hejka,
Czy może ktoś lunnać czy tu jest wszystko oki napisane??
Z gór wielkie THX

[SQL] pobierz, plaintext 
DROP TABLE Wiadomosc;
DROP TABLE Znajomi;
DROP TABLE Zainteresowania;
DROP TABLE O_user;
DROP TABLE USERS;
DROP TABLE Wojewodztwa;
 
CREATE TABLE Wojewodztwa (
id_wojewodztwa     NUMBER(10)     CONSTRAINT pk_wojewodztwa PRIMARY KEY,
wojewodztwo        VARCHAR(20)
);
 
CREATE TABLE USERS (
user_id            NUMBER(10)     CONSTRAINT pk_USERS PRIMARY KEY,
user_login VARCHAR2(15),
user_haslo         VARCHAR2(10),
user_imie          VARCHAR2(15),
user_plec          VARCHAR2(15),
user_wiek          VARCHAR2(10),
user_wojewodztwo   NUMBER(10)     REFERENCES Wojewodztwa (id_wojewodztwa)
);
 
CREATE TABLE O_user (
id_o_user          NUMBER(10)     CONSTRAINT pk_O_user PRIMARY KEY,
uder_id            NUMBER(10)     REFERENCES USERS (user_id),
wyksztalcenie      VARCHAR2(15),
strona_www         VARCHAR(20),
zodiak             VARCHAR2(15),
wzrost             NUMBER(5),
kol_wlos           VARCHAR2(20),
o_sobie            VARCHAR2 (200)
);
 
CREATE TABLE Zainteresowania (
id_zaint           NUMBER(10)     CONSTRAINT pk_Zainteresowania PRIMARY KEY,
user_id            NUMBER(10)     REFERENCES USERS (user_id),
sport_dziedzina    VARCHAR2(20),
ul_rodz_film       VARCHAR2(20),
ul_rodz_muzyki     VARCHAR2(20)
);
 
CREATE TABLE Znajomi (
id_znajomi         NUMBER(10)     CONSTRAINT pk_Znajomi PRIMARY KEY,
user_id1           NUMBER(10)     REFERENCES USERS (user_id),
user_id2           NUMBER(10)     REFERENCES USERS (user_id),
status1            VARCHAR2(20),
status2            VARCHAR2(20)
);
 
CREATE TABLE Wiadomosc (
wiad_id           NUMBER(10)      CONSTRAINT pk_Wiadomosc PRIMARY KEY,
wiad_tresc        VARCHAR2(200),
wiad_do           NUMBER(10)      REFERENCES USERS (user_id),
wiad_od           NUMBER(10)      REFERENCES USERS (user_id),
wiad_przeczytane  VARCHAR2(20),
wiad_data         VARCHAR2(10),
wiad_temat        VARCHAR2(10),
wiad_czyj         VARCHAR2(20)
);
[SQL] pobierz, plaintext 

[mosqi83]

1. Nie za bardzo rozumiem co wyraża tabela Znajomi, ale prawdopodobnie zostało to źle zaprojektowane.

2. Wydaje się, że lepiej byłoby tabele Zainteresowania i O_user złączyć z tabelą USERS, a niektóre pola (np. zodiak, wyksztalcenie) rozbić tak jak w przypadku pola user_wojewodztwo.

[JoShiMa]

1. Nie za bardzo rozumiem co wyraża tabela Znajomi, ale prawdopodobnie zostało to źle zaprojektowane.

2. Wydaje się, że lepiej byłoby tabele Zainteresowania i O_user złączyć z tabelą USERS, a niektóre pola (np. zodiak, wyksztalcenie) rozbić tak jak w przypadku pola user_wojewodztwo.

Oczywiście, że źle. Po co tam dwa klucze obce. Każda para znajomych powinna być jako osobny rekord i klucz podwójny na parze id1 id2

[IP!]

Do tabeli Wojewodztwa wkradł się jeden VARCHAR bez 2