RE: [Embedlets-dev] JAPL source code

[Ted K. <tk...@ya...>]

Chris wrote:

> 
> I would like to submit the following for your review:
> org
>  |_japl
>     |_ JaplListener.java
>     |_ JaplEvent.java
>     |_ network
>         |_ MediaController.java
>         |_ MediaEvent.java
>         |_ MediaListener.java
> 

Hey, good start!  A few of thoughts though...  

1) I just looked through the standard library APIs that come with J2SE and
classes like SAXException, UIDefaults, URLDecoder, etc. and all keep the
letters of their acronyms in uppercase.  If we follow this convention (which I
would vote for) then JaplListener would be JAPLEvent, etc.

2) How about shortening 'network' to just 'net' and then putting all of the
MediaController related classes inside a subpackage called 'mediacontroller'? 
For example: org.japl.net.mediacontroller.MediaController,
org.japl.net.mediacontroller.MediaEvent,
org.japl.net.mediacontroller.MediaListener.  This way other people can place
network related JAPL interfaces inside the 'net' package and they won't all get
mixed up with other JAPL interface classes.

3) Should JAPLListener and JAPLEvent be placed inside of an 'event' subpackage
or a 'util' subpackage similar to what is done in the standard library APIs?


Ted

__________________________________
Do you Yahoo!?
SBC Yahoo! DSL - Now only $29.95 per month!
http://sbc.yahoo.com