Что касается конструкторов, да, они были определены правильно.Пока «подпись» конструкторов отделена, класс будет компилироваться.
Что касается методов доступа (get) и mutator (set), используется следующее:
Аксессор:
public void getName()
{
return name;
}
Мутатор:
public String setName(String name)
{
this.name = name;
}
После этого поможет ваш код:)