@mrsgreenpotato@TheLongPrice Isn’t the string reference just in the GetNodeOrNull call? I prefer to just export the nodes I need, so I don’t build up Node-structure dependencies in the code. Therefore I don’t have “Magic strings” in my code (almost) at all.
Correct, I’d also use exported values if possible. But that isn’t flexible enough in some cases, because you’d need to individually export all possible attributes that the node might have.
For it to be more flexible, you could have something like this:
for child inget_children():
if child is ClassNameHere:
return child
That would give you the same result as described in the article, without string reference. You could make a static func for it and call it a day :)
@mrsgreenpotato what case do you have in mind where an export is not flexible enough? Also, do you return a Variant or generic Node(2D/3D) of a function like that or are you working with Generics then? So you do: GetNode<Bumpable>() for example?
@mrsgreenpotato ahh ok, could you extend from a base node type and add this function respectively?
We’ve defined “entities” in our project and each Entity-Type has a script for some basic functionality (to define that it IS an entity etc.).
In our project a CharacterBody3D entity is an “Actor”, while a RigidBody3D is an “Item” for example. And we’re basically only working with these and define components for them that we can retrieve via a Dictionary<StringName, Resource>
@mrsgreenpotato (2/2) and in our case, we’ve put every piece of data into resources where we export the values we need. If we need a node in a scene, we just export it and it’s mostly a direct childNode of the scene. All in all we have a system where the example zombie consists of the following components: Ai, Model, Stats, Info, Sfx, Chemistry. These have their own scenes that get added to the entityNode. In the end there’s no need for such a helper function basically :D (at least yet)
That would definitely be an improvement, using classes is probably more robust. In my case, the project wasn’t too huge and it was solo, so I never hit issues with string references.
It’s a good practice to adhere to proper design patterns even on smaller projects. I also work solo on small projects :). It doesn’t really cost you anything to work with classes, it’d even say it’s easier once you get a hang of it
Removed by mod
@mrsgreenpotato @TheLongPrice Isn’t the string reference just in the GetNodeOrNull call? I prefer to just export the nodes I need, so I don’t build up Node-structure dependencies in the code. Therefore I don’t have “Magic strings” in my code (almost) at all.
Correct, I’d also use exported values if possible. But that isn’t flexible enough in some cases, because you’d need to individually export all possible attributes that the node might have.
For it to be more flexible, you could have something like this:
for child in get_children(): if child is ClassNameHere: return childThat would give you the same result as described in the article, without string reference. You could make a static func for it and call it a day :)
@mrsgreenpotato what case do you have in mind where an export is not flexible enough? Also, do you return a Variant or generic Node(2D/3D) of a function like that or are you working with Generics then? So you do: GetNode<Bumpable>() for example?
Removed by mod
@mrsgreenpotato ahh ok, could you extend from a base node type and add this function respectively?
We’ve defined “entities” in our project and each Entity-Type has a script for some basic functionality (to define that it IS an entity etc.).
In our project a CharacterBody3D entity is an “Actor”, while a RigidBody3D is an “Item” for example. And we’re basically only working with these and define components for them that we can retrieve via a Dictionary<StringName, Resource>
@mrsgreenpotato (2/2) and in our case, we’ve put every piece of data into resources where we export the values we need. If we need a node in a scene, we just export it and it’s mostly a direct childNode of the scene. All in all we have a system where the example zombie consists of the following components: Ai, Model, Stats, Info, Sfx, Chemistry. These have their own scenes that get added to the entityNode. In the end there’s no need for such a helper function basically :D (at least yet)
Removed by mod
That would definitely be an improvement, using classes is probably more robust. In my case, the project wasn’t too huge and it was solo, so I never hit issues with string references.
It’s a good practice to adhere to proper design patterns even on smaller projects. I also work solo on small projects :). It doesn’t really cost you anything to work with classes, it’d even say it’s easier once you get a hang of it