cppcheck/lib/checkclass.h

313 lines
11 KiB
C
Raw Normal View History

/*
* Cppcheck - A tool for static C/C++ code analysis
* Copyright (C) 2007-2010 Daniel Marjamäki and Cppcheck team.
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* the Free Software Foundation, either version 3 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*/
//---------------------------------------------------------------------------
#ifndef CheckClassH
#define CheckClassH
//---------------------------------------------------------------------------
#include "check.h"
#include "settings.h"
2010-07-26 16:46:37 +02:00
#include <map>
class Token;
/// @addtogroup Checks
/// @{
2010-01-22 19:29:24 +01:00
/** @brief %Check classes. Uninitialized member variables, non-conforming operators, missing virtual destructor, etc */
class CheckClass : public Check
{
public:
2010-01-22 19:29:24 +01:00
/** @brief This constructor is used when registering the CheckClass */
CheckClass() : Check(), hasSymbolDatabase(false)
{ }
2010-03-17 22:16:18 +01:00
/** @brief This constructor is used when running checks. */
2010-07-26 16:46:37 +02:00
CheckClass(const Tokenizer *tokenizer, const Settings *settings, ErrorLogger *errorLogger);
~CheckClass();
/** @brief Run checks on the normal token list */
void runChecks(const Tokenizer *tokenizer, const Settings *settings, ErrorLogger *errorLogger)
{
CheckClass checkClass(tokenizer, settings, errorLogger);
// can't be a simplified check .. the 'sizeof' is used.
checkClass.noMemset();
}
/** @brief Run checks on the simplified token list */
void runSimplifiedChecks(const Tokenizer *tokenizer, const Settings *settings, ErrorLogger *errorLogger)
{
CheckClass checkClass(tokenizer, settings, errorLogger);
// Coding style checks
checkClass.constructors();
checkClass.operatorEq();
checkClass.privateFunctions();
checkClass.operatorEqRetRefThis();
checkClass.thisSubtraction();
checkClass.operatorEqToSelf();
checkClass.virtualDestructor();
checkClass.checkConst();
}
2010-01-22 19:29:24 +01:00
/** @brief %Check that all class constructors are ok */
void constructors();
2010-01-22 19:29:24 +01:00
/** @brief %Check that all private functions are called */
void privateFunctions();
2010-01-22 19:29:24 +01:00
/**
* @brief %Check that the memsets are valid.
* The 'memset' function can do dangerous things if used wrong. If it
* is used on STL containers for instance it will clear all its data
* and then the STL container may leak memory or worse have an invalid state.
* It can also overwrite the virtual table.
2010-01-22 19:29:24 +01:00
* Important: The checking doesn't work on simplified tokens list.
*/
void noMemset();
2010-03-17 22:16:18 +01:00
/** @brief 'operator=' should return something. */
2010-01-22 19:29:24 +01:00
void operatorEq();
2010-01-22 19:29:24 +01:00
/** @brief 'operator=' should return reference to *this */
void operatorEqRetRefThis(); // Warning upon no "return *this;"
2010-01-22 19:29:24 +01:00
/** @brief 'operator=' should check for assignment to self */
void operatorEqToSelf(); // Warning upon no check for assignment to self
2010-01-22 19:29:24 +01:00
/** @brief The destructor in a base class should be virtual */
void virtualDestructor();
2010-01-22 19:29:24 +01:00
/** @brief warn for "this-x". The indented code may be "this->x" */
void thisSubtraction();
/** @brief can member function be const? */
void checkConst();
2010-01-22 19:29:24 +01:00
/**
* @brief Access control. This needs to be public, otherwise it
* doesn't work to compile with Borland C++
*/
enum AccessControl { Public, Protected, Private };
private:
/**
* @brief Create symbol database. For performance reasons, only call
* it if it's needed.
*/
void createSymbolDatabase();
/**
* @brief Prevent creating symbol database more than once.
*
* Initialize this flag to false in the constructors. If this flag
* is true the createSymbolDatabase should just bail out. If it is
* false the createSymbolDatabase will set it to true and create
* the symbol database.
*/
bool hasSymbolDatabase;
2010-01-22 19:29:24 +01:00
/** @brief Information about a member variable. Used when checking for uninitialized variables */
class Var
{
public:
Var(const std::string &name_, bool init_ = false, bool priv_ = false, bool mutable_ = false, bool static_ = false, bool class_ = false, Var *next_ = 0)
2010-04-15 20:08:51 +02:00
: name(name_),
init(init_),
priv(priv_),
isMutable(mutable_),
isStatic(static_),
isClass(class_),
2010-04-15 20:08:51 +02:00
next(next_)
{
}
2010-01-22 19:29:24 +01:00
/** @brief name of variable */
const std::string name;
2010-01-22 19:29:24 +01:00
/** @brief has this variable been initialized? */
bool init;
2010-01-22 19:29:24 +01:00
/** @brief is this variable declared in the private section? */
bool priv;
2010-01-22 19:29:24 +01:00
/** @brief is this variable mutable? */
bool isMutable;
/** @brief is this variable static? */
bool isStatic;
/** @brief is this variable a class (or unknown type)? */
bool isClass;
2010-01-22 19:29:24 +01:00
/** @brief next Var item */
Var *next;
2010-04-13 19:25:08 +02:00
private:
Var& operator=(const Var&); // disallow assignments
};
2010-07-26 16:46:37 +02:00
struct Func
{
enum Type { Constructor, CopyConstructor, OperatorEqual, Destructor, Function };
Func()
: tokenDef(NULL),
token(NULL),
access(Public),
hasBody(false),
isInline(false),
isConst(false),
isVirtual(false),
isPure(false),
2010-07-26 16:46:37 +02:00
isStatic(false),
isFriend(false),
isExplicit(false),
2010-07-26 16:46:37 +02:00
isOperator(false),
type(Function)
{
}
const Token *tokenDef; // function name token in class definition
const Token *token; // function name token in implementation
AccessControl access; // public/protected/private
bool hasBody; // has implementation
bool isInline; // implementation in class definition
bool isConst; // is const
bool isVirtual; // is virtual
bool isPure; // is pure virtual
2010-07-26 16:46:37 +02:00
bool isStatic; // is static
bool isFriend; // is friend
bool isExplicit; // is explicit
2010-07-26 16:46:37 +02:00
bool isOperator; // is operator
Type type; // constructor, destructor, ...
};
struct BaseInfo
{
AccessControl access; // public/protected/private
std::string name;
};
2010-07-26 16:46:37 +02:00
struct SpaceInfo
{
bool isNamespace;
std::string className;
const Token *classDef; // class/struct/namespace token
const Token *classStart; // '{' token
const Token *classEnd; // '}' token
unsigned int numConstructors;
std::list<Func> functionList;
Var *varlist;
std::vector<BaseInfo> derivedFrom;
2010-07-26 16:46:37 +02:00
SpaceInfo *nest;
AccessControl access;
};
/** @brief Information about all namespaces/classes/structrues */
std::multimap<std::string, SpaceInfo *> spaceInfoMMap;
bool argsMatch(const Token *first, const Token *second, const std::string &path, unsigned int depth) const;
2010-01-22 19:29:24 +01:00
/**
* @brief parse a scope for a constructor or member function and set the "init" flags in the provided varlist
* @param tok1 pointer to class declaration
* @param ftok pointer to the function that should be checked
* @param varlist variable list (the "init" flag will be set in these variables)
* @param callstack the function doesn't look into recursive function calls.
*/
void initializeVarList(const Token *tok1, const Token *ftok, Var *varlist, std::list<std::string> &callstack);
2010-01-22 19:29:24 +01:00
/** @brief initialize a variable in the varlist */
void initVar(Var *varlist, const std::string &varname);
2010-01-22 19:29:24 +01:00
/**
* @brief get varlist from a class definition
* @param tok1 pointer to class definition
*/
Var *getVarList(const Token *tok1);
bool isMemberVar(const std::string &classname, const std::vector<BaseInfo> &derivedFrom, const Var *varlist, const Token *tok);
bool checkConstFunc(const std::string &classname, const std::vector<BaseInfo> &derivedFrom, const Var *varlist, const Token *tok);
static const Token *initBaseInfo(const Token *tok, std::vector<BaseInfo> &derivedFrom);
/** @brief check if this function is virtual in the base classes */
bool isVirtual(const std::vector<BaseInfo> &derivedFrom, const Token *functionToken) const;
// Reporting errors..
void noConstructorError(const Token *tok, const std::string &classname, bool isStruct);
void uninitVarError(const Token *tok, const std::string &classname, const std::string &varname);
void operatorEqVarError(const Token *tok, const std::string &classname, const std::string &varname);
void unusedPrivateFunctionError(const Token *tok, const std::string &classname, const std::string &funcname);
void memsetClassError(const Token *tok, const std::string &memfunc);
void memsetStructError(const Token *tok, const std::string &memfunc, const std::string &classname);
void operatorEqReturnError(const Token *tok);
void virtualDestructorError(const Token *tok, const std::string &Base, const std::string &Derived);
void thisSubtractionError(const Token *tok);
void operatorEqRetRefThisError(const Token *tok);
void operatorEqToSelfError(const Token *tok);
void checkConstError(const Token *tok, const std::string &classname, const std::string &funcname);
void checkConstError2(const Token *tok1, const Token *tok2, const std::string &classname, const std::string &funcname);
void getErrorMessages()
{
noConstructorError(0, "classname", false);
uninitVarError(0, "classname", "varname");
operatorEqVarError(0, "classname", "");
unusedPrivateFunctionError(0, "classname", "funcname");
memsetClassError(0, "memfunc");
memsetStructError(0, "memfunc", "classname");
operatorEqReturnError(0);
virtualDestructorError(0, "Base", "Derived");
thisSubtractionError(0);
operatorEqRetRefThisError(0);
operatorEqToSelfError(0);
checkConstError(0, "class", "function");
}
std::string name() const
{
return "Class";
}
std::string classInfo() const
{
return "Check the code for each class.\n"
"* Missing constructors\n"
"* Are all variables initialized by the constructors?\n"
"* [[CheckMemset|Warn if memset, memcpy etc are used on a class]]\n"
"* If it's a base class, check that the destructor is virtual\n"
"* Are there unused private functions\n"
"* 'operator=' should return reference to self\n"
"* 'operator=' should check for assignment to self\n"
"* Constness for member functions\n";
}
};
/// @}
//---------------------------------------------------------------------------
#endif