Skip to content

Initial commit of anagram checker for Python assignment 1#1

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

Initial commit of anagram checker for Python assignment 1#1
jnahmiach wants to merge 1 commit intomainfrom
assignment-1

Conversation

@jnahmiach
Copy link
Copy Markdown
Owner

TITLE: UofT-DSI | Python - Assignment 1

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

I have added Python to the anagram checker to compare two words to see if they are anagrams

What did you learn from the changes you have made?

The main lesson was applications of data types like sets. Also, was reminded about the importance of how indented things are as that caused a couple of problems when I was not careful enough.

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

I was also considering putting the two input words into a tuple so I didn't need to repeat the loop to convert each word into a set of letters. Given we only had two words, I felt that using one loop per word was simpler and more readable, but I'm actually not sure if this was the right decision?

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

My biggest challenge was modifying/transforming values and getting the expected outputs. When testing, I eventually just added a few print steps in key places so I could see the output at a certain point and debug from there vs guessing.

How were these changes tested?

I used the provided test words to ensure I saw the expected output for each.

Checklist

  • I can confirm that my changes are working as intended

Copy link
Copy Markdown

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