Just a matter of personal style/opinion, but could you move these fields above the constructor, so they're the first thing in the class? I was slightly confused when I looked at the constructor and didn't see the fields.