-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Cpp tips and tricks #1383
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
base: main
Are you sure you want to change the base?
Cpp tips and tricks #1383
Conversation
navigation.md and homepage needs to be updated? |
Yes, we need to update README.md and navigation.md. I'd also mention something of the following:
|
I agree with you, but I want to get the simple things down in an article first. |
There's still some formatting problems: some C++ keywords are not formatted, the subpoints of the I'm not convinced we should be pushing a lot of what's currently in the article: Defining The The statement that C++ is, by default, iostreams doesn't really make sense as one can avoid using them. Some of the only necessary knowledge is disabling the sync and the |
Everything has been formatted afaict. You can highlight specific places that haven't.
Yes it's hacky, so I included a note that it's not recommended. But you may encounter this when looking at other people's submissions.
I think it's important to point out the naming conflicts, which can be confusing.
You can, but my understanding is that C-style I/O is only provide compatibility with C libraries.
The explanations aren't strictly necessary, but I think it's better to explain why than just provide a template for copy-pasting. |
@adamant-pwn is the page good enough for a new article? There is a balance between quality control and not getting stuck forever. I think the info as it is currently is more beneficial published and then tweaked later than not having anything on it. |
I still think we should implement some of the suggestions from #1383 (comment). We certainly have to update |
I agree with @adamant-pwn. No offense but the article seems low effort and no time was put in to improve it. Instead there was another request to "just publish it". Formatting isn't consistent: "cout every time you read from cin". No formatting even exists on this line at quick glance. I'd prefer intro paragraphs on why we need faster io and other topics but I sort of feel at this point that there's potentially too much time reviewing relative to the effort of the article. |
Resolves #903
As always, feel free to make any comments or changes.