Skip to content

Completed Assignment 1, section 1-3#1

Open
elioc1341 wants to merge 1 commit intomainfrom
assignment-one
Open

Completed Assignment 1, section 1-3#1
elioc1341 wants to merge 1 commit intomainfrom
assignment-one

Conversation

@elioc1341
Copy link
Owner

## What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

  • The first change was to add a new file into the repository (Assignment 1, with PNG file)
  • The second change was to update the SQL code from the assignment

## What did you learn from the changes you have made?

  • learned different functions of SQL (filter, aggregation, condtion, etc.), execution order
  • database designs, and relation types, and how they can be used together

## Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

  • For questions like WHERE and DATE sections, I thought it might make sense to join the customer table so it's more intuitive for users to understand the data. customer_id is not as intuitive for everyday understanding.

## Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

  • Was a little confused about certain questions for example the WHERE question asked for product_id = 4 or 9, but the tables didn't have product_id = 9. I had to double check with other queries and the database itself to make sure I was querying the correct information
  • I know Section 3 isn't required this week, but wanted to try it out. I had to google how to handle dates in SQLLite, since the traditional date functions weren't working. The strftime function was really helpful, and just learned to treat the date questions as text.

## How were these changes tested?

  • All of the changes in SQL could freely be tested since they were all select statements.
  • As a failsafe check I would also put the data into Excel and pivot tables just to double check the math was working out.

## A reference to a related issue in your repository (if applicable)
N/A

## Checklist

  • I can confirm that my changes are working as intended

Copy link

@JzfL JzfL left a comment

Choose a reason for hiding this comment

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

Great work 30/30!

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