mirror of
https://github.com/isocpp/CppCoreGuidelines.git
synced 2024-03-22 13:30:58 +08:00
Fix data member names
Rename `fx` to `x`, `fy` to `y`, `fz` to `z`, and `fe` to `e` to match names used in the narative and other member function. Fix pull request #456.
This commit is contained in:
parent
782e070fcd
commit
b4260e6fa9
|
@ -765,10 +765,13 @@ There are cases where checking early is dumb because you may not ever need the v
|
||||||
|
|
||||||
class Jet { // Physics says: e*e < x*x + y*y + z*z
|
class Jet { // Physics says: e*e < x*x + y*y + z*z
|
||||||
|
|
||||||
float fx, fy, fz, fe;
|
float x;
|
||||||
|
float y;
|
||||||
|
float z;
|
||||||
|
float e;
|
||||||
public:
|
public:
|
||||||
Jet(float x, float y, float z, float e)
|
Jet(float x, float y, float z, float e)
|
||||||
:fx(x), fy(y), fz(z), fe(e)
|
:x(x), y(y), z(z), e(e)
|
||||||
{
|
{
|
||||||
// Should I check here that the values are physically meaningful?
|
// Should I check here that the values are physically meaningful?
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue
Block a user