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

Refactor C++ roadmap: Consolidate operator discussions and introduce control flow section #5998

Open
whoismtrx opened this issue Jul 1, 2024 · 7 comments
Labels
cpp Roadmap: cpp reviewed request reviewed and will potentially be acted on during roadmap migration. topic-change Missing or deprecated topics in roadmap

Comments

@whoismtrx
Copy link
Contributor

whoismtrx commented Jul 1, 2024

Roadmap URL

https://roadmap.sh/cpp

Suggestions

Hey everyone! I hope you're all doing well. Currently, I'm reviewing a C++ roadmap and I noticed something.

  • Observation: In the basic operations section, there are topics such as arithmetic operators, logical operators, bitwise operators, and loops for/while. However, there is another section called functions that covers operators and lambdas.
  • Concern: It seems like the operators mentioned in the functions section are the same as the ones discussed in the basic operations section.
  • Suggestion: It would be better to discuss overloading operators in the functions section instead of duplicating the content.
  • addition: Create a new section called control flow & statements to cover loops like for/while/do while and conditions like if/else/switch/go to.
  • removals: Remove the duplicate content about operators from the functions section. Move the discussion about loops from the operator section to the appropriate control flow and statements section.
@whoismtrx whoismtrx added the topic-change Missing or deprecated topics in roadmap label Jul 1, 2024
@github-actions github-actions bot added the cpp Roadmap: cpp label Jul 1, 2024
@0x00Jeff
Copy link

I think he got a point

@whoismtrx
Copy link
Contributor Author

Hey @dansholds! It's been about three weeks since I opened this issue, and I haven't received any feedback yet. I still believe it's crucial to fix this as soon as possible for people who are interested in learning C++. I hope we can refactor the roadmap soon. Thank you!

@dansholds
Copy link
Collaborator

@whoismtrx All the roadmaps are currently going through migration to the new Editor, cpp roadmap is one of them, so this can be looked at/actioned when the time comes 👍

@dansholds dansholds added the reviewed request reviewed and will potentially be acted on during roadmap migration. label Jul 22, 2024
@dansholds
Copy link
Collaborator

In the meantime @whoismtrx Can you edit your initial post to be more structured rather than a paragraph such as addition, removals etc in bullet point format so it's easier to parse and understand when it comes to implement.

@whoismtrx
Copy link
Contributor Author

Hey @dansholds, can you please take a look at the changes I made to my initial post and let me know your thoughts?

@dansholds
Copy link
Collaborator

@whoismtrx all good

@Steindvart
Copy link
Contributor

I join to the suggestions @whoismtrx. These corrections will improve the quality of the C++ roadmap and will make it more logical.

Since the topic of "functions" was touched upon in this discussion, I want also to make my suggestion about this.

  • Suggestion: It is worth moving the section "Static Polymorphism", which includes "Function Overloading", from "OOP" topic to the section "functions".

  • Why: Because these do not relate directly to classical OOP concepts (such as inheritance, encapsulation, and dynamic polymorphism). More broadly, C++ provides and supports several paradigms, including procedural programming, OOP, and metaprogramming, and elements of each of these paradigms can be used together. However, if static polymorphism is used in OOP classes, this does not make static polymorphism of function part of the OOP concept.

@dansholds

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cpp Roadmap: cpp reviewed request reviewed and will potentially be acted on during roadmap migration. topic-change Missing or deprecated topics in roadmap
Projects
None yet
Development

No branches or pull requests

4 participants