0

Tengo un problema a la hora de poder insertar mis datos dentro de mi base de datos en la table articulos, me aparece este error: ``Las columnas de tipo 'DOUBLE' no pueden contener valores de tipo 'CHAR'`. No sé muy bien porque ocurre. Me encuentro trabajando con una arquitectura por capas.

Mi table articulo es la siguiente:

create table articulo(
 cod char(4) not null,
 nom varchar(40) not null,
 pre double not null,
 sto int not null,
 primary key (cod)
)

Este es mi DaoArticuloImp, en lo que respecta a la clase que dice Operación esa si funciona bien, ya lo habia probado anteriormente y funcionaba sin ningún problema.

public class DaoArticuloImp implements DaoArticulo{

@Override
public String grabar(String cod, String nom, double pre, int sto) {
    String sql ="insert into articulo values('"+cod+"','"+nom+"','"+pre+"','"+sto+"')";
    return Operacion.ejecutar(sql);
}

@Override
public Object[] buscar(String cod) {
    String sql = "select * from articulo where cod='"+cod+"'";
    return Operacion.buscar(sql);
}

@Override
public String actualizar(String cod, double pre, int sto) {
    String sql = "update articulo set pre='"+pre+"' and sto='"+sto+"' where cod='"+cod+"'";
    return Operacion.ejecutar(sql);
}

@Override
public String eliminar(String cod) {
    String sql = "delete from articulo where cod='"+cod+"'";
    return Operacion.ejecutar(sql);
}

@Override
public List Listar() {
    String sql = "select * from articulo";
    return Operacion.listar(sql);
}
}

Este es mi ServicioArticuloImp:

public class ServicioArticuloImp implements ServicioArticulo{
private DaoArticulo da = new DaoArticuloImp();

@Override
public String grabar(String cod, String nom, double pre, int sto) {
    String msg = da.grabar(cod, nom, pre, sto);
    if(msg == null){
        msg="Articulo Grabado";
    }
    return msg;
}

@Override
public Object[] buscar(String cod) {
    return da.buscar(cod);
}

@Override
public String actualizar(String cod, double pre, int sto) {
    String msg = da.actualizar(cod, pre, sto);
    if(msg == null){
        msg="Articulo Actualizado";
    }
    return msg;
}

@Override
public String eliminar(String cod) {
    String msg = da.eliminar(cod);
    if(msg == null){
        msg="Articulo Eliminado";
    }
    return msg;
}

@Override
public List Listar() {
    return da.Listar();
}

}

Y esta es mi clase prueba, en donde me aparece error, la conexión a la base de datos funciona, pero la inserción de los datos, no.

public class Prueba {

public static void main(String[] args) {
    
    if(new Conexion().getConexion()!=null){
        System.out.println("Si");
    } else {
        System.out.println("No");
    }
    
    ServicioArticulo sa = new ServicioArticuloImp();
    String msg = sa.grabar("C001","PAN",2,2);
    System.out.println(msg);
}

}
4
  • 1
    @Benito-B y no ocurre sólo con pre, sino también con sto, el cual es de tipo int en la tabla. La cadena debería ser declarada así: String sql ="insert into articulo values('"+cod+"','"+nom+"',"+pre+","+sto+")"; PERO, todavía eso no es suficiente. La solución segura y sólida en todo aquí es usar consultas preparadas. Más bien, que no funcione ahora es una buena noticia, porque si dejas funcionando este código así tienes abierto en el código un enorme agujero de seguridad.
    – A. Cedano
    Commented el 3 jul. 2023 a las 10:58
  • Efectivamente, @A.Cedano. Coincido en todo lo que has dicho, especialmente en el uso de consultas preparadas (PreparedStatement). La mejor manera de evitar inyección SQL y además te ayuda a localizar problemas de este tipo :)
    – Benito-B
    Commented el 3 jul. 2023 a las 13:04
  • Lectura recomendada: es.stackoverflow.com/questions/10518/… Commented el 3 jul. 2023 a las 13:55
  • A las variables tipo double o int o bit (numéricos) le tienes que quitar las comillas simples. Commented el 3 jul. 2023 a las 14:49

0

Examina otras preguntas con la etiqueta o formula tu propia pregunta.