Skip to content

adding the anagram functions and completed assignment-1#1

Open
Issy90 wants to merge 3 commits intomainfrom
assignment-1
Open

adding the anagram functions and completed assignment-1#1
Issy90 wants to merge 3 commits intomainfrom
assignment-1

Conversation

@Issy90
Copy link
Copy Markdown
Owner

@Issy90 Issy90 commented Aug 20, 2025

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

What did you learn from the changes you have made?

I have prior programming knowledge so this was easy to complete

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

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

How were these changes tested?

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

Checklist

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

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.

You are on the right track. Your code is checking if each letter in word_a is in word_b, but you’re not removing matched letters!

This means repeated letters can cause false positives. For example: "aabb", "abbb" would return True, but it should be False. Please revise

@Issy90 Issy90 requested a review from juliagallucci August 27, 2025 22:30
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.

Good. Your functions work correctly. One note for improvement would be for simplicity and readability, you can use Python’s built-in sorted() function to compare the characters of both words instead of manually looping and replacing.

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