// ...

Month month() const; // do

int month(); // don't

// ...

{

string val;

cin >> val;

// ...

int index = 0; // bad

for (int i = 0; i < v.size(); ++i)

if (v[i] == val) {

index = i;

break;

}

// ...

char ch;

int i;

string s;

char ch2;

X& operator=(const X& a);

X(const X&);

{

if (p == nullptr) throw Nullptr_error{};

int n = strlen(p);

auto buf = new char[n];

if (buf == nullptr) throw Allocation_error{};

for (int i = 0; i < n; ++i) buf[i] = p[i];

// ... manipulate buffer ...

X x;

x.ch = 'a';

x.s = string(n); // give x.s space for *ps

for (int i = 0; i < x.s.size(); ++i) x.s[i] = buf[i]; // copy buf into x.s

delete buf;

return x;

{

X x = waste("Typical argument");

// ...

int i;

string s;

int j;

X() :i{666}, s{"qqq"} { } // j is uninitialized

X(int ii) :i{ii} {} // s is "" and j is uninitialized

// ...

int i {666};

string s {"qqq"};

int j {0};

X2() = default; // all members are initialized to their defaults

X2(int ii) :i{ii} {} // s and j initialized to their defaults

// ...

int i;

string s;

int j;

X3(int ii = 666, const string& ss = "qqq", int jj = 0)

:i{ii}, s{ss}, j{jj} { } // all members are initialized to their defaults

// ...

string s;

int i;

Foo& operator=(Foo&& a);

// ...

// ...

Vector2(Vector2&& a) { *this = a; } // just use the copy

Vector2& operator=(Vector2&& a) { *this = a; } // just use the copy

//...

T* elem;

int sz;

{

using std::swap; // make std::swap available

swap(a,b); // calls N::swap if it exists, otherwise std::swap

{

switch (n) { // good

case 0: // ...

case 7: // ...

}

std::string s;

std::string get_s() { return s; }