Hacknet-Pathfinder
Hacknet-Pathfinder copied to clipboard
Arkhist
Add ConvertToXElement method to ElementInfo
I don't see the need for two versions of this method to exist (I would remove the static method). However, if there are to be two, one of them should probably call the other instead of having all of the code repeated.
I don't see the need for two versions of this method to exist (I would remove the static method). However, if there are to be two, one of them should probably call the other instead of having all of the code repeated.
I think the static one can be useful in some specific cases, however I probably will call the static one in the other giving this as parameter
I don't see what the point of the static method would serve, why are you duplicating functionality? In any case you have an ElementInfo you can just call the Convert method on it, it actually would cause an NRE in the method if the ElementInfo was null, which would be the only reason I could conceive for maybe having a static method, and even if you solve the NRE, how is it any different from info?.ConvertToXElement()?
