addChild() sollte typisierte non-owning Pointer zurückgeben, nicht size_t #6

New Issue

sebastian · 2026-04-16T19:42:42Z

sebastian commented

2026-04-16 19:42:42 +00:00

Problem: addChild() gibt aktuell size_t zurück. Um später auf das Kind zuzugreifen, braucht man getChildAtIndex(id) + dynamic_cast:

inventoryContainerID = uiContainer->addChild(std::move(inv));
// später:
auto* c = dynamic_cast<UiInventoryContainer*>(rootContainer->getChildAtIndex(inventoryContainerID));

Lösung:

template<typename T>
T* addChild(std::unique_ptr<T> child) {
    T* ptr = child.get();
    children.push_back(std::move(child));
    child->parent = this;
    return ptr;  // non-owning, Ownership bleibt bei children
}

Problem: addChild() gibt aktuell size_t zurück. Um später auf das Kind zuzugreifen, braucht man getChildAtIndex(id) + dynamic_cast: ``` inventoryContainerID = uiContainer->addChild(std::move(inv)); // später: auto* c = dynamic_cast<UiInventoryContainer*>(rootContainer->getChildAtIndex(inventoryContainerID)); ``` Lösung: ``` template<typename T> T* addChild(std::unique_ptr<T> child) { T* ptr = child.get(); children.push_back(std::move(child)); child->parent = this; return ptr; // non-owning, Ownership bleibt bei children } ```

sebastian added the

refactoring

engine

labels 2026-04-16 19:42:42 +00:00

sebastian added this to the Quick Wins milestone 2026-04-16 19:50:26 +00:00

sebastian added this to the Dicewars project 2026-04-16 20:59:29 +00:00

sebastian started working 2026-04-17 06:09:08 +00:00

sebastian referenced this issue from a commit

2026-04-17 06:34:30 +00:00

UPD: addChild() gibt typisierte Pointer zurück, keinen Index, closes #6

sebastian closed this issue

2026-04-17 06:34:30 +00:00