Codacy Quality Check

Posted 2020-10-30

Codacy Quality CheckList

• Variable ‘var‘ is reassigned a value before the old one has been used.

  double scaleFactor = 1.0;
  UnitTypeEnum bodyLenUnit = kUnitType_NULL;
  object->LengthUnit(bodyLenUnit);
  scaleFactor = ParasolidBridgeUtility::UnitScaleFactor(bodyLenUnit);

• Prefer prefix ++/-- operators for non-primitive types.

  for (iter=innerLoopVec.begin(); innerLoopVec.end()!=iter; iter++)
  {
  ...
  }

• Assignment of function parameter has no effect outside the function.

  void Foo(A* pA) 
  {
  delete pA;
  pA = nullptr;
  }

• Class ‘XXX‘ has a constructor with 1 argument that is not explicit.

  class A
  {
  public:
  A(int){} //default contruct won‘t be provided
  }；

• The scope of the variable ‘var‘ can be reduced.

• Array index ‘j‘ is used before limits check.

  while ( name.Text[j] != ‘>‘ && j < name.TextLength )
  {
      ...
  }