AI: trim DataStore & ObjAny

Author: fchn289

src/obj_anywhere/DataStore.hpp

@@ -4,12 +4,11 @@
  * SPDX-License-Identifier: BSD-3-Clause
  */
 // ***********************************************************************************************
-// - REQ: this class is to enhance safety of shared_ptr:
-//   . store any type data: eg for ObjAnywhere
-//   * store same type data: eg for DataDomino
-//   . mem safe by SafePtr
-// - Unique Value:
-//   . store
+// - REQ: this class is to
+//   . store data of any type: eg for ObjAnywhere
+//   * store multi-copy of data of the same type: eg for DataDomino
+//   . fetch data by unique key(eg string)
+//   . ensure mem safe of stored data by SafePtr
 //
 // - MT safe: NO
 // - mem safe: yes
@@ -46,7 +45,7 @@ class DataStore
 template<typename aDataKey>
 void DataStore<aDataKey>::emplace(const aDataKey& aKey, SafePtr<void> aData)
 {
-    if (aData.get() == nullptr)
+    if (! aData)
     {
         HID("(DataStore) erase key=" << aKey);
         key_data_S_.erase(aKey);
@@ -76,7 +75,7 @@ SafePtr<aDataT> DataStore<aDataKey>::get(const aDataKey& aKey) const
 template<typename aDataKey>
 void DataStore<aDataKey>::replace(const aDataKey& aKey, SafePtr<void> aData)
 {
-    if (aData.get() == nullptr)
+    if (! aData)
     {
         HID("(DataStore) erase key=" << aKey);
         key_data_S_.erase(aKey);

src/obj_anywhere/ObjAnywhere.cpp

@@ -23,8 +23,8 @@ void ObjAnywhere::init(UniLog& oneLog)
         WRN("(ObjAnywhere) !!! Refuse dup init.")
     else
     {
-        HID("(ObjAnywhere) Succeed.");
         name_obj_S_ = make_shared<DataStore<ObjName>>();
+        HID("(ObjAnywhere) Succeed.");
     }
 }
 }  // namespace

src/obj_anywhere/ObjAnywhere.hpp

@@ -13,22 +13,12 @@
 //
 // - more value:
 //   . ObjAnywhere not include any Obj.hpp so no cross-include conflict
-//   * ObjAnywhere stores shared_ptr<Obj> - real store, correct destruct, lifespan mgr
+//   * ObjAnywhere stores SafePtr<Obj> - real store, correct destruct, lifespan management
 //
 // - core: name_obj_S_
 //
 // - mem-safe: true (when use SafePtr instead of shared_ptr)
-//
-// - note:
-//   . Obj can be destructed by its own destructor when shared_ptr<Obj>.use_count()==0
-//   . it's possible after deinit() that Obj still exists since its use_count()>0
-//   . if req to store same Obj multiply, use DataDomino
-//
-// - can DataDomino replace ObjAnywhere?
-//   . somewhat yes
-//   . but ObjAnywhere will be more complex since ObjIndex is replaced by EvName
-//   . and DataDomino will be more complex to manage sharing/static store_
-//   . = merge DataDomino & ObjAnywhere, so now giveup
+// - MT safe: false (only use in main-thread)
 // ***********************************************************************************************
 #pragma once