Skip to content

Merge add bd structure#5

Closed
zephyr0021 wants to merge 2 commits intomainfrom
add-database
Closed

Merge add bd structure#5
zephyr0021 wants to merge 2 commits intomainfrom
add-database

Conversation

@zephyr0021
Copy link
Owner

No description provided.

@YrioTries
Copy link

YrioTries commented Apr 17, 2025

Привет, в целом всё отлично, но есть одно уточнение:

Присутствует небольшая избыточность в таблице film_genre, там есть первичный ключ id который никак не используется. Можно сделать первичным ключом сочетание film_id и genre_id (PK из (film_id, genre_id)) тем самым убрать избыточность

Единственное у тебя два состояния дружбы либо да, либо нет. Можно было бы потупить как с жанрами, где была бы отдельная таблица состояний дружбы. Но это на твоё усмотрение, потому что в целом такое решение даже выгоднее, при наличие только 2 состояний.

Не знаю на сколько обязательно тз проверять, наличие ссылки на таблицу и несколько примеров sql-запросов в readme, но это уже так, фантики.

Подумай по поводу избыточности, а так всё отлично

@zephyr0021 zephyr0021 changed the title Merge add-database Merge add bd structure Apr 18, 2025
@zephyr0021 zephyr0021 closed this Apr 18, 2025
@zephyr0021 zephyr0021 deleted the add-database branch April 18, 2025 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants