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

  • From: "John J. Boyer" <johnjboyer@xxxxxxxxxxxxx>
  • To: brailleblaster@xxxxxxxxxxxxx
  • Date: Tue, 16 Nov 2010 16:45:00 -0600

Sure. i just wanted to have a constructor in case I wanted to do 
something with it in the future.
I thought you would be one of those who would be horrified by my code.

John

On Tue, Nov 16, 2010 at 09:52:54PM +0000, Michael Whapples wrote:
> There's one other thing which could be mentioned, default constructors. 
> While it doesn't harm to have it in there, java will automatically 
> generate the default no-args constructor if there is no other 
> constructor defined, so making specifying one surplus to requirements.
> 
> Michael Whapples
> On 16/11/10 20:50, 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
> >
> >
> >
> 
> 

-- 
My websites:
GodTouches Digital Ministry, Inc. http://www.godtouches.org
Abilitiessoft, Inc. http://www.abilitiessoft.com
Location: Madison, WI, USA


Other related posts: