Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Article: Rust Refactoring for Beginners #1

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

michalfita
Copy link

@michalfita michalfita commented Jul 6, 2023

Purpose

This PR is illustration of my article "Rust Refactoring for Beginners" on Medium.

The article teaches how to properly use  type at basic level.
Removal of the infinite loop for sake of the counter used in the program.
Demonstation of encapsulation and drying code.
@neerajbg
Copy link
Owner

Hi Michal,

Thanks for the updated code. Unfortunately this PR is having conflicts. Actually I had also updated my code after few more learnings like let else block.

However I will incorporate your suggestion with some more learnings, next is struct functions and modules.

Thanks for your help and support. This is really helping me a lot to learn this great language.

Neeraj

@michalfita
Copy link
Author

Commits of this PR illustrate this article if that helps:
https://betterprogramming.pub/rust-refactoring-for-beginners-15a3270ce45d
I don't have time to solve your conflicts at the moment. Hopefully, after you read the article you'd manage to incorporate most of it into your code.

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