O que há de errado nesse Beans?

public class Bin {
	
	private String nome=null;
	private String data=null;
	private int hora; 

	public String getNome(){
	return nome;
	}
	public void setNome(String nome){
	this.nome=nome;
	}
	
	public String getData(){
	return data;
	}
	public void setData(String data){
	this.data=data;
	}
public int getHora(){
    return hora;
    } 
    public void setHora(int hora){
    this.hora=hora;
    }

public void Inserir(){	
Connection con=null;
	String query1="Insert into Principal values(?,?,?,?)";
	try{
	int id=cx.id(idMaxa)+1;	
	con=cx.getConect();
	PreparedStatement ps=con.prepareStatement(query1);
	ps.setInt(1,id);
	ps.setString(2,nome);
	ps.setString(3,data);
	ps.setInt(4,hora);
.....
}

public class Atendimento

public void telaCadastro(){
	Bin bn= new Bin();
	bn.setNome(txNomeAtend.getText());
	bn.setData(txDia.getText());
	bn.setHora(Integer.parseInt(txHoraIni.getText()));
}

//botão que gera a acao
btn.addActionListener(new ActionListener(){
	public void actionPerformed(ActionEvent e){
	Bin bne= new Bin();	
	telaCadastro();	
	bne.Inserir();
	}}
	);