[brailleblaster] Re: Here are the liblouisutdml Java bindings.

  • From: "Sina Bahram" <sbahram@xxxxxxxxx>
  • To: <brailleblaster@xxxxxxxxxxxxx>
  • Date: Tue, 16 Nov 2010 16:07:14 -0500

thisIsWrittenInCamelCase

This isn't

Furthermore classes, interfaces, and especially type labels have initial caps

Out of curiosity, have you had a chance to go over a java programming book? 
Lots of these questions, and many more, are answered in
one.

Take care,
Sina
 
-----Original Message-----
From: brailleblaster-bounce@xxxxxxxxxxxxx 
[mailto:brailleblaster-bounce@xxxxxxxxxxxxx] On Behalf Of John J. Boyer
Sent: Tuesday, November 16, 2010 4:02 PM
To: brailleblaster@xxxxxxxxxxxxx
Subject: [brailleblaster] Re: Here are the liblouisutdml Java bindings.

This is for calling liblouisutdml from Java. There is corresponnding C code 
which uses JNI. Mode bits can be combined with aon or |
in calls. 
What is camel case, or iis this a typo? 

John

On Tue, Nov 16, 2010 at 03:50:11PM -0500, Sina Bahram wrote:
> May I please suggest using camel case?
> 
> This is not only hard to read, frankly it's painful, *smile*.
>  
> Also, why not have this be an interface, since that's exactly what it is?
> 
> These constants need to be externalized.
> 
> There's far too much in this one class. Wouldn't this benefit from being 
> broken up?
> 
> I am guessing you are using the native keyword for purposes of JNI? I 
> can't think of another logical reason to ever use it, and I think it might be 
> overused in this particular class, but we can
examine that later.
> 
> You don't need void if you have an empty argument list ... I know it's 
> popular in c, but nobody does that in java. I forgot that java even allows 
> that in the first place.
> 
> If your constants are bit flags, then why not actually make them such 
> via bit masks? Right now they are just taking up entire integers for no good 
> reason. You might simply want to use an enum for this
an index into a single integer after a bitwise and.
> 
> Again, I'd recommend javadoc.
> 
> Take care,
> Sina
> 
> -----Original Message-----
> From: brailleblaster-bounce@xxxxxxxxxxxxx 
> [mailto:brailleblaster-bounce@xxxxxxxxxxxxx] On Behalf Of John J. 
> Boyer
> Sent: Tuesday, November 16, 2010 3:08 PM
> To: brailleblaster@xxxxxxxxxxxxx
> Subject: [brailleblaster] Here are the liblouisutdml Java bindings.
> 
> They are attached for your delectation or horror, probably depending 
> on your experience with Java. I've made them as simple as possible, so 
> there is no exception handling. The library has its own 
> error-reporting code. Errors are placed in the log file given as an argument 
> to most of the methods. I figured that BrailleBlaster
could analyze that log file and give the user much more useful information than 
exceptions.
> 
> I've used double-asterisk comments for documentation, but I haven't 
> bothered with html markup. The html markkup I've seen in Java programs 
> doesn't impress me as very good anyway.
> 
> Happy reading,
> John
> 
> --
> John J. Boyer; President, Chief Software Developer Abilitiessoft, Inc.
> http://www.abilitiessoft.com
> Madison, Wisconsin USA
> Developing software for people with disabilities
> 
> 

--
John J. Boyer; President, Chief Software Developer Abilitiessoft, Inc.
http://www.abilitiessoft.com
Madison, Wisconsin USA
Developing software for people with disabilities



Other related posts: