Skip to content

complete assignment 1 anagram checker#1

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

complete assignment 1 anagram checker#1
ganzichuan wants to merge 1 commit intomainfrom
assignment-1

Conversation

@ganzichuan
Copy link
Owner

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

I completed Assignment 1 by updating assignment_1.ipynb with an anagram checker. In Part 1, I wrote a function that checks whether two words are anagrams without considering uppercase and lowercase differences. In Part 2, I expanded the function by adding an is_case_sensitive boolean parameter so the program can check anagrams either with or without case sensitivity.

What did you learn from the changes you have made?

From this assignment, I learned how to define and modify Python functions, how to use string methods such as .lower(), and how sorting letters can be used to compare two words for anagrams. I also learned that in a Jupyter Notebook.

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

not really

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

not sure

How were these changes tested?

I tested the code using the required sample inputs from the assignment instructions:
anagram_checker("Silent", "listen") returned True
anagram_checker("Silent", "Night") returned False
anagram_checker("night", "Thing") returned True
anagram_checker("Silent", "listen", False) returned True
anagram_checker("Silent", "listen", True) returned False
anagram_checker("Silent", "Listen", True) returned False
I also reran the notebook cells to confirm that the updated function worked correctly in both parts.

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

Checklist

  • [X ] I can confirm that my changes are working as intended

Copy link

@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.

Good!

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