Functions from includes/ now class based
Posted: Sat May 03, 2014 21:38 UTC
I spent half the day (roughly) moving the ibank functions from includes/ into classes/BadIbank.php . That file contains the new "mega" class BadIbank, which contains all the ibank functions.
I purposefully prefixed it "Bad", because it doesn't conform to our goals for classes. Ideally, a method in a class should have no globals, a clearly defined input and output, decent commenting, and so forth.
BadIbank breaks pretty much all of those rules and others. However, I wanted to get us to a point where the includes/ directory was gone, and functions belong in classes that are auto-loaded.
Its a step in that direction.
I purposefully prefixed it "Bad", because it doesn't conform to our goals for classes. Ideally, a method in a class should have no globals, a clearly defined input and output, decent commenting, and so forth.
BadIbank breaks pretty much all of those rules and others. However, I wanted to get us to a point where the includes/ directory was gone, and functions belong in classes that are auto-loaded.
Its a step in that direction.