Skip to content

Conversation

Aliwahid17
Copy link

it will help beginner c programmers to know where to start writing code.

@siebenschlaefer
Copy link
Contributor

I'm slightly in favor of that change. armstrong-numbers seems to be the first exercise for many students, let's keep the entry barriers low.
But I don't like this two includes, the exercise can be solved with or without functions from <stdio.h> or <math.h>.
(Disclaimer: I'm a mentor, not a maintainer. I can only voice my opinion.)

@ryanplusplus
Copy link
Member

I'm slightly in favor of that change. armstrong-numbers seems to be the first exercise for many students, let's keep the entry barriers low. But I don't like this two includes, the exercise can be solved with or without functions from <stdio.h> or <math.h>. (Disclaimer: I'm a mentor, not a maintainer. I can only voice my opinion.)

I agree. I don't mind the addition of the skeleton function, but the includes aren't necessary and I think they should be removed.

Comment on lines +2 to +3
#include <stdio.h>
#include <math.h>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
#include <stdio.h>
#include <math.h>

@wolf99
Copy link
Contributor

wolf99 commented Jul 2, 2022

Is hello-world no longer the first exercise?

If this is not obvious, should there be some update to the docs?

Sorry last time I mentored was pre-v3!

@siebenschlaefer
Copy link
Contributor

Yes, hello-world is still the first one. But since it can be solved by replacing NULL with "Hello, world!", that doesn't force students to understand what functions are, where they are defined/declared, why there's no main(), etc.
I've talked to quite a few students (in mentoring sessions or on gitter) who struggled with the first "real" exercise: armstrong-numbers.
BTW: This is what the C track currently looks like:
c_track_exercises

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.

4 participants