-
-
Notifications
You must be signed in to change notification settings - Fork 632
Add Josephus Problem in C++ (Fixes #3332) #5142
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
Add Josephus Problem in C++ (Fixes #3332) #5142
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @apurvavats! Thanks for contributing to this project! We are a rather small team, so it may take some time to process this request. In the meantime, there are several ways you can make yourself a part of The Renegade Coder community. For instance, you can:
- Subscribe to the monthly newsletter
- Become a patron
- Join us on Discord
- And, check out the rest of the list which includes links to The Renegade Coder YouTube channel and Twitter account
Thanks for your help!
rzuckerm
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please take a look at the project description for the appropriate usage message.
|
I did the required changes, please review it |
rzuckerm
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What you had before was a lot closer than what you have now. All that needed to be done was to fix the usage message. Also, I'd recommend assigning the usage message to a variable so that you don't have to keep repeating that string.
|
updated the message, please review again. |
|
After checking the same program in different languages what i concluded is the solution requires only the final output and not not the output after every iteration, so please confirm once what are the requirements. |
The output is just supposed to be the number of the person remaining. |
rzuckerm
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@apurvavats Thanks for your contribution! After looking at the Description section of this, I can see how that would be confusing. However, the Requirements and Testing section clarify what the expected output is. I'll take this up with the founder of this repo and see what he says.
|
@rzuckerm can you please add the label hactoberfest accepted |
The whole repo is marked with the "hacktoberfest" label, so it should count. |
|
Thank you so much |
|
@apurvavats The Josephus Problem description has been updated. |
|
@rzuckerm it still does not appear in my hactoberfest profile. |
|
@apurvavats Sorry, I don't know what to tell you. According to the Hacktobestfest rules:
Our repo is tagged with the |
Congrats on taking the first step to contributing to the Sample Programs repository maintained by The Renegade Coder!
For simplicity, please make sure that your pull request includes one and only one contribution.
Please fill one of the sections below as applicable.
Please also add any other relevant information to the Notes section at the bottom.
You may delete or just ignore any other sections.
For more information please refer to our contributing documentation
I Am Adding a New Code Snippet in an Existing Language
Add { Josephus Problem} in {C++}format