Skip to content

Add solution for the assignment-1#1

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

Add solution for the assignment-1#1
MikeVitvi wants to merge 1 commit intomainfrom
assignment-1

Conversation

@MikeVitvi
Copy link
Copy Markdown
Owner

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

I'm trying to add the code of an anagram checker in Python. Refactoring the exiting code to include case sensitive as option in the anagram checker function. Add comments to the code to clarify and understand and test the code.

What did you learn from the changes you have made?

I've learned or at least tried to learn how to implement an anagram checker function. How to modify the function to include case sensitive as an option. Importance of testing the function with various inputs to insure it is correct. How to structure the code with comments for better readability and understanding and how to use booleans to control the behaviour as a function based on user preferences

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?

Starting the assignments in the assignment_1.ipynb file was quite challenging for me. Before delving into the tasks, I spent some time testing and experimenting with the concepts covered in the lessons. This included understanding logical operators, exploring the OHIP Eye Exam Function, reviewing sample functions, formatting strings using escape sequences and f-strings, and grasping slicing techniques.

How were these changes tested?

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

IndentationError: expected an indented block (I faced this error)

@mentions of the person or team responsible for reviewing proposed changes (At least 2 people)

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.

Great job! Your code is well executed and commented- Nice inclusion of a docstring !

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