diff options
-rw-r--r-- | src/account-server/dalstorage.cpp | 208 | ||||
-rw-r--r-- | src/account-server/dalstorage.hpp | 295 | ||||
-rw-r--r-- | src/account-server/dalstoragesql.hpp | 7 | ||||
-rw-r--r-- | src/chat-server/partyhandler.cpp | 10 | ||||
-rw-r--r-- | src/dal/dataprovider.cpp | 10 | ||||
-rw-r--r-- | src/dal/dataprovider.h | 17 |
6 files changed, 219 insertions, 328 deletions
diff --git a/src/account-server/dalstorage.cpp b/src/account-server/dalstorage.cpp index c23ccefd..a22409d1 100644 --- a/src/account-server/dalstorage.cpp +++ b/src/account-server/dalstorage.cpp @@ -114,6 +114,11 @@ void DALStorage::close() mDb->disconnect(); } +/** + * Gets an account from a prepared SQL statement + * + * @return the account found + */ Account *DALStorage::getAccountBySQL() { try { @@ -197,6 +202,9 @@ Account *DALStorage::getAccountBySQL() /** * Get an account by user name. + * + * @param userName the owner of the account. + * @return the account associated to the user name. */ Account *DALStorage::getAccount(const std::string &userName) { @@ -211,6 +219,10 @@ Account *DALStorage::getAccount(const std::string &userName) /** * Get an account by ID. + * + * @param accountID the ID of the account. + * + * @return the account associated with the ID. */ Account *DALStorage::getAccount(int accountID) { @@ -223,6 +235,13 @@ Account *DALStorage::getAccount(int accountID) return getAccountBySQL(); } +/** + * Gets a character from a prepared SQL statement + * + * @param owner the account the character is in. + * + * @return the character found by the query. + */ Character *DALStorage::getCharacterBySQL(Account *owner) { Character *character; @@ -383,6 +402,11 @@ Character *DALStorage::getCharacterBySQL(Account *owner) /** * Gets a character by database ID. + * + * @param id the ID of the character. + * @param owner the account the character is in. + * + * @return the character associated to the ID. */ Character *DALStorage::getCharacter(int id, Account *owner) { @@ -395,6 +419,13 @@ Character *DALStorage::getCharacter(int id, Account *owner) return getCharacterBySQL(owner); } +/** + * Gets a character by character name. + * + * @param name of the character + * + * @return the character associated to the name + */ Character *DALStorage::getCharacter(const std::string &name) { std::ostringstream sql; @@ -406,6 +437,10 @@ Character *DALStorage::getCharacter(const std::string &name) return getCharacterBySQL(NULL); } +/** + * Tells if the user name already exists. + * @return true if the user name exists. + */ bool DALStorage::doesUserNameExist(const std::string &name) { try { @@ -432,7 +467,7 @@ bool DALStorage::doesUserNameExist(const std::string &name) } /** - * Tells if the email address already exists + * Tells if the email address already exists. * @return true if the email address exists. */ bool DALStorage::doesEmailAddressExist(const std::string &email) @@ -460,7 +495,7 @@ bool DALStorage::doesEmailAddressExist(const std::string &email) } /** - * Tells if the character's name already exists + * Tells if the character's name already exists. * @return true if character's name exists. */ bool DALStorage::doesCharacterNameExist(const std::string& name) @@ -487,6 +522,16 @@ bool DALStorage::doesCharacterNameExist(const std::string& name) return true; } +/** + * Updates the data for a single character, does not update the owning account + * or the characters name. Primary usage should be storing characterdata + * received from a game server. + * + * @param ptr Character to store values in the database. + * @param startTransaction set to false if this method is called as + * nested transaction. + * @return true on success + */ bool DALStorage::updateCharacter(Character *character, bool startTransaction) { @@ -685,6 +730,11 @@ bool DALStorage::updateCharacter(Character *character, /** * Save changes of a skill to the database permanently. + * + * @param character Character thats skill has changed. + * @param skill_id Identifier of the changed skill. + * + * @exception dbl::DbSqlQueryExecFailure. * @deprecated Use DALStorage::updateExperience instead!!! */ void DALStorage::flushSkill(const Character * character, int skill_id) @@ -695,6 +745,7 @@ void DALStorage::flushSkill(const Character * character, int skill_id) /** * Add an account to the database. + * @param account the new account. */ void DALStorage::addAccount(Account *account) { @@ -874,6 +925,8 @@ void DALStorage::flush(Account *account) /** * Delete an account and its associated data from the database. + * + * @param account the account to delete. */ void DALStorage::delAccount(Account *account) { @@ -889,6 +942,8 @@ void DALStorage::delAccount(Account *account) /** * Update the date and time of the last login. + * + * @param account the account that recently logged in. */ void DALStorage::updateLastLogin(const Account *account) { @@ -899,6 +954,15 @@ void DALStorage::updateLastLogin(const Account *account) mDb->execSql(sql.str()); } +/** + * Write a modification message about Character points to the database. + * + * @param CharId ID of the character + * @param CharPoints Number of character points left for the character + * @param CorrPoints Number of correction points left for the character + * @param AttribId ID of the modified attribute + * @param AttribValue New value of the modified attribute + */ void DALStorage::updateCharacterPoints(int charId, int charPoints, int corrPoints, int attribId, int attribValue) @@ -923,6 +987,12 @@ void DALStorage::updateCharacterPoints(int charId, mDb->execSql(sql.str()); } +/** + * Write a modification message about character skills to the database. + * @param CharId ID of the character + * @param SkillId ID of the skill + * @param SkillValue new skill points + */ void DALStorage::updateExperience(int charId, int skillId, int skillValue) { try @@ -969,6 +1039,12 @@ void DALStorage::updateExperience(int charId, int skillId, int skillValue) } } +/** + * Inserts a record about a status effect into the database + * @param charId ID of the character in the database + * @param statusId ID of the status effect + * @param time Time left on the status effect + */ void DALStorage::insertStatusEffect(int charId, int statusId, int time) { try @@ -991,7 +1067,7 @@ void DALStorage::insertStatusEffect(int charId, int statusId, int time) /** - * Add a guild + * Add a new guild. */ void DALStorage::addGuild(Guild *guild) { @@ -1022,7 +1098,7 @@ void DALStorage::addGuild(Guild *guild) } /** - * Remove guild + * Delete a guild. */ void DALStorage::removeGuild(Guild *guild) { @@ -1033,6 +1109,9 @@ void DALStorage::removeGuild(Guild *guild) mDb->execSql(sql.str()); } +/** + * Add member to guild. + */ void DALStorage::addGuildMember(int guildId, int memberId) { std::ostringstream sql; @@ -1053,6 +1132,9 @@ void DALStorage::addGuildMember(int guildId, int memberId) } } +/** + * Remove member from guild. + */ void DALStorage::removeGuildMember(int guildId, int memberId) { std::ostringstream sql; @@ -1072,6 +1154,9 @@ void DALStorage::removeGuildMember(int guildId, int memberId) } } +/** + * Save guild member rights. + */ void DALStorage::setMemberRights(int guildId, int memberId, int rights) { std::ostringstream sql; @@ -1092,7 +1177,8 @@ void DALStorage::setMemberRights(int guildId, int memberId, int rights) } /** - * get a list of guilds + * Get the list of guilds. + * @return a list of guilds */ std::list<Guild*> DALStorage::getGuildList() { @@ -1163,6 +1249,9 @@ std::list<Guild*> DALStorage::getGuildList() return guilds; } +/** + * Gets the value of a quest variable. + */ std::string DALStorage::getQuestVar(int id, const std::string &name) { try @@ -1187,6 +1276,12 @@ std::string DALStorage::getQuestVar(int id, const std::string &name) return std::string(); } +/** + * Gets the string value of a map specific world state variable. + * + * @param name Name of the requested world-state variable. + * @param map_id Id of the specific map. + */ std::string DALStorage::getWorldStateVar(const std::string &name, int map_id) { try @@ -1215,13 +1310,26 @@ std::string DALStorage::getWorldStateVar(const std::string &name, int map_id) return std::string(); } +/** + * Sets the value of a world state variable. + * + * @param name Name of the world-state vairable. + * @param value New value of the world-state variable. + */ void DALStorage::setWorldStateVar(const std::string &name, const std::string &value) { return setWorldStateVar(name, -1, value); } +/** + * Sets the value of a world state variable of a specific map. + * + * @param name Name of the world-state vairable. + * @param mapId ID of the specific map + * @param value New value of the world-state variable. + */ void DALStorage::setWorldStateVar(const std::string &name, - int map_id, + int mapId, const std::string &value) { try @@ -1232,9 +1340,9 @@ void DALStorage::setWorldStateVar(const std::string &name, std::ostringstream deleteStateVar; deleteStateVar << "DELETE FROM " << WORLD_STATES_TBL_NAME << " WHERE state_name = '" << name << "'"; - if (map_id >= 0) + if (mapId >= 0) { - deleteStateVar << " AND map_id = '" << map_id << "'"; + deleteStateVar << " AND map_id = '" << mapId << "'"; } deleteStateVar << ";"; mDb->execSql(deleteStateVar.str()); @@ -1248,9 +1356,9 @@ void DALStorage::setWorldStateVar(const std::string &name, << " moddate = '" << time(NULL) << "' " << " WHERE state_name = '" << name << "'"; - if (map_id >= 0) + if (mapId >= 0) { - updateStateVar << " AND map_id = '" << map_id << "'"; + updateStateVar << " AND map_id = '" << mapId << "'"; } updateStateVar << ";"; mDb->execSql(updateStateVar.str()); @@ -1266,9 +1374,9 @@ void DALStorage::setWorldStateVar(const std::string &name, insertStateVar << "INSERT INTO " << WORLD_STATES_TBL_NAME << " (state_name, map_id, value , moddate) VALUES (" << "'" << name << "', "; - if (map_id >= 0) + if (mapId >= 0) { - insertStateVar << "'" << map_id << "', "; + insertStateVar << "'" << mapId << "', "; } else { @@ -1284,6 +1392,9 @@ void DALStorage::setWorldStateVar(const std::string &name, } } +/** + * Sets the value of a quest variable. + */ void DALStorage::setQuestVar(int id, const std::string &name, const std::string &value) { @@ -1309,6 +1420,12 @@ void DALStorage::setQuestVar(int id, const std::string &name, } } +/** + * Sets a ban on an account (hence on all its characters). + * + * @param id character identifier. + * @param duration duration in minutes. + */ void DALStorage::banCharacter(int id, int duration) { try @@ -1336,6 +1453,13 @@ void DALStorage::banCharacter(int id, int duration) } } +/** + * Delete a character in the database. + * + * @param charId character identifier. + * @param startTransaction indicates wheter the function should run in + * its own transaction or is called inline of another transaction + */ void DALStorage::delCharacter(int charId, bool startTransaction = true) const { if (startTransaction) @@ -1402,12 +1526,23 @@ void DALStorage::delCharacter(int charId, bool startTransaction = true) const } } +/** + * Delete a character in the database. The object itself is not touched + * by this function! + * + * @param character character object. + * @param startTransaction indicates wheter the function should run in + * its own transaction or is called inline of another transaction + */ void DALStorage::delCharacter(Character *character, bool startTransaction = true) const { delCharacter(character->getDatabaseID(), startTransaction); } +/** + * Removes expired bans from accounts + */ void DALStorage::checkBannedAccounts() { try @@ -1426,6 +1561,12 @@ void DALStorage::checkBannedAccounts() } } +/** + * Set the level on an account. + * + * @param id The id of the account + * @param level The level to set for the account + */ void DALStorage::setAccountLevel(int id, int level) { try @@ -1442,6 +1583,12 @@ void DALStorage::setAccountLevel(int id, int level) } } +/** + * Set the level on a character. + * + * @param id The id of the character + * @param level The level to set for the character + */ void DALStorage::setPlayerLevel(int id, int level) { try @@ -1458,6 +1605,11 @@ void DALStorage::setPlayerLevel(int id, int level) } } +/** + * Store letter. + * + * @param letter The letter to store + */ void DALStorage::storeLetter(Letter *letter) { std::ostringstream sql; @@ -1512,6 +1664,11 @@ void DALStorage::storeLetter(Letter *letter) } } +/** + * Retrieve post + * + * @param playerId The id of the character requesting his post + */ Post *DALStorage::getStoredPost(int playerId) { Post *p = new Post(); @@ -1552,6 +1709,10 @@ Post *DALStorage::getStoredPost(int playerId) return p; } +/** + * Delete a letter from the database. + * @param letter The letter to delete. + */ void DALStorage::deletePost(Letter *letter) { mDb->beginTransaction(); @@ -1583,6 +1744,14 @@ void DALStorage::deletePost(Letter *letter) } } +/** + * Synchronizes the base data in the connected SQL database with the xml + * files like items.xml. + * This method is called once after initialization of DALStorage. + * Probably this function should be called if a gm requests an online + * reload of the xml files to load new items or monsters without server + * restart. + */ void DALStorage::syncDatabase() { xmlDocPtr doc = xmlReadFile(DEFAULT_ITEM_FILE, NULL, 0); @@ -1693,6 +1862,12 @@ void DALStorage::syncDatabase() xmlFreeDoc(doc); } +/** + * Sets the status of a character to online (true) or offline (false). + * + * @param charId Id of the character. + * @param online True to mark the character as being online. + */ void DALStorage::setOnlineStatus(int charId, bool online) { try @@ -1730,6 +1905,9 @@ void DALStorage::setOnlineStatus(int charId, bool online) } } +/** + * Store a transaction. + */ void DALStorage::addTransaction(const Transaction &trans) { try @@ -1752,6 +1930,9 @@ void DALStorage::addTransaction(const Transaction &trans) } } +/** + * Retrieve the last \num transactions that were stored. + */ std::vector<Transaction> DALStorage::getTransactions(unsigned int num) { std::vector<Transaction> transactions; @@ -1783,6 +1964,9 @@ std::vector<Transaction> DALStorage::getTransactions(unsigned int num) return transactions; } +/** + * Retrieve all transactions since the given \a date. + */ std::vector<Transaction> DALStorage::getTransactions(time_t date) { std::vector<Transaction> transactions; diff --git a/src/account-server/dalstorage.hpp b/src/account-server/dalstorage.hpp index f6c0e18a..cb7be468 100644 --- a/src/account-server/dalstorage.hpp +++ b/src/account-server/dalstorage.hpp @@ -25,9 +25,9 @@ #include <map> #include <vector> -#include "../dal/dataprovider.h" +#include "dal/dataprovider.h" -#include "../common/transaction.hpp" +#include "common/transaction.hpp" class Account; class Character; @@ -37,377 +37,102 @@ class Letter; class Post; /** - * A storage class that relies on DAL. + * The high level interface to the database. Through the storage you can access + * all accounts, characters, guilds, worlds states, transactions, etc. */ class DALStorage { public: - /** - * Constructor. - */ DALStorage(); - - /** - * Destructor. - */ ~DALStorage(); - /** - * Connect to the database and initialize it if necessary. - */ void open(); - - /** - * Disconnect from the database. - */ void close(); - /** - * Get an account by user name. - * - * @param userName the owner of the account. - * - * @return the account associated to the user name. - */ - Account *getAccount(const std::string& userName); - - /** - * Get an account by ID. - * - * @param accountID the ID of the account. - * - * @return the account associated with the ID. - */ + Account *getAccount(const std::string &userName); Account *getAccount(int accountID); - /** - * Gets a character by database ID. - * - * @param id the ID of the character. - * @param owner the account the character is in. - * - * @return the character associated to the ID. - */ Character *getCharacter(int id, Account *owner); - - /** - * Gets a character by character name. - * - * @param name of the character - * - * @return the character associated to the name - */ Character *getCharacter(const std::string &name); - /** - * Add a new account. - * - * @param account the new account. - */ void addAccount(Account *account); - - /** - * Delete an account. - * - * @param account the account to delete. - */ void delAccount(Account *account); - /** - * Update the date and time of the last login. - * - * @param account the account that recently logged in. - */ void updateLastLogin(const Account *account); - /** - * Write a modification message about Character points to the database. - * - * @param CharId ID of the character - * @param CharPoints Number of character points left for the character - * @param CorrPoints Number of correction points left for the character - * @param AttribId ID of the modified attribute - * @param AttribValue New value of the modified attribute - */ void updateCharacterPoints(int charId, int charPoints, int corrPoints, int attribId, int attribValue); - /** - * Write a modification message about character skills to the database. - * @param CharId ID of the character - * @param SkillId ID of the skill - * @param SkillValue new skill points - */ void updateExperience(int charId, int skillId, int skillValue); - /** - * Inserts a record about a status effect into the database - * @param charId ID of the character in the database - * @param statusId ID of the status effect - * @param time Time left on the status effect - */ void insertStatusEffect(int charId, int statusId, int time); - /** - * Sets a ban on an account (hence on all its characters). - * - * @param id character identifier. - * @param duration duration in minutes. - */ void banCharacter(int id, int duration); - /** - * Delete a character in the database. - * - * @param charId character identifier. - * @param startTransaction indicates wheter the function should run in - * its own transaction or is called inline of another transaction - */ void delCharacter(int charId, bool startTransaction) const; - - /** - * Delete a character in the database. The object itself i not touched - * by this function! - * - * @param character character object. - * @param startTransaction indicates wheter the function should run in - * its own transaction or is called inline of another transaction - */ void delCharacter(Character *character, bool startTransaction) const; - /** - * Removes expired bans from accounts - */ void checkBannedAccounts(); - /** - * Tells if the user name already exists. - * @return true if the user name exists. - */ bool doesUserNameExist(const std::string &name); - - /** - * Tells if the email address already exists. - * @return true if the email address exists. - */ bool doesEmailAddressExist(const std::string &email); - - /** - * Tells if the character name already exists. - * @return true if the character name exists. - */ bool doesCharacterNameExist(const std::string &name); - /** - * Updates the data for a single character, does not update the - * owning account or the characters name. - * Primary usage should be storing characterdata received from a - * game server. - * returns true if succefull, false otherwise. - * @param ptr Character to store values in the database. - * @param startTransaction set to false if this method is called as - * nested transaction. - * @return true on success - */ bool updateCharacter(Character *ptr, bool startTransaction = true); - /** - * Save changes of a skill to the database permanently. - * - * @param character Character thats skill has changed. - * @param skill_id Identifier of the changed skill. - * - * @exception dbl::DbSqlQueryExecFailure. - */ void flushSkill(const Character *character, int skill_id); - /** - * Add a new guild. - */ void addGuild(Guild *guild); - - /** - * Delete a guild. - */ void removeGuild(Guild *guild); - /** - * Add member to guild. - */ void addGuildMember(int guild_id, int memberId); - - /** - * Remove member from guild. - */ void removeGuildMember(int guildId, int memberId); - /** - * Save guild member rights. - */ void setMemberRights(int guildId, int memberId, int rights); - /** - * Get guild list. - * @return a list of guilds - */ std::list<Guild*> getGuildList(); - /** - * Save changes to the database permanently. - * - * @exception dal::DbSqlQueryExecFailure. - */ - void flushAll(); void flush(Account *); - /** - * Gets the value of a quest variable. - */ std::string getQuestVar(int id, const std::string &); - - /** - * Sets the value of a quest variable. - */ void setQuestVar(int id, const std::string &, const std::string &); - /** - * Gets the string value of a map specific world state variable. - * - * @param name Name of the requested world-state variable. - * @param map_id Id of the specific map. - */ std::string getWorldStateVar(const std::string &name, int map_id = -1); - - /** - * Sets the value of a world state variable. - * - * @param name Name of the world-state vairable. - * @param value New value of the world-state variable. - */ - void setWorldStateVar(const std::string &name, const std::string &value); - - /** - * Sets the value of a world state variable of a specific map. - * - * @param name Name of the world-state vairable. - * @param map_id ID of the specific map - * @param value New value of the world-state variable. - */ - void setWorldStateVar(const std::string &name, int map_id, + void setWorldStateVar(const std::string &name, + const std::string &value); + void setWorldStateVar(const std::string &name, int mapId, const std::string &value); - /** - * Set the level on an account - * - * @param id The id of the account - * @param level The level to set for the account - */ void setAccountLevel(int id, int level); - - /** - * Set the level on a character - * - * @param id The id of the character - * @param level The level to set for the character - */ void setPlayerLevel(int id, int level); - /** - * Store letter - * - * @param letter The letter to store - */ void storeLetter(Letter *letter); - - /** - * Retrieve post - * - * @param playerId The id of the character requesting his post - */ Post *getStoredPost(int playerId); - - /** - * Delete a letter from the database. - * @param letter The letter to delete. - */ void deletePost(Letter* letter); /** - * Add item to auction - * - * @param itemId The id of the item for auction - * @param player The player who put the item up for auction - * @param gold The amount of money to buy it - */ - void addAuctionItem(unsigned int itemId, int playerId, unsigned int gold); - - /** - * Gets the version of the local item database. - * - * @return Version of the item database. + * Returns the version of the local item database. */ unsigned int getItemDatabaseVersion() const { return mItemDbVersion; } - /** - * Sets the status of a character to online (true) or offline (false). - * - * @param charId Id of the character. - * @param online True to mark the character as being online. - */ void setOnlineStatus(int charId, bool online); - /** - * Store a transaction - */ void addTransaction(const Transaction &trans); - /** - * Retrieve a series of transactions - * Either based on number of transactions last saved - * or by all transactions since a date - */ std::vector<Transaction> getTransactions(unsigned int num); std::vector<Transaction> getTransactions(time_t date); private: - /** - * Copy constructor. - */ + // Prevent copying DALStorage(const DALStorage &rhs); - - - /** - * Assignment operator. - */ DALStorage &operator=(const DALStorage &rhs); - /** - * Gets an account from a prepared SQL statement - * - * @return the account found - */ Account *getAccountBySQL(); - - /** - * Gets a character from a prepared SQL statement - * - * @param owner the account the character is in. - * - * @return the character found by the query. - */ Character *getCharacterBySQL(Account *owner); - /** - * Synchronizes the base data in the connected SQL database with the xml - * files like items.xml. - * This method is called once after initialization of DALStorage. - * Probably this function should be called if a gm requests an online - * reload of the xml files to load new items or monsters without server - * restart. - */ void syncDatabase(); dal::DataProvider *mDb; /**< the data provider */ diff --git a/src/account-server/dalstoragesql.hpp b/src/account-server/dalstoragesql.hpp index 3a0f9731..bfdbc5a5 100644 --- a/src/account-server/dalstoragesql.hpp +++ b/src/account-server/dalstoragesql.hpp @@ -30,9 +30,6 @@ #error "(dalstorage.hpp) no database backend defined" #endif - -#include <string> - // TODO: Fix problem with PostgreSQL null primary key's. /** @@ -83,12 +80,12 @@ static const char *CHAR_STATUS_EFFECTS_TBL_NAME = "mana_char_status_effects"; /** * TABLE: mana_inventories. */ -static const char *INVENTORIES_TBL_NAME("mana_inventories"); +static const char *INVENTORIES_TBL_NAME = "mana_inventories"; /** * TABLE: mana_items. */ -static const char *ITEMS_TBL_NAME("mana_items"); +static const char *ITEMS_TBL_NAME = "mana_items"; /** * TABLE: mana_guilds. diff --git a/src/chat-server/partyhandler.cpp b/src/chat-server/partyhandler.cpp index 7dd9e9ca..e0f884fb 100644 --- a/src/chat-server/partyhandler.cpp +++ b/src/chat-server/partyhandler.cpp @@ -22,13 +22,13 @@ #include "chatclient.hpp" #include "party.hpp" -#include "../account-server/dalstorage.hpp" -#include "../account-server/serverhandler.hpp" +#include "account-server/dalstorage.hpp" +#include "account-server/serverhandler.hpp" -#include "../net/messagein.hpp" -#include "../net/messageout.hpp" +#include "net/messagein.hpp" +#include "net/messageout.hpp" -#include "../protocol.h" +#include "protocol.h" #include <algorithm> diff --git a/src/dal/dataprovider.cpp b/src/dal/dataprovider.cpp index febdc583..c4fd0db8 100644 --- a/src/dal/dataprovider.cpp +++ b/src/dal/dataprovider.cpp @@ -19,12 +19,12 @@ */ #include "dataprovider.h" -#include "../utils/logger.h" + +#include "utils/logger.h" namespace dal { - /** * Constructor. */ @@ -35,7 +35,6 @@ DataProvider::DataProvider() { } - /** * Destructor. */ @@ -44,9 +43,10 @@ DataProvider::~DataProvider() { } - /** * Get the connection status. + * + * @return true if connected. */ bool DataProvider::isConnected() const throw() @@ -55,7 +55,7 @@ bool DataProvider::isConnected() const } /** - * Get the DataBase Name + * Get the database name. */ std::string DataProvider::getDbName() const { diff --git a/src/dal/dataprovider.h b/src/dal/dataprovider.h index c6dc2a79..5ab21cc1 100644 --- a/src/dal/dataprovider.h +++ b/src/dal/dataprovider.h @@ -56,24 +56,12 @@ typedef enum { class DataProvider { public: - /** - * Constructor. - */ DataProvider() throw(); - /** - * Destructor. - */ - virtual - ~DataProvider() + virtual ~DataProvider() throw(); - /** - * Get the connection status. - * - * @return true if connected. - */ bool isConnected() const throw(); @@ -119,9 +107,6 @@ class DataProvider */ virtual void disconnect() = 0; - /** - * Get the Database Name. - */ std::string getDbName() const; /** |