Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ALEX P Week3 Databases #19

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

op117
Copy link

@op117 op117 commented Jan 10, 2025

No description provided.

Copy link

@dyaboykyl dyaboykyl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

Save all answers in a text file / MD file.

1. What columns violate 1NF?
- the columns food_code and food_description violate 1NF because they contain non-atomic values.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor: non-atomic is probably not the term you're looking for. Maybe non-singular?

- Dinners: information about dinner events (dinner_id, dinner_date, venue_code, venue_description).
- Venues: information about venues (venue_code, venue_description).
- Foods: store information about food (food_code, food_description).
- Dinner_Foods: relationship between dinners and food (dinner_id, food_code).

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor: A relationship is not necessarily an entity


2. What entities do you recognize that could be extracted?
- Members: store information about club members (member_id, member_name, member_address).
- Dinners: information about dinner events (dinner_id, dinner_date, venue_code, venue_description).

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor: the dinner the entity should only describe information about the dinner (venue_description describes the venue)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants