[Feedback Needed] Changes to the bank class

In V1.0.0 beta 16 and prior, the Bank class includes Bank.getNearestBank.
In V1.0.0 beta 17 and above, this is going to change slightly, in what way though I've yet to decide.

Option 1
Bank.getNearestBank will be renamed to Bank.getNearest and Bank.getLoaded will be added.

Option 2 (My favorite)
A class called Banks will be added alongside the other entity retrieval classes with plenty of helpers including Banks.getNearestTo(Locatable), Banks.getLoadedWithin(Area), etc. I feel like this is more appropriate because currently the Bank class resides within the interfaces package, while the retrieval has nothing to do with that. This would also allow for the most expansion and the most extensive methods related to this.

Option 3
Expand the GameObjects class to include getNearestBank, getLoadedBanks, and several others. It'll make the GameObjects class feel cluttered and will limit expansion because we don't want to clutter it further.

Please voice your opinion

 

Good enough for me, expect this option 2 in the next version

http://i.gyazo.com/db79df839c421d5bb1c6dd1205c1350b.png

Edit: I realize that I had redundant method arguments in a couple of the jdoc methods, too much copy pasting lol

 

Banks.getNearest() would return the nearest bank relative to the local player, while Banks.getNearestTo(range) would return the nearest bank relative to the range.