Mirrors/CppCoreGuidelines
1
0
Fork 0
mirror of https://github.com/isocpp/CppCoreGuidelines.git synced 2025-12-17 12:44:42 +03:00
Code Issues Actions Packages Projects Releases Wiki Activity

fixed a typos because of zero-based indexing

Browse Source 
Another misunderstanding: how container can be modified (side-effect in a call of `f(&v[i])` ) if we passing only an address to element, not a address to container?
This commit is contained in:
Alexolut
2015-11-25 11:19:08 +03:00
parent ff78f166b5
commit e238597f6c
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
CppCoreGuidelines.md
View File

@@ -8247,10 +8247,10 @@ Readability. Error prevention. Efficiency.
for (int i = 1; i < v.size(); ++i) // touches two elements: can't be a range-for for (int i = 1; i < v.size(); ++i) // touches two elements: can't be a range-for
cout << v[i] + v[i-1] << '\n'; cout << v[i] + v[i-1] << '\n';
for (int i = 1; i < v.size(); ++i) // possible side-effect: can't be a range-for for (int i = 0; i < v.size(); ++i) // possible side-effect: can't be a range-for
cout << f(&v[i]) << '\n'; cout << f(&v[i]) << '\n';
for (int i = 1; i < v.size(); ++i) { // body messes with loop variable: can't be a range-for for (int i = 0; i < v.size(); ++i) { // body messes with loop variable: can't be a range-for
if (i % 2) if (i % 2)
++i; // skip even elements ++i; // skip even elements
else else