Pues no veo que pueda estar mal.. los de tipo entero tambien llevan apostrofes?

Todos son strings , (texto)

con mi formato tambien te tira error? que te marca?

Creo que si pones que error te marca estaria mas facil de poder ayudarte......saludos....

yo te recomiendo que mejor lo hagas en MVC

asi te evitas ese tipo de problema x ke las estructiuras serian mas simples

bueno aki te dejo un ejemplo que yo realize y funciona perfectamente

espero te sirva y si tienes duda pues pregunta

yo manejo esto en MVC osea que este solo es el codigo para realizar una insercion

falta el codigo del Modelo y logico la interfas

pero espero te sirva mi codigo y te des una idea

ok

bye

package PerMor;
import java.sql.*;
import java.util.*;
import javax.swing.*;

public class AusuarioCtrl {

public static int insertar(PasswordM mor){

Connection con=null;
PreparedStatement ps=null;
String sql="INSERT INTO login VALUES (?,?,?)";
int reg=-1;

try{
mor.validaP();
con=Conexion.getConexion();

if(con!=null){

ps=con.prepareStatement(sql);
ps.setString(1,mor.getUsuario());
ps.setString(2,mor.getPassword());
ps.setString(3,mor.getTipo());

con.setAutoCommit(true);
reg=ps.executeUpdate();
}
}catch(SQLException sqle){
System.out.println("Error-->Insertar Producto: "+sqle.getMessage());
JOptionPane.showMessageDialog(null,"El Nombre De Usuario\nya se Encuentra en la Base de Datos","Registro Duplicado",JOptionPane.INFORMATION_MESSAGE,new ImageIcon("imagenes/dobleuser.gif"));
}catch(Exception ex){
String excepcion;
excepcion=String.valueOf(ex);

JOptionPane.showMessageDialog(null," "+excepcion.substring(32));
}finally{
try{
if(con!=null){
con.close();
ps.close();
}
}catch(Exception e){}
}


return reg;
}
}