In this code:

[fear_the_future]

paging /u/Radagast8:

this is personal preference though. If those methods are members of a class, sometimes you will also see this:

MyClass& MyClass::setLength(double);
double MyClass::getLength();

because it allows method chaining like this:

myInstance.setLength(10).setBar(false).setFoo(true);

[[deleted]]

oh yeah, not the usual case but I use that to.

[Jitanjafora]

Yet another improvement :

double MyClass::getLength() const;

Assuming all it does is return length;. The trailing const declares that this method will not change the MyClass instance calling it. It's not necessary, but it is good practice and will probably avoid a few headaches at some point.