Quote:
So in essance, what your saying is that it is better to use non primitive since your values are being translated to sql columns that could potentially have null values.
It's simply a matter of Boolean fields being first-class objects, which allows them to contain
null values, rather than being restricted to the true/false value of a primitive boolean (which defaults to
false). It's really more a matter of your domain dictating which type is appropriate for your use case than a matter of "correctness" per se.
Quote:
It seemed that when i converted my booleans to Booleans, it did not yell at me to change my getters to getXXX rather than isXXX and thus working fine with current code. Will i ever see a problem with this?
As sagimann indicated, the accessor methods that are auto-generated by Eclipse simply follow convention depending on the field type. Primitive boolean fields are referenced by isXXX and setXXX, while first-class objects (such as Boolean) are referenced by the getXXX and setXXX convention. You didn't get "yelled at" because you could, conceivably, name them whatever you want - but in the name of "purity", I would personally recommend that you follow standard convention for whatever type you choose.
Again, it is really your domain that determines whether or not your choice will cause any problems down the line.
Hope that helps.