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

Update armstrong.c #2

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

Conversation

srinjoy07pal
Copy link

basically easy to understand for the reader if divided into several functions. Makes the code readable,precise &simple.

basically easy to understand for the reader if divided into several functions. Makes the code readable,precise &simple.
using a function makes the code more simple and easy to understand . So here is a version of the program with function
@JoDrak
Copy link

JoDrak commented Dec 27, 2023

Definitely helps to understand, it's nice to see there are people trying to contribute to the repository. I think on Chapter 3, in the exercise he uses the keyword extern but in none of the resources he provided on Chapter 3 talks about this specific keyword. This repository needs a lots of reviews it seems.

@JoDrak
Copy link

JoDrak commented Dec 27, 2023

I'll be adding information on what is missing if you or anyone reading this (as I go through while reading and practicing), whom understand better what is being covered on the repository, can try contributing to new changes to .md files to cover missing informations. NOTES.md

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.

None yet

2 participants