From c47ef8b3f2ec6ea0e26748af17cb12b7f9cd20c9 Mon Sep 17 00:00:00 2001 From: Andrei Karas Date: Thu, 28 Feb 2013 21:35:19 +0300 Subject: Improve some other files. --- src/inputmanager.cpp | 132 ++++++++++++++++++++++++++++++--------------------- src/inputmanager.h | 2 +- src/inventory.cpp | 45 ++++++++++-------- src/inventory.h | 6 +-- 4 files changed, 107 insertions(+), 78 deletions(-) diff --git a/src/inputmanager.cpp b/src/inputmanager.cpp index 11b810caf..eb3613640 100644 --- a/src/inputmanager.cpp +++ b/src/inputmanager.cpp @@ -75,10 +75,12 @@ void InputManager::init() { for (unsigned int i = 0; i < Input::KEY_TOTAL; i++) { + KeyFunction &kf = mKey[i]; for (unsigned int f = 0; f < KeyFunctionSize; f ++) { - mKey[i].values[f].type = INPUT_UNKNOWN; - mKey[i].values[f].value = -1; + KeyItem &ki = kf.values[f]; + ki.type = INPUT_UNKNOWN; + ki.value = -1; } } @@ -100,10 +102,11 @@ void InputManager::retrieve() { for (int i = 0; i < Input::KEY_TOTAL; i++) { - if (*keyData[i].configField) + const char *const cf = keyData[i].configField; + if (*cf) { KeyFunction &kf = mKey[i]; - std::string keyStr = config.getValue(keyData[i].configField, ""); + const std::string keyStr = config.getValue(cf, ""); if (keyStr.empty()) continue; @@ -148,13 +151,15 @@ void InputManager::store() const { for (int i = 0; i < Input::KEY_TOTAL; i++) { - if (*keyData[i].configField) + const char *const cf = keyData[i].configField; + if (*cf) { std::string keyStr; + const KeyFunction &kf = mKey[i]; for (size_t i2 = 0; i2 < KeyFunctionSize; i2 ++) { - const KeyItem &key = mKey[i].values[i2]; + const KeyItem &key = kf.values[i2]; if (key.type != INPUT_UNKNOWN) { std::string tmp = "k"; @@ -186,7 +191,7 @@ void InputManager::store() const if (keyStr.empty()) keyStr = "-1"; - config.setValue(keyData[i].configField, keyStr); + config.setValue(cf, keyStr); } } } @@ -198,13 +203,17 @@ void InputManager::resetKeys() KeyFunction &key = mKey[i]; for (size_t i2 = 1; i2 < KeyFunctionSize; i2 ++) { - key.values[i2].type = INPUT_UNKNOWN; - key.values[i2].value = -1; + KeyItem &ki2 = key.values[i2]; + ki2.type = INPUT_UNKNOWN; + ki2.value = -1; } - key.values[0].type = keyData[i].defaultType1; - key.values[0].value = keyData[i].defaultValue1; - key.values[1].type = keyData[i].defaultType2; - key.values[1].value = keyData[i].defaultValue2; + const KeyData &kd = keyData[i]; + KeyItem &val0 = key.values[0]; + val0.type = kd.defaultType1; + val0.value = kd.defaultValue1; + KeyItem &val1 = key.values[1]; + val1.type = kd.defaultType2; + val1.value = kd.defaultValue2; } } @@ -215,13 +224,17 @@ void InputManager::makeDefault(const int i) KeyFunction &key = mKey[i]; for (size_t i2 = 1; i2 < KeyFunctionSize; i2 ++) { - key.values[i2].type = INPUT_UNKNOWN; - key.values[i2].value = -1; + KeyItem &ki2 = key.values[i2]; + ki2.type = INPUT_UNKNOWN; + ki2.value = -1; } - key.values[0].type = keyData[i].defaultType1; - key.values[0].value = keyData[i].defaultValue1; - key.values[1].type = keyData[i].defaultType2; - key.values[1].value = keyData[i].defaultValue2; + const KeyData &kd = keyData[i]; + KeyItem &val0 = key.values[0]; + val0.type = kd.defaultType1; + val0.value = kd.defaultValue1; + KeyItem &val1 = key.values[1]; + val1.type = kd.defaultType2; + val1.value = kd.defaultValue2; update(); } @@ -235,31 +248,31 @@ bool InputManager::hasConflicts(int &key1, int &key2) const */ for (int i = 0; i < Input::KEY_TOTAL; i++) { - if (!*keyData[i].configField) + const KeyData &kdi = keyData[i]; + if (!*kdi.configField) continue; + const KeyFunction &ki = mKey[i]; for (size_t i2 = 0; i2 < KeyFunctionSize; i2 ++) { - if (mKey[i].values[i2].value == Input::KEY_NO_VALUE) + const KeyItem &vali2 = ki.values[i2]; + if (vali2.value == Input::KEY_NO_VALUE) continue; size_t j; for (j = i, j++; j < Input::KEY_TOTAL; j++) { - - if ((keyData[i].grp & keyData[j].grp) == 0 || - !*keyData[i].configField) - { + if ((kdi.grp & keyData[j].grp) == 0 || !*kdi.configField) continue; - } for (size_t j2 = 0; j2 < KeyFunctionSize; j2 ++) { + const KeyItem &valj2 = mKey[j].values[j2]; // Allow for item shortcut and emote keys to overlap // as well as emote and ignore keys, but no other keys - if (mKey[j].values[j2].type != INPUT_UNKNOWN - && mKey[i].values[i2].value == mKey[j].values[j2].value - && mKey[i].values[i2].type == mKey[j].values[j2].type) + if (valj2.type != INPUT_UNKNOWN + && vali2.value == valj2.value + && vali2.type == valj2.type) { key1 = static_cast(i); key2 = static_cast(j); @@ -277,7 +290,7 @@ void InputManager::callbackNewKey() mSetupInput->newKeyCallback(mNewKeyIndex); } -bool InputManager::isActionActive(const int index) const +bool InputManager::isActionActive(const int index) { if (keyboard.isActionActive(index)) return true; @@ -296,10 +309,11 @@ KeyFunction &InputManager::getKey(int index) std::string InputManager::getKeyStringLong(const int index) const { std::string keyStr; + const KeyFunction &ki = mKey[index]; for (size_t i = 0; i < KeyFunctionSize; i ++) { - const KeyItem &key = mKey[index].values[i]; + const KeyItem &key = ki.values[i]; std::string str; if (key.type == INPUT_KEYBOARD) { @@ -330,10 +344,11 @@ std::string InputManager::getKeyStringLong(const int index) const std::string InputManager::getKeyValueString(const int index) const { std::string keyStr; + const KeyFunction &ki = mKey[index]; for (size_t i = 0; i < KeyFunctionSize; i ++) { - const KeyItem &key = mKey[index].values[i]; + const KeyItem &key = ki.values[i]; std::string str; if (key.type == INPUT_KEYBOARD) { @@ -376,9 +391,9 @@ void InputManager::addActionKey(const int action, const int type, KeyFunction &key = mKey[action]; for (size_t i = 0; i < KeyFunctionSize; i ++) { - if (key.values[i].type == INPUT_UNKNOWN - || (key.values[i].type == type - && key.values[i].value == val)) + const KeyItem &val2 = key.values[i]; + if (val2.type == INPUT_UNKNOWN || (val2.type == type + && val2.value == val)) { idx = static_cast(i); break; @@ -388,8 +403,10 @@ void InputManager::addActionKey(const int action, const int type, { for (size_t i = 1; i < KeyFunctionSize; i ++) { - key.values[i - 1].type = key.values[i].type; - key.values[i - 1].value = key.values[i].value; + KeyItem &val1 = key.values[i - 1]; + KeyItem &val2 = key.values[i]; + val1.type = val2.type; + val1.value = val2.value; } idx = KeyFunctionSize - 1; } @@ -417,8 +434,9 @@ void InputManager::unassignKey() KeyFunction &key = mKey[mNewKeyIndex]; for (size_t i = 0; i < KeyFunctionSize; i ++) { - key.values[i].type = INPUT_UNKNOWN; - key.values[i].value = -1; + KeyItem &val = key.values[i]; + val.type = INPUT_UNKNOWN; + val.value = -1; } update(); } @@ -553,42 +571,42 @@ void InputManager::updateConditionMask() { mMask = 1; if (keyboard.isEnabled()) - mMask += COND_ENABLED; + mMask |= COND_ENABLED; if ((!chatWindow || !chatWindow->isInputFocused()) && !NpcDialog::isAnyInputFocused() && !InventoryWindow::isAnyInputFocused() && (!tradeWindow || !tradeWindow->isInpupFocused())) { - mMask += COND_NOINPUT; + mMask |= COND_NOINPUT; } if (!player_node || !player_node->getAway()) - mMask += COND_NOAWAY; + mMask |= COND_NOAWAY; if (!setupWindow || !setupWindow->isVisible()) - mMask += COND_NOSETUP; + mMask |= COND_NOSETUP; if (Game::instance() && Game::instance()->getValidSpeed()) - mMask += COND_VALIDSPEED; + mMask |= COND_VALIDSPEED; if (gui && !gui->getFocusHandler()->getModalFocused()) - mMask += COND_NOMODAL; + mMask |= COND_NOMODAL; const NpcDialog *const dialog = NpcDialog::getActive(); if (!dialog || !dialog->isTextInputFocused()) - mMask += COND_NONPCINPUT; + mMask |= COND_NONPCINPUT; if (!player_node || !player_node->getDisableGameModifiers()) - mMask += COND_EMODS; + mMask |= COND_EMODS; if (!isActionActive(Input::KEY_STOP_ATTACK) && !isActionActive(Input::KEY_UNTARGET)) { - mMask += COND_NOTARGET; + mMask |= COND_NOTARGET; } if (!player_node || player_node->getFollow().empty()) - mMask += COND_NOFOLLOW; + mMask |= COND_NOFOLLOW; } bool InputManager::checkKey(const KeyData *const key) const @@ -603,6 +621,8 @@ bool InputManager::checkKey(const KeyData *const key) const bool InputManager::invokeKey(const KeyData *const key, const int keyNum) { + // no validation to keyNum because it validated in caller + if (checkKey(key)) { InputEvent evt(keyNum, mMask); @@ -635,7 +655,8 @@ void InputManager::updateKeyActionMap(KeyToActionMap &actionMap, for (size_t i = 0; i < Input::KEY_TOTAL; i ++) { const KeyFunction &key = mKey[i]; - if (keyData[i].action) + const KeyData &kd = keyData[i]; + if (kd.action) { for (size_t i2 = 0; i2 < KeyFunctionSize; i2 ++) { @@ -644,7 +665,7 @@ void InputManager::updateKeyActionMap(KeyToActionMap &actionMap, actionMap[ki.value].push_back(static_cast(i)); } } - if (keyData[i].configField && (keyData[i].grp & Input::GRP_GUICHAN)) + if (kd.configField && (kd.grp & Input::GRP_GUICHAN)) { for (size_t i2 = 0; i2 < KeyFunctionSize; i2 ++) { @@ -653,7 +674,7 @@ void InputManager::updateKeyActionMap(KeyToActionMap &actionMap, idMap[ki.value] = static_cast(i); } } - if (keyData[i].configField && (keyData[i].grp & Input::GRP_REPEAT)) + if (kd.configField && (kd.grp & Input::GRP_REPEAT)) { for (size_t i2 = 0; i2 < KeyFunctionSize; i2 ++) { @@ -698,11 +719,12 @@ int InputManager::getKeyIndex(const int value, const int grp, for (size_t i = 0; i < Input::KEY_TOTAL; i++) { const KeyFunction &key = mKey[i]; + const KeyData &kd = keyData[i]; for (size_t i2 = 0; i2 < KeyFunctionSize; i2 ++) { - if (value == key.values[i2].value - && (grp & keyData[i].grp) != 0 - && key.values[i2].type == type) + const KeyItem &vali2 = key.values[i2]; + if (value == vali2.value && (grp & kd.grp) != 0 + && vali2.type == type) { return static_cast(i); } diff --git a/src/inputmanager.h b/src/inputmanager.h index a5325e357..da3095c81 100644 --- a/src/inputmanager.h +++ b/src/inputmanager.h @@ -126,7 +126,7 @@ class InputManager final void unassignKey(); - bool isActionActive(const int index) const A_WARN_UNUSED; + static bool isActionActive(const int index) A_WARN_UNUSED; /** * Set the index of the new key to be assigned. diff --git a/src/inventory.cpp b/src/inventory.cpp index a75a7a12e..3dce0e079 100644 --- a/src/inventory.cpp +++ b/src/inventory.cpp @@ -81,12 +81,13 @@ Item *Inventory::findItem(const int itemId, const unsigned char color) const { for (unsigned i = 0; i < mSize; i++) { - if (mItems[i] && mItems[i]->mId == itemId) + Item *const item = mItems[i]; + if (item && item->mId == itemId) { - if (color == 0 || mItems[i]->mColor == color - || (color == 1 && mItems[i]->mColor <= 1)) + if (color == 0 || item->mColor == color + || (color == 1 && item->mColor <= 1)) { - return mItems[i]; + return item; } } } @@ -110,7 +111,8 @@ void Inventory::setItem(const int index, const int id, const int quantity, return; } - if (!mItems[index] && id > 0) + Item *const item1 = mItems[index]; + if (!item1 && id > 0) { Item *const item = new Item(id, quantity, refine, color, equipment); item->setInvIndex(index); @@ -118,14 +120,14 @@ void Inventory::setItem(const int index, const int id, const int quantity, mUsed++; distributeSlotsChangedEvent(); } - else if (id > 0 && mItems[index]) + else if (id > 0 && item1) { - mItems[index]->setId(id, color); - mItems[index]->setQuantity(quantity); - mItems[index]->setRefine(refine); - mItems[index]->setEquipment(equipment); + item1->setId(id, color); + item1->setQuantity(quantity); + item1->setRefine(refine); + item1->setEquipment(equipment); } - else if (mItems[index]) + else if (item1) { removeItemAt(index); } @@ -141,7 +143,8 @@ void Inventory::removeItem(const int id) { for (unsigned i = 0; i < mSize; i++) { - if (mItems[i] && mItems[i]->mId == id) + const Item *const item = mItems[i]; + if (item && item->mId == id) removeItemAt(i); } } @@ -162,9 +165,11 @@ bool Inventory::contains(const Item *const item) const if (!item) return false; + const int id = item->mId; for (unsigned i = 0; i < mSize; i++) { - if (mItems[i] && mItems[i]->mId == item->mId) + const Item *const item1 = mItems[i]; + if (item1 && item1->mId == id) return true; } @@ -206,21 +211,23 @@ void Inventory::distributeSlotsChangedEvent() (*i)->slotsChanged(this); } -Item *Inventory::findItemBySprite(std::string spritePath, - const Gender gender, const int race) const +const Item *Inventory::findItemBySprite(std::string spritePath, + const Gender gender, + const int race) const { spritePath = removeSpriteIndex(spritePath); // logger->log1("Inventory::FindItemBySprite sprite: " + spritePath); - std::string spritePathShort = extractNameFromSprite(spritePath); + const std::string spritePathShort = extractNameFromSprite(spritePath); // logger->log1("Inventory::FindItemBySprite spriteShort: " + spritePathShort); int partialIndex = -1; for (unsigned i = 0; i < mSize; i++) { - if (mItems[i]) + const Item *const item = mItems[i]; + if (item) { - std::string path = mItems[i]->getInfo().getSprite(gender, race); + std::string path = item->getInfo().getSprite(gender, race); if (!path.empty()) { path = removeSpriteIndex(path); @@ -228,7 +235,7 @@ Item *Inventory::findItemBySprite(std::string spritePath, // logger->log("Inventory::FindItemBySprite normal: " + path); if (spritePath == path) - return mItems[i]; + return item; path = extractNameFromSprite(path); // logger->log("Inventory::FindItemBySprite short: " + path); diff --git a/src/inventory.h b/src/inventory.h index aedf18b24..46a0b3f58 100644 --- a/src/inventory.h +++ b/src/inventory.h @@ -153,9 +153,9 @@ class Inventory final bool isMainInventory() const A_WARN_UNUSED { return mType == INVENTORY; } - Item *findItemBySprite(std::string spritePath, - const Gender gender, - const int race) const A_WARN_UNUSED; + const Item *findItemBySprite(std::string spritePath, + const Gender gender, + const int race) const A_WARN_UNUSED; std::string getName() const A_WARN_UNUSED; -- cgit v1.2.3-70-g09d2