enforce unique constraint on property id

kr0ner · Nov 26, 2024 · 580912e · 580912e
1 parent 17d05cb
commit 580912e
Showing 1 changed file with 23 additions and 18 deletions.
diff --git a/src/property.h b/src/property.h
@@ -9,17 +9,18 @@
 
 namespace detail {
 struct Property;
-struct Mapper {
-    Mapper(const Property* other);
-
-   protected:
+class Mapper {
+   public:
     Mapper() = default;
+    Mapper(const Property other);
 
-   protected:
-    static inline std::unordered_map<std::uint16_t, const Property*> map;
+    Property getProperty(const std::uint16_t id);
+
+   private:
+    static inline std::unordered_map<std::uint16_t, Property> map;
 };
 
-struct Property : public Mapper {
+struct Property {
     std::string_view name;
     std::uint16_t id{0U};
     Type type{Type::et_default};
@@ -38,28 +39,32 @@ struct Property : public Mapper {
     }
 
    protected:
-    Property getProperty(const std::uint16_t id) {
-        if (auto it = map.find(id); it != map.end()) {
-            return *(it->second);
-        }
-        return {"UNKNOWN", id};
-    }
+    Property getProperty(const std::uint16_t id) { return Mapper().getProperty(id); }
 };
 
-inline Mapper::Mapper(const Property* other) {
-    map.insert({other->id, other});
+inline Mapper::Mapper(const Property other) {
+    map.insert({other.id, other});
 }
+
+Property Mapper::getProperty(const std::uint16_t id) {
+    if (auto it = map.find(id); it != map.end()) {
+        return it->second;
+    }
+    return {"UNKNOWN", id};
+}
+
 }  // namespace detail
 
 #define PROPERTY(NAME, VALUE, ...)                                          \
     static constexpr detail::Property k##NAME{#NAME, VALUE, ##__VA_ARGS__}; \
+    static constexpr bool unique##VALUE{true};                              \
     static inline detail::Mapper k##NAME##_MAPPING {                        \
-        &k##NAME                                                            \
+        k##NAME                                                             \
     }
 
 struct Property : public detail::Property {
-    Property(const Property& p) : detail::Property{p.name, p.id, p.type} {}
-    Property(const detail::Property& p) : detail::Property{p.name, p.id, p.type} {}
+    constexpr Property(const Property& p) : detail::Property{p.name, p.id, p.type} {}
+    constexpr Property(const detail::Property& p) : detail::Property{p.name, p.id, p.type} {}
     Property(const std::uint16_t _id) : detail::Property{getProperty(_id)} {}
 
     PROPERTY(INDEX_NOT_FOUND, 0x0000);