diff --git a/.gitignore b/.gitignore index f22bfa2..e6ea691 100644 --- a/.gitignore +++ b/.gitignore @@ -4,6 +4,7 @@ build example1 *.a +.vs/ .vscode/ /SQLiteCpp.sln *.ncb diff --git a/include/SQLiteCpp/Backup.h b/include/SQLiteCpp/Backup.h index 21ad662..ea6f71f 100644 --- a/include/SQLiteCpp/Backup.h +++ b/include/SQLiteCpp/Backup.h @@ -14,6 +14,7 @@ #include #include +#include // Forward declaration to avoid inclusion of in a header struct sqlite3_backup; @@ -114,13 +115,12 @@ public: int executeStep(const int aNumPage = -1); /// Return the number of source pages still to be backed up as of the most recent call to executeStep(). - int getRemainingPageCount(); + int getRemainingPageCount() const; /// Return the total number of pages in the source database as of the most recent call to executeStep(). - int getTotalPageCount(); + int getTotalPageCount() const; private: - // TODO: use std::unique_ptr with a custom deleter to call sqlite3_backup_finish() sqlite3_backup* mpSQLiteBackup = nullptr; ///< Pointer to SQLite Database Backup Handle }; diff --git a/include/SQLiteCpp/Column.h b/include/SQLiteCpp/Column.h index d9e7c34..0a025d5 100644 --- a/include/SQLiteCpp/Column.h +++ b/include/SQLiteCpp/Column.h @@ -55,10 +55,6 @@ public: */ explicit Column(const Statement::TStatementPtr& aStmtPtr, int aIndex); - // default destructor: the finalization will be done by the destructor of the last shared pointer - // default copy constructor and assignment operator are perfectly suited : - // they copy the Statement::Ptr which in turn increments the reference counter. - /** * @brief Return a pointer to the named assigned to this result column (potentially aliased) * diff --git a/include/SQLiteCpp/Database.h b/include/SQLiteCpp/Database.h index 3f55e50..01b6de9 100644 --- a/include/SQLiteCpp/Database.h +++ b/include/SQLiteCpp/Database.h @@ -92,8 +92,8 @@ extern const int OPEN_NOFOLLOW; // SQLITE_OPEN_NOFOLLOW extern const int OK; ///< SQLITE_OK (used by check() bellow) -extern const char* VERSION; ///< SQLITE_VERSION string from the sqlite3.h used at compile time -extern const int VERSION_NUMBER; ///< SQLITE_VERSION_NUMBER from the sqlite3.h used at compile time +extern const char* const VERSION; ///< SQLITE_VERSION string from the sqlite3.h used at compile time +extern const int VERSION_NUMBER; ///< SQLITE_VERSION_NUMBER from the sqlite3.h used at compile time /// Return SQLite version string using runtime call to the compiled library const char* getLibVersion() noexcept; @@ -342,7 +342,7 @@ public: * * @return the sqlite result code. */ - int tryExec(const std::string aQueries) noexcept + int tryExec(const std::string& aQueries) noexcept { return tryExec(aQueries.c_str()); } @@ -403,7 +403,7 @@ public: * * @throw SQLite::Exception in case of error */ - bool tableExists(const char* apTableName); + bool tableExists(const char* apTableName) const; /** * @brief Shortcut to test if a table exists. @@ -416,7 +416,7 @@ public: * * @throw SQLite::Exception in case of error */ - bool tableExists(const std::string& aTableName) + bool tableExists(const std::string& aTableName) const { return tableExists(aTableName.c_str()); } @@ -566,7 +566,7 @@ public: static Header getHeaderInfo(const std::string& aFilename); // Parse SQLite header data from a database file. - Header getHeaderInfo() + Header getHeaderInfo() const { return getHeaderInfo(mFilename); } diff --git a/include/SQLiteCpp/Savepoint.h b/include/SQLiteCpp/Savepoint.h index d389720..681cde0 100644 --- a/include/SQLiteCpp/Savepoint.h +++ b/include/SQLiteCpp/Savepoint.h @@ -66,7 +66,7 @@ class Savepoint { * Exception is thrown in case of error, then the Savepoint is NOT * initiated. */ - Savepoint(Database& aDatabase, std::string name); + Savepoint(Database& aDatabase, const std::string& name); // Savepoint is non-copyable Savepoint(const Savepoint&) = delete; @@ -88,8 +88,8 @@ class Savepoint { void rollback(); private: - Database& mDatabase; ///< Reference to the SQLite Database Connection - std::string msName; ///< Name of the Savepoint - bool mbReleased; ///< True when release has been called + Database& mDatabase; ///< Reference to the SQLite Database Connection + std::string msName; ///< Name of the Savepoint + bool mbReleased = false; ///< True when release has been called }; } // namespace SQLite diff --git a/include/SQLiteCpp/Statement.h b/include/SQLiteCpp/Statement.h index 704d7de..ce95893 100644 --- a/include/SQLiteCpp/Statement.h +++ b/include/SQLiteCpp/Statement.h @@ -74,18 +74,14 @@ public: Statement(aDatabase, aQuery.c_str()) {} - /** - * @brief Move an SQLite statement. - * - * @param[in] aStatement Statement to move - */ - Statement(Statement&& aStatement) noexcept; - Statement& operator=(Statement&& aStatement) noexcept = default; - // Statement is non-copyable Statement(const Statement&) = delete; Statement& operator=(const Statement&) = delete; + Statement(Statement&& aStatement) noexcept; + Statement& operator=(Statement&& aStatement) noexcept = default; + // TODO: Change Statement move constructor to default + /// Finalize and unregister the SQL query from the SQLite Database Connection. /// The finalization will be done by the destructor of the last shared pointer ~Statement() = default; @@ -702,12 +698,12 @@ private: std::string mQuery; //!< UTF-8 SQL Query sqlite3* mpSQLite; //!< Pointer to SQLite Database Connection Handle TStatementPtr mpPreparedStatement; //!< Shared Pointer to the prepared SQLite Statement Object - int mColumnCount{0}; //!< Number of columns in the result of the prepared statement - bool mbHasRow{false}; //!< true when a row has been fetched with executeStep() - bool mbDone{false}; //!< true when the last executeStep() had no more row to fetch + int mColumnCount = 0; //!< Number of columns in the result of the prepared statement + bool mbHasRow = false; //!< true when a row has been fetched with executeStep() + bool mbDone = false; //!< true when the last executeStep() had no more row to fetch /// Map of columns index by name (mutable so getColumnIndex can be const) - mutable std::map mColumnNames; + mutable std::map> mColumnNames; }; diff --git a/include/SQLiteCpp/Transaction.h b/include/SQLiteCpp/Transaction.h index d7f26ca..a2198f9 100644 --- a/include/SQLiteCpp/Transaction.h +++ b/include/SQLiteCpp/Transaction.h @@ -87,8 +87,8 @@ public: void commit(); private: - Database& mDatabase; ///< Reference to the SQLite Database Connection - bool mbCommited; ///< True when commit has been called + Database& mDatabase; ///< Reference to the SQLite Database Connection + bool mbCommited = false; ///< True when commit has been called }; diff --git a/src/Backup.cpp b/src/Backup.cpp index 183d314..4c91e69 100644 --- a/src/Backup.cpp +++ b/src/Backup.cpp @@ -69,15 +69,16 @@ int Backup::executeStep(const int aNumPage /* = -1 */) } // Get the number of remaining source pages to be copied in this backup process -int Backup::getRemainingPageCount() +int Backup::getRemainingPageCount() const { return sqlite3_backup_remaining(mpSQLiteBackup); } // Get the number of total source pages to be copied in this backup process -int Backup::getTotalPageCount() +int Backup::getTotalPageCount() const { return sqlite3_backup_pagecount(mpSQLiteBackup); } + } // namespace SQLite diff --git a/src/Column.cpp b/src/Column.cpp index 108f7f9..0ba512d 100644 --- a/src/Column.cpp +++ b/src/Column.cpp @@ -78,7 +78,7 @@ double Column::getDouble() const noexcept const char* Column::getText(const char* apDefaultValue /* = "" */) const noexcept { auto pText = reinterpret_cast(sqlite3_column_text(mStmtPtr.get(), mIndex)); - return (pText?pText:apDefaultValue); + return (pText ? pText : apDefaultValue); } // Return a pointer to the blob value (*not* NULL terminated) of the column specified by its index starting at 0 diff --git a/src/Database.cpp b/src/Database.cpp index 88f63c1..30479b9 100644 --- a/src/Database.cpp +++ b/src/Database.cpp @@ -27,6 +27,7 @@ namespace SQLite { +const int OK = SQLITE_OK; const int OPEN_READONLY = SQLITE_OPEN_READONLY; const int OPEN_READWRITE = SQLITE_OPEN_READWRITE; const int OPEN_CREATE = SQLITE_OPEN_CREATE; @@ -42,10 +43,8 @@ const int OPEN_NOFOLLOW = SQLITE_OPEN_NOFOLLOW; const int OPEN_NOFOLLOW = 0; #endif -const int OK = SQLITE_OK; - -const char* VERSION = SQLITE_VERSION; -const int VERSION_NUMBER = SQLITE_VERSION_NUMBER; +const char* const VERSION = SQLITE_VERSION; +const int VERSION_NUMBER = SQLITE_VERSION_NUMBER; // Return SQLite version string using runtime call to the compiled library const char* getLibVersion() noexcept @@ -142,7 +141,7 @@ Column Database::execAndGet(const char* apQuery) } // Shortcut to test if a table exists. -bool Database::tableExists(const char* apTableName) +bool Database::tableExists(const char* apTableName) const { Statement query(*this, "SELECT count(*) FROM sqlite_master WHERE type='table' AND name=?"); query.bind(1, apTableName); @@ -439,8 +438,8 @@ void Database::backup(const char* apFilename, BackupType aType) Database otherDatabase(apFilename, SQLite::OPEN_READWRITE | SQLite::OPEN_CREATE); // For a 'Save' operation, data is copied from the current Database to the other. A 'Load' is the reverse. - Database& src = (aType == Save ? *this : otherDatabase); - Database& dest = (aType == Save ? otherDatabase : *this); + Database& src = (aType == BackupType::Save ? *this : otherDatabase); + Database& dest = (aType == BackupType::Save ? otherDatabase : *this); // Set up the backup procedure to copy between the "main" databases of each connection Backup bkp(dest, src); diff --git a/src/Savepoint.cpp b/src/Savepoint.cpp index b3d13a2..2475a09 100644 --- a/src/Savepoint.cpp +++ b/src/Savepoint.cpp @@ -18,8 +18,8 @@ namespace SQLite { // Begins the SQLite savepoint -Savepoint::Savepoint(Database& aDatabase, std::string aName) - : mDatabase(aDatabase), msName(aName), mbReleased(false) { +Savepoint::Savepoint(Database& aDatabase, const std::string& aName) + : mDatabase(aDatabase), msName(aName) { // workaround because you cannot bind to SAVEPOINT // escape name for use in query Statement stmt(mDatabase, "SELECT quote(?)"); diff --git a/src/Statement.cpp b/src/Statement.cpp index 64e16d2..bef22b0 100644 --- a/src/Statement.cpp +++ b/src/Statement.cpp @@ -275,7 +275,7 @@ int Statement::getColumnIndex(const char* apName) const for (int i = 0; i < mColumnCount; ++i) { const char* pName = sqlite3_column_name(getPreparedStatement(), i); - mColumnNames[pName] = i; + mColumnNames.emplace(pName, i); } } @@ -288,7 +288,7 @@ int Statement::getColumnIndex(const char* apName) const return iIndex->second; } -const char * Statement::getColumnDeclaredType(const int aIndex) const +const char* Statement::getColumnDeclaredType(const int aIndex) const { checkIndex(aIndex); const char * result = sqlite3_column_decltype(getPreparedStatement(), aIndex); diff --git a/src/Transaction.cpp b/src/Transaction.cpp index 2a8857a..abfb9b8 100644 --- a/src/Transaction.cpp +++ b/src/Transaction.cpp @@ -21,8 +21,7 @@ namespace SQLite // Begins the SQLite transaction Transaction::Transaction(Database& aDatabase, TransactionBehavior behavior) : - mDatabase(aDatabase), - mbCommited(false) + mDatabase(aDatabase) { const char *stmt; switch (behavior) { @@ -43,8 +42,7 @@ Transaction::Transaction(Database& aDatabase, TransactionBehavior behavior) : // Begins the SQLite transaction Transaction::Transaction(Database &aDatabase) : - mDatabase(aDatabase), - mbCommited(false) + mDatabase(aDatabase) { mDatabase.exec("BEGIN"); }