From 3efda43abd11979adfd048fc62fe4f09e702f772 Mon Sep 17 00:00:00 2001 From: Andrei Karas Date: Wed, 10 Apr 2013 14:57:39 +0300 Subject: improve npchandler class. --- src/net/eathena/npchandler.cpp | 42 +++++++++++++++++++++++------------------- src/net/eathena/npchandler.h | 33 +++++++++++++++++++-------------- 2 files changed, 42 insertions(+), 33 deletions(-) (limited to 'src/net/eathena') diff --git a/src/net/eathena/npchandler.cpp b/src/net/eathena/npchandler.cpp index 530ba0416..e2d5a003a 100644 --- a/src/net/eathena/npchandler.cpp +++ b/src/net/eathena/npchandler.cpp @@ -112,20 +112,20 @@ void NpcHandler::handleMessage(Net::MessageIn &msg) mDialog = nullptr; } -void NpcHandler::talk(int npcId) +void NpcHandler::talk(const int npcId) const { MessageOut outMsg(CMSG_NPC_TALK); outMsg.writeInt32(npcId); outMsg.writeInt8(0); // Unused } -void NpcHandler::nextDialog(int npcId) +void NpcHandler::nextDialog(const int npcId) const { MessageOut outMsg(CMSG_NPC_NEXT_REQUEST); outMsg.writeInt32(npcId); } -void NpcHandler::closeDialog(int npcId) +void NpcHandler::closeDialog(const int npcId) { MessageOut outMsg(CMSG_NPC_CLOSE); outMsg.writeInt32(npcId); @@ -133,29 +133,30 @@ void NpcHandler::closeDialog(int npcId) const NpcDialogs::iterator it = mNpcDialogs.find(npcId); if (it != mNpcDialogs.end()) { - if ((*it).second.dialog) - (*it).second.dialog->close(); - if ((*it).second.dialog == mDialog) + NpcDialog *const dialog = (*it).second.dialog; + if (dialog) + dialog->close(); + if (dialog == mDialog) mDialog = nullptr; mNpcDialogs.erase(it); } } -void NpcHandler::listInput(int npcId, unsigned char value) +void NpcHandler::listInput(const int npcId, const unsigned char value) const { MessageOut outMsg(CMSG_NPC_LIST_CHOICE); outMsg.writeInt32(npcId); outMsg.writeInt8(value); } -void NpcHandler::integerInput(int npcId, int value) +void NpcHandler::integerInput(const int npcId, const int value) const { MessageOut outMsg(CMSG_NPC_INT_RESPONSE); outMsg.writeInt32(npcId); outMsg.writeInt32(value); } -void NpcHandler::stringInput(int npcId, const std::string &value) +void NpcHandler::stringInput(const int npcId, const std::string &value) const { MessageOut outMsg(CMSG_NPC_STR_RESPONSE); outMsg.writeInt16(static_cast(value.length() + 9)); @@ -164,22 +165,23 @@ void NpcHandler::stringInput(int npcId, const std::string &value) outMsg.writeInt8(0); // Prevent problems with string reading } -void NpcHandler::buy(int beingId) +void NpcHandler::buy(const int beingId) const { MessageOut outMsg(CMSG_NPC_BUY_SELL_REQUEST); outMsg.writeInt32(beingId); outMsg.writeInt8(0); // Buy } -void NpcHandler::sell(int beingId) +void NpcHandler::sell(const int beingId) const { MessageOut outMsg(CMSG_NPC_BUY_SELL_REQUEST); outMsg.writeInt32(beingId); outMsg.writeInt8(1); // Sell } -void NpcHandler::buyItem(int beingId A_UNUSED, int itemId, - unsigned char color A_UNUSED, int amount) +void NpcHandler::buyItem(const int beingId A_UNUSED, const int itemId, + const unsigned char color A_UNUSED, + const int amount) const { MessageOut outMsg(CMSG_NPC_BUY_REQUEST); outMsg.writeInt16(8); // One item (length of packet) @@ -187,7 +189,8 @@ void NpcHandler::buyItem(int beingId A_UNUSED, int itemId, outMsg.writeInt16(static_cast(itemId)); } -void NpcHandler::sellItem(int beingId A_UNUSED, int itemId, int amount) +void NpcHandler::sellItem(const int beingId A_UNUSED, + const int itemId, const int amount) const { MessageOut outMsg(CMSG_NPC_SELL_REQUEST); outMsg.writeInt16(8); // One item (length of packet) @@ -195,7 +198,7 @@ void NpcHandler::sellItem(int beingId A_UNUSED, int itemId, int amount) outMsg.writeInt16(static_cast(amount)); } -int NpcHandler::getNpc(Net::MessageIn &msg, bool haveLength) +int NpcHandler::getNpc(Net::MessageIn &msg, const bool haveLength) { if (haveLength) msg.readInt16(); // length @@ -231,9 +234,10 @@ int NpcHandler::getNpc(Net::MessageIn &msg, bool haveLength) } else { - if (mDialog && mDialog != diag->second.dialog) + NpcDialog *const dialog = diag->second.dialog; + if (mDialog && mDialog != dialog) mDialog->restoreCamera(); - mDialog = diag->second.dialog; + mDialog = dialog; if (mDialog) mDialog->saveCamera(); } @@ -241,14 +245,14 @@ int NpcHandler::getNpc(Net::MessageIn &msg, bool haveLength) } void NpcHandler::processNpcCutin(Net::MessageIn &msg A_UNUSED, - int npcId A_UNUSED) + int npcId A_UNUSED) const { msg.readString(64); // image name msg.readInt8(); // type } void NpcHandler::processNpcViewPoint(Net::MessageIn &msg A_UNUSED, - int npcId A_UNUSED) + int npcId A_UNUSED) const { msg.readInt32(); // type msg.readInt32(); // x diff --git a/src/net/eathena/npchandler.h b/src/net/eathena/npchandler.h index fc6a3fb6e..196da8398 100644 --- a/src/net/eathena/npchandler.h +++ b/src/net/eathena/npchandler.h @@ -44,33 +44,38 @@ class NpcHandler final : public MessageHandler, public Ea::NpcHandler A_DELETE_COPY(NpcHandler) - void handleMessage(Net::MessageIn &msg); + void handleMessage(Net::MessageIn &msg) override; - void talk(int npcId); + void talk(const int npcId) const override; - void nextDialog(int npcId); + void nextDialog(const int npcId) const override; - void closeDialog(int npcId); + void closeDialog(const int npcId) override; - void listInput(int npcId, unsigned char value); + void listInput(const int npcId, + const unsigned char value) const override; - void integerInput(int npcId, int value); + void integerInput(const int npcId, const int value) const override; - void stringInput(int npcId, const std::string &value); + void stringInput(const int npcId, + const std::string &value) const override; - void buy(int beingId); + void buy(const int beingId) const override; - void sell(int beingId); + void sell(const int beingId) const override; - void buyItem(int beingId, int itemId, unsigned char color, int amount); + void buyItem(const int beingId, const int itemId, + const unsigned char color, + const int amount) const override; - void sellItem(int beingId, int itemId, int amount); + void sellItem(const int beingId, const int itemId, + const int amount) const override; - int getNpc(Net::MessageIn &msg, bool haveLength); + int getNpc(Net::MessageIn &msg, const bool haveLength) override; - void processNpcCutin(Net::MessageIn &msg, int npcId); + void processNpcCutin(Net::MessageIn &msg, const int npcId) const; - void processNpcViewPoint(Net::MessageIn &msg, int npcId); + void processNpcViewPoint(Net::MessageIn &msg, const int npcId) const; }; } // namespace EAthena -- cgit v1.2.3-70-g09d2