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

code refactor and Improvements #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Dhaxor
Copy link

@Dhaxor Dhaxor commented Dec 22, 2022

I made some refactor to the code to help anybody that might want to contribute to it and also made some improvements to it.
Here are a list of things I did

  • Use #pragma once instead of including header files multiple times. This will prevent issues with multiple inclusion and make the code more readable.

  • Use more descriptive variable names. For example, instead of using i for the loop variable, use a more descriptive name like arg_index. This will make the code easier to understand.

  • Added comments to the code to explain what each function does and why it is needed. This will make the code easier to understand and maintain.

  • Add error checking to the calls to atof and atoi to make sure that the input is valid.

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.

1 participant