Vou complementar o comentário do Diego,
conforme o padrão JavaBeans, você deveria ter também um contrutor sem parâmetros, e além do construtor ser público os métodos get e set também devem ter seu nível de acesso estabelecido, baseado na seguinte regra:
Modificador de acesso - restrição
public - todos acessam
protected - O recurso é acessado pela classe, pelas classes do mesmo package e pelas subclasses.
não especificado (o seu caso), o acesso é permitido apenas para a classe e para as classes da mesma package, não sendo permitido para as subclasses e para as demais classes do sistema.
private - o mais restretivo, permite o acesso ao recurso apenas para os objetos da classe.
Aqui recurso pode ser o método ou propriedade.
veja o link:
http://java.sun.com/docs/books/tutorial/java/javaOO/accesscontrol.html
Outro aspecto importante é com relação o uso ou não da chamada dos métodos set.
Se você fizer o uso das chamadas de método, em caso de ter que fazer alguma pré-checagem antes de aceitar o valor, não terá que refatorar o seu código, caso contrário será necessário.
Valeu.
