Skip to content

assignment_1 completed#1

Open
StephanieShishis wants to merge 1 commit intomainfrom
assignment-1
Open

assignment_1 completed#1
StephanieShishis wants to merge 1 commit intomainfrom
assignment-1

Conversation

@StephanieShishis
Copy link
Copy Markdown
Owner

TITLE: UofT-DSI | Python - Assignment 1

What changes are you trying to make?

In this assignment I built the base anagram checker using a function and expanded the functionality through the use of booleans and if/else statements.

What did you learn from the changes you have made?

I learned that you can separate the functions and methods to run separately with if/else statements based on different conditions (for example case sensitivity) vs just committing to a full chunk of code that only does one or the other.

Was there another approach you were thinking about making?

I was thinking of creating two new variables that were sorted and then use the == in Part 1 but it was a little redundant.

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

In Part 1, at first I did not use the .lower() method so the code was coming back as all false. I realized it was picking up on the case sensitivity and this had to be integrated with the sorted() function.

How were these changes tested?

I ran the code to see if it aligned with the True/False values expected.

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

Checklist

  • [YES] Created a branch with the correct naming convention.
  • [YES] Ensured that the repository is public.
  • [YES] Reviewed the PR description guidelines and adhered to them.
  • [YES] Verify that the link is accessible in a private browser window.

Copy link
Copy Markdown

@juliagallucci juliagallucci left a comment

Choose a reason for hiding this comment

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

Excellent!

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