ai perf: Dom hash

Author: fchn289

src/domino/DataDomino.hpp

@@ -46,6 +46,8 @@ class DataDomino : public aDominoType
 
 protected:
     void rmEv_(Domino::Event aValidEv) noexcept override;
+    S_PTR<void> getData_(Domino::Event aEv) const noexcept { return ev_data_S_.get<void>(aEv); }
+    bool replaceDataOK_(Domino::Event aEv, S_PTR<void> aData) noexcept { return ev_data_S_.replaceOK(aEv, std::move(aData)); }
 
 private:
     // -------------------------------------------------------------------------------------------

src/domino/HdlrDomino.hpp

@@ -63,6 +63,8 @@ class HdlrDomino : public aDominoType
     virtual bool rmOneHdlrOK_(Domino::Event aValidEv, const SharedMsgCB& aValidHdlr) noexcept;  // by aValidHdlr
 
     void rmEv_(Domino::Event aValidEv) noexcept override;
+    size_t nHdlr_(Domino::Event aEv) const noexcept { return ev_hdlr_S_.count(aEv); }
+    bool rmOneHdlrOK_(Domino::Event aEv) noexcept { return ev_hdlr_S_.erase(aEv); }
 
     // -------------------------------------------------------------------------------------------
 private:

src/domino/MultiHdlrDomino.hpp

@@ -123,17 +123,18 @@ Domino::Event MultiHdlrDomino<aDominoType>::multiHdlrOnSameEv(const Domino::EvNa
 template<class aDominoType>
 size_t MultiHdlrDomino<aDominoType>::nHdlr(const Domino::EvName& aEN) const noexcept
 {
-    auto&& ev_hdlrs = ev_hdlrs_S_.find(this->getEventBy(aEN));
-    return (ev_hdlrs == ev_hdlrs_S_.end() ? 0 : ev_hdlrs->second.size()) + aDominoType::nHdlr(aEN);
+    const auto ev = this->getEventBy(aEN);
+    auto&& ev_hdlrs = ev_hdlrs_S_.find(ev);
+    return (ev_hdlrs == ev_hdlrs_S_.end() ? 0 : ev_hdlrs->second.size()) + aDominoType::nHdlr_(ev);
 }
 
 // ***********************************************************************************************
 template<class aDominoType>
 void MultiHdlrDomino<aDominoType>::rmAllHdlr(const Domino::EvName& aEN) noexcept
 {
-    aDominoType::rmOneHdlrOK(aEN);
-
-    ev_hdlrs_S_.erase(this->getEventBy(aEN));
+    const auto ev = this->getEventBy(aEN);
+    aDominoType::rmOneHdlrOK_(ev);
+    ev_hdlrs_S_.erase(ev);
 }
 
 // ***********************************************************************************************

src/domino/WbasicDatDom.hpp

@@ -41,6 +41,7 @@ class WbasicDatDom : public aDominoType
     // forbid ouside use base directly
     using aDominoType::getData;
     using aDominoType::replaceDataOK;
+    bool isWrCtrl_(Domino::Event aEv) const noexcept { return aEv < wrCtrl_.size() ? wrCtrl_[aEv] : false; }
     // -------------------------------------------------------------------------------------------
     std::vector<bool> wrCtrl_;
 
@@ -52,8 +53,9 @@ class WbasicDatDom : public aDominoType
 template<typename aDominoType>
 S_PTR<void> WbasicDatDom<aDominoType>::getData(const Domino::EvName& aEvName) const noexcept
 {
-    if (not isWrCtrl(aEvName))
-        return aDominoType::getData(aEvName);
+    const auto ev = this->getEventBy(aEvName);
+    if (not isWrCtrl_(ev))
+        return aDominoType::getData_(ev);
 
     WRN("(WbasicDatDom) Failed!!! EvName=" << aEvName << " is not write-protect so unavailable via this func!!!");
     return nullptr;
@@ -63,15 +65,15 @@ S_PTR<void> WbasicDatDom<aDominoType>::getData(const Domino::EvName& aEvName) co
 template<typename aDominoType>
 bool WbasicDatDom<aDominoType>::isWrCtrl(const Domino::EvName& aEvName) const noexcept
 {
-    const auto ev = this->getEventBy(aEvName);
-    return ev < wrCtrl_.size() ? wrCtrl_[ev] : false;
+    return isWrCtrl_(this->getEventBy(aEvName));
 }
 
 // ***********************************************************************************************
 template<typename aDominoType>
 bool WbasicDatDom<aDominoType>::replaceDataOK(const Domino::EvName& aEvName, S_PTR<void> aData) noexcept
 {
-    if (isWrCtrl(aEvName)) {
+    const auto ev = this->getEventBy(aEvName);
+    if (isWrCtrl_(ev)) {
         WRN("(WbasicDatDom) Failed!!! EvName=" << aEvName << " is not write-protect so unavailable via this func!!!")
         return false;
     }
@@ -92,8 +94,9 @@ void WbasicDatDom<aDominoType>::rmEv_(Domino::Event aValidEv) noexcept
 template<typename aDominoType>
 S_PTR<void> WbasicDatDom<aDominoType>::wbasic_getData(const Domino::EvName& aEvName) const noexcept
 {
-    if (isWrCtrl(aEvName))
-        return aDominoType::getData(aEvName);
+    const auto ev = this->getEventBy(aEvName);
+    if (isWrCtrl_(ev))
+        return aDominoType::getData_(ev);
 
     WRN("(WbasicDatDom) Failed!!! EvName=" << aEvName << " is not write-protect so unavailable via this func!!!");
     return nullptr;
@@ -103,7 +106,8 @@ S_PTR<void> WbasicDatDom<aDominoType>::wbasic_getData(const Domino::EvName& aEvN
 template<typename aDominoType>
 bool WbasicDatDom<aDominoType>::wbasic_replaceDataOK(const Domino::EvName& aEvName, S_PTR<void> aData) noexcept
 {
-    if (isWrCtrl(aEvName))
+    const auto ev = this->getEventBy(aEvName);
+    if (isWrCtrl_(ev))
         return aDominoType::replaceDataOK(aEvName, std::move(aData));
     else {
         WRN("(WbasicDatDom) Failed!!! EvName=" << aEvName << " is not write-protect so unavailable via this func!!!")
@@ -115,13 +119,13 @@ bool WbasicDatDom<aDominoType>::wbasic_replaceDataOK(const Domino::EvName& aEvNa
 template<typename aDominoType>
 bool WbasicDatDom<aDominoType>::wrCtrlOk(const Domino::EvName& aEvName, const bool aNewState) noexcept
 {
-    if (aDominoType::getData(aEvName))
+    const auto ev = this->newEvent(aEvName);
+    if (aDominoType::getData_(ev))
     {
         WRN("(WbasicDatDom) !!! Failed to change wrCtrl when aleady own data(out-of-ctrl), EvName=" << aEvName);
         return false;
     }
 
-    const auto ev = this->newEvent(aEvName);
     if (ev >= wrCtrl_.size())
         wrCtrl_.resize(ev + 1);  // resize() can inc size()
     wrCtrl_[ev] = aNewState;