libpointmatcher icon indicating copy to clipboard operation
libpointmatcher copied to clipboard

Unclear convention for input variable names

Open pomerlef opened this issue 10 years ago • 1 comments

For now, we use the same input variable name as the member name, especially in the constructor. This might causes some problem on the the long run.

Problematic example:

class foo
{
  int variableName;

  //constructor
  foo(int variableName);
};
foo::foo(int variableName):
  variableName(variableName)
{}

Possible solution: add the suffix _in to the input variables:

foo::foo(int variableName_in):
  variableName(variableName_in)
{}

pomerlef avatar Mar 17 '16 18:03 pomerlef

We need an opinion on that...

pomerlef avatar Oct 10 '18 20:10 pomerlef