Ideas:chad3f:RPFramework: Difference between revisions

← Older edit

Content deleted Content added

VisualWikitext

Revision as of 02:40, 31 May 2007 view source imported>Chad3f →RPClass Initlialization ← Older edit		Latest revision as of 02:52, 31 May 2007 view source imported>Chad3f →Message Protocol
(72 intermediate revisions by the same user not shown)
Line 39: This has the following problems/short-comings: -* It requires the generate methods to be explicitly called from some initialization code. The was an actual stendhal problem were several generateRPClass() methods were defined, but never called. -* The generate methods have to be called in the proper order. This requires the initialization code to know the implementation details (at least the class tree) of the RPClass's that it's configuring. As a result, unnecessary code-coupling is formed. If it is done wrong, or the heirarcy changes after the fact and nobody thinks to check those fragile dependancies, it could break. -* The parent RPClass is assigned after creation [via isA()], so there's the chance it could be missed. Or worse yet, called twice with different parents (that could have some nasty unexpected effects). - The assigned parent is typically assigned by name, rather than object, which can make it prone to spelling errors. If you're lucky, it will throw an exception, if not a hard to track problem may arise. Also passing the name means a common base class has to be resolved several times.▼ ▲-* The assigned parent is typically assigned by name, rather than object, which can make it prone to spelling errors. If you're lucky, it will throw an exception, if not a hard to track problem may arise. Also passing the name means a common base class has to be resolved several times. === Proposed Replacement === Using static RPClass ~~constaints~~constants, and relying on the JVM to enforce initialization order, many of these potential problems can be made nearly ~~imposible~~impossible. The replacement code would look like: Line 56 ⟶ 55: class SomeClass extends RPObject { public static final RPClass RPCLASS = createRPClass(); private static RPClass createRPClass() { RPClass rpclass = new RPClass("some"); Line 66 ⟶ 65: class OtherClass extends SomeClass { public static final RPClass RPCLASS = createRPClass(); private static RPClass createRPClass() { RPClass rpclass = new RPClass(SomeClass.RPCLASS, "other"); Line 73 ⟶ 72: } } class AnotherClass extends SomeClass { public static final RPClass RPCLASS = createRPClass(); private static RPClass createRPClass() { RPClass rpclass = new RPClass(SomeClass.RPCLASS, "another"); Line 91 ⟶ 90: In this code, when one of these java classes is loaded, the static initializer will automatically call it's createRPClass() method. If the class has a super-RPClass [such as OtherClass.createRPClass() uses], before the reference to SomeClass.RPCLASS is used, the JVM will have loaded SomeClass and called it's createRPClass(). If the parent RPClass is only assignable during the constructor of sub-classes [i.e. normal use of isA() goes away], then it will be more obvious that the parent has been omitted. Of course, someone could always call the RPClass constructor with no parent argument, and still forget the super, unless the only constructor(s) available required the parent, in which case the '''new RPClass(null, "name")''' would be very obvious as having no parent. Benefits: -* Guarantee order of creation. -* Compile-time parent checking (look ma' no strings). -* Never having to resolve RPClass's by name (in init code anyway), since the JVM will assign the runtime ~~constaint~~constant once, and everything else uses this value. -* No explicit need to force class definition from a common setup class/method, as long as the class containing the RPCLASS constant is referenced before any RPObject instances using that class are bound to their RPClass. And this can be guaranteed if the constant is used when setting the RPObject's class (with the same benefits as RPClass gets). ---- == Message Protocol == Line 108 ⟶ 110: ''TBW'' ---- [[Ideas:chad3f\|Back to Ideas]]