Skip to main content
Cambio de la conversion a int.
Origen Enlace
Bicho
  • 1.9k
  • 4
  • 9
  • 16

Debes tener cuidado cuando tienes arreglos limitados; en tu caso tienes un arreglo con límite 6 que se encuentra en un for que recorre 13 veces; entonces vas a tener un error porque no hay posiciones > 6.

Puedes utilizar índices diferentes para los arreglos con límite [6]. En el siguiente código utilizo j y k para esos arreglos:

numPosicionesPares[j];
numPosicionesImpares[k];

Y sumas en 1 el índice: j++;

Entonces quedaría así:

String codBarras = "1234567890123";
    
    boolean codigoValidado = false;
    int[] cadenaOriginal = new int[13];
    int[] numPosicionesPares = new int[13];//Aumenté el tamaño
    int[] numPosicionesImpares = new int[13];//Aumenté el tamaño
    int sumaPar = 0,sumaImpar=0, j=0,k=0;
    
    for (int i = 1; i < cadenaOriginal.length; i++) {
        //asignamos a la cadena El string del codigo de barras
        cadenaOriginal[i] = Integer.parseInt(codBarras.substring((i - 1), (i - 1) + 1));
        if ((i % 2) == 0) {
            numPosicionesPares[j] = cadenaOriginal[i - 1];cadenaOriginal[i];
            //paso 1
            sumaPar+=numPosicionesPares[j];
            j++;
        } else {
            numPosicionesImpares[k] = cadenaOriginal[i - 1];cadenaOriginal[i];
            //paso2
            sumaImpar+=numPosicionesImpares[k];
            k++;
        }
    }
    
    int multiSumaImpar= sumaImpar * 3;
    //paso 4
    int sumaParMasMulti = (sumaPar + multiSumaImpar);
    //paso 5
    int sumaParMasMultiRedondeado = (int) Math.ceil(sumaParMasMulti);   
    int digitoDeControl = sumaParMasMultiRedondeado-sumaParMasMulti;
    if(cadenaOriginal[12] == digitoDeControl) {
        codigoValidado = true;
    }
    System.out.println(codigoValidado); //Imprime false

Otra observación es aquí: cadenaOriginal[12] ojo con el límite que como lo tienes con 13 te da error.

Debes tener cuidado cuando tienes arreglos limitados; en tu caso tienes un arreglo con límite 6 que se encuentra en un for que recorre 13 veces; entonces vas a tener un error porque no hay posiciones > 6.

Puedes utilizar índices diferentes para los arreglos con límite [6]. En el siguiente código utilizo j y k para esos arreglos:

numPosicionesPares[j];
numPosicionesImpares[k];

Y sumas en 1 el índice: j++;

Entonces quedaría así:

String codBarras = "1234567890123";
    
    boolean codigoValidado = false;
    int[] cadenaOriginal = new int[13];
    int[] numPosicionesPares = new int[13];//Aumenté el tamaño
    int[] numPosicionesImpares = new int[13];//Aumenté el tamaño
    int sumaPar = 0,sumaImpar=0, j=0,k=0;
    
    for (int i = 1; i < cadenaOriginal.length; i++) {
        //asignamos a la cadena El string del codigo de barras
        cadenaOriginal[i] = Integer.parseInt(codBarras.substring(i, i + 1));
        if ((i % 2) == 0) {
            numPosicionesPares[j] = cadenaOriginal[i - 1];
            //paso 1
            sumaPar+=numPosicionesPares[j];
            j++;
        } else {
            numPosicionesImpares[k] = cadenaOriginal[i - 1];
            //paso2
            sumaImpar+=numPosicionesImpares[k];
            k++;
        }
    }
    
    int multiSumaImpar= sumaImpar * 3;
    //paso 4
    int sumaParMasMulti = (sumaPar + multiSumaImpar);
    //paso 5
    int sumaParMasMultiRedondeado = (int) Math.ceil(sumaParMasMulti);   
    int digitoDeControl = sumaParMasMultiRedondeado-sumaParMasMulti;
    if(cadenaOriginal[12] == digitoDeControl) {
        codigoValidado = true;
    }
    System.out.println(codigoValidado); //Imprime false

Otra observación es aquí: cadenaOriginal[12] ojo con el límite que como lo tienes con 13 te da error.

Debes tener cuidado cuando tienes arreglos limitados; en tu caso tienes un arreglo con límite 6 que se encuentra en un for que recorre 13 veces; entonces vas a tener un error porque no hay posiciones > 6.

Puedes utilizar índices diferentes para los arreglos con límite [6]. En el siguiente código utilizo j y k para esos arreglos:

numPosicionesPares[j];
numPosicionesImpares[k];

Y sumas en 1 el índice: j++;

Entonces quedaría así:

String codBarras = "1234567890123";
    
    boolean codigoValidado = false;
    int[] cadenaOriginal = new int[13];
    int[] numPosicionesPares = new int[13];//Aumenté el tamaño
    int[] numPosicionesImpares = new int[13];//Aumenté el tamaño
    int sumaPar = 0,sumaImpar=0, j=0,k=0;
    
    for (int i = 1; i < cadenaOriginal.length; i++) {
        //asignamos a la cadena El string del codigo de barras
        cadenaOriginal[i] = Integer.parseInt(codBarras.substring((i - 1), (i - 1) + 1));
        if ((i % 2) == 0) {
            numPosicionesPares[j] = cadenaOriginal[i];
            //paso 1
            sumaPar+=numPosicionesPares[j];
            j++;
        } else {
            numPosicionesImpares[k] = cadenaOriginal[i];
            //paso2
            sumaImpar+=numPosicionesImpares[k];
            k++;
        }
    }
    
    int multiSumaImpar= sumaImpar * 3;
    //paso 4
    int sumaParMasMulti = (sumaPar + multiSumaImpar);
    //paso 5
    int sumaParMasMultiRedondeado = (int) Math.ceil(sumaParMasMulti);   
    int digitoDeControl = sumaParMasMultiRedondeado-sumaParMasMulti;
    if(cadenaOriginal[12] == digitoDeControl) {
        codigoValidado = true;
    }
    System.out.println(codigoValidado); //Imprime false

Otra observación es aquí: cadenaOriginal[12] ojo con el límite que como lo tienes con 13 te da error.

Mejoras en la respuesta.
Origen Enlace
Bicho
  • 1.9k
  • 4
  • 9
  • 16

Debes tener cuidado cuando tienes arreglos limitados; en tu caso tienes un arreglo con límite 6 que se encuentra en un for que recorre 13 veces; entonces vas a tener un error porque no hay posiciones > 6.

Puedes utilizar índices diferentes para los arreglos con límite [6]. En el siguiente código utilizo j y k para esos arreglos:

numPosicionesPares[j];
numPosicionesImpares[k];

Y sumas en 1 el índice: j++;

Entonces quedaría así:

String codBarras = "1234567890123";
    
    boolean codigoValidado = false;
    int[] cadenaOriginal = new int[13];
    int[] numPosicionesPares = new int[13];//Aumenté el tamaño
    int[] numPosicionesImpares = new int[13];//Aumenté el tamaño
    int sumaPar = 0,sumaImpar=0, j=0,k=0;
    
    for (int i = 0;1; i < cadenaOriginal.length; i++) {
        //asignamos a la cadena El string del codigo de barras
        cadenaOriginal[i] = Integer.parseInt(codBarras.substring(i, i + 1));
        if ((cadenaOriginal[i]i % 2) == 0) {
            numPosicionesPares[j] = cadenaOriginal[i];cadenaOriginal[i - 1];
            //paso 1
            sumaPar+=numPosicionesPares[j];
            j++;
        } else {
            numPosicionesImpares[k] = cadenaOriginal[i];cadenaOriginal[i - 1];
            //paso2
            sumaImpar+=numPosicionesImpares[k];
            k++;
        }
    }
    
    int multiSumaImpar= sumaImpar * 3;
    //paso 4
    int sumaParMasMulti = (sumaPar + multiSumaImpar);
    //paso 5
    int sumaParMasMultiRedondeado = (int) Math.ceil(sumaParMasMulti);   
    int digitoDeControl = sumaParMasMultiRedondeado-sumaParMasMulti;
    if(cadenaOriginal[12] == digitoDeControl) {
        codigoValidado = true;
    }
    System.out.println(codigoValidado); //Imprime false

Otra observación es aquí: cadenaOriginal[12] ojo con el límite que como lo tienes con 13 te da error.

Debes tener cuidado cuando tienes arreglos limitados; en tu caso tienes un arreglo con límite 6 que se encuentra en un for que recorre 13 veces; entonces vas a tener un error porque no hay posiciones > 6.

Puedes utilizar índices diferentes para los arreglos con límite [6]. En el siguiente código utilizo j y k para esos arreglos:

numPosicionesPares[j];
numPosicionesImpares[k];

Y sumas en 1 el índice: j++;

Entonces quedaría así:

String codBarras = "1234567890123";
    
    boolean codigoValidado = false;
    int[] cadenaOriginal = new int[13];
    int[] numPosicionesPares = new int[13];//Aumenté el tamaño
    int[] numPosicionesImpares = new int[13];//Aumenté el tamaño
    int sumaPar = 0,sumaImpar=0, j=0,k=0;
    
    for (int i = 0; i < cadenaOriginal.length; i++) {
        //asignamos a la cadena El string del codigo de barras
        cadenaOriginal[i] = Integer.parseInt(codBarras.substring(i, i + 1));
        if ((cadenaOriginal[i] % 2) == 0) {
            numPosicionesPares[j] = cadenaOriginal[i];
            //paso 1
            sumaPar+=numPosicionesPares[j];
            j++;
        } else {
            numPosicionesImpares[k] = cadenaOriginal[i];
            //paso2
            sumaImpar+=numPosicionesImpares[k];
            k++;
        }
    }
    
    int multiSumaImpar= sumaImpar * 3;
    //paso 4
    int sumaParMasMulti = (sumaPar + multiSumaImpar);
    //paso 5
    int sumaParMasMultiRedondeado = (int) Math.ceil(sumaParMasMulti);   
    int digitoDeControl = sumaParMasMultiRedondeado-sumaParMasMulti;
    if(cadenaOriginal[12] == digitoDeControl) {
        codigoValidado = true;
    }
    System.out.println(codigoValidado); //Imprime false

Otra observación es aquí: cadenaOriginal[12] ojo con el límite que como lo tienes con 13 te da error.

Debes tener cuidado cuando tienes arreglos limitados; en tu caso tienes un arreglo con límite 6 que se encuentra en un for que recorre 13 veces; entonces vas a tener un error porque no hay posiciones > 6.

Puedes utilizar índices diferentes para los arreglos con límite [6]. En el siguiente código utilizo j y k para esos arreglos:

numPosicionesPares[j];
numPosicionesImpares[k];

Y sumas en 1 el índice: j++;

Entonces quedaría así:

String codBarras = "1234567890123";
    
    boolean codigoValidado = false;
    int[] cadenaOriginal = new int[13];
    int[] numPosicionesPares = new int[13];//Aumenté el tamaño
    int[] numPosicionesImpares = new int[13];//Aumenté el tamaño
    int sumaPar = 0,sumaImpar=0, j=0,k=0;
    
    for (int i = 1; i < cadenaOriginal.length; i++) {
        //asignamos a la cadena El string del codigo de barras
        cadenaOriginal[i] = Integer.parseInt(codBarras.substring(i, i + 1));
        if ((i % 2) == 0) {
            numPosicionesPares[j] = cadenaOriginal[i - 1];
            //paso 1
            sumaPar+=numPosicionesPares[j];
            j++;
        } else {
            numPosicionesImpares[k] = cadenaOriginal[i - 1];
            //paso2
            sumaImpar+=numPosicionesImpares[k];
            k++;
        }
    }
    
    int multiSumaImpar= sumaImpar * 3;
    //paso 4
    int sumaParMasMulti = (sumaPar + multiSumaImpar);
    //paso 5
    int sumaParMasMultiRedondeado = (int) Math.ceil(sumaParMasMulti);   
    int digitoDeControl = sumaParMasMultiRedondeado-sumaParMasMulti;
    if(cadenaOriginal[12] == digitoDeControl) {
        codigoValidado = true;
    }
    System.out.println(codigoValidado); //Imprime false

Otra observación es aquí: cadenaOriginal[12] ojo con el límite que como lo tienes con 13 te da error.

Cambio de la condición.
Origen Enlace
Bicho
  • 1.9k
  • 4
  • 9
  • 16

Debes tener cuidado cuando tienes arreglos limitados; en tu caso tienes un arreglo con límite 6 que se encuentra en un for que recorre 13 veces; entonces vas a tener un error porque no hay posiciones > 6.

Puedes utilizar índices diferentes para los arreglos con límite [6]. En el siguiente código utilizo j y k para esos arreglos:

numPosicionesPares[j];
numPosicionesImpares[k];

Y sumas en 1 el índice: j++;

Entonces quedaría así:

String codBarras = "1234567890123";
    
    boolean codigoValidado = false;
    int[] cadenaOriginal = new int[13];
    int[] numPosicionesPares = new int[13];//Aumenté el tamaño
    int[] numPosicionesImpares = new int[13];//Aumenté el tamaño
    int sumaPar = 0,sumaImpar=0, j=0,k=0;
    
    for (int i = 0; i < cadenaOriginal.length; i++) {
        //asignamos a la cadena El string del codigo de barras
        cadenaOriginal[i] = Integer.parseInt(codBarras.substring(i, i + 1));
        System.out.println(cadenaOriginal[i]);
        if ((icadenaOriginal[i] % 2) == 0) {
            numPosicionesPares[j] = cadenaOriginal[i];
            //paso 1
            sumaPar+=numPosicionesPares[j];
            j++;
        } else {
            numPosicionesImpares[k] = cadenaOriginal[i];
            //paso2
            sumaImpar+=numPosicionesImpares[k];
            k++;
        }
    }
    
    int multiSumaImpar= sumaImpar * 3;
    //paso 4
    int sumaParMasMulti = (sumaPar + multiSumaImpar);
    //paso 5
    int sumaParMasMultiRedondeado = (int) Math.ceil(sumaParMasMulti);   
    int digitoDeControl = sumaParMasMultiRedondeado-sumaParMasMulti;
    if(cadenaOriginal[12] == digitoDeControl) {
        codigoValidado = true;
    }
    System.out.println(codigoValidado); //Imprime false

Otra observación es aquí: cadenaOriginal[12] ojo con el límite que como lo tienes con 13 te da error.

Debes tener cuidado cuando tienes arreglos limitados; en tu caso tienes un arreglo con límite 6 que se encuentra en un for que recorre 13 veces; entonces vas a tener un error porque no hay posiciones > 6.

Puedes utilizar índices diferentes para los arreglos con límite [6]. En el siguiente código utilizo j y k para esos arreglos:

numPosicionesPares[j];
numPosicionesImpares[k];

Y sumas en 1 el índice: j++;

Entonces quedaría así:

String codBarras = "1234567890123";
    
    boolean codigoValidado = false;
    int[] cadenaOriginal = new int[13];
    int[] numPosicionesPares = new int[13];//Aumenté el tamaño
    int[] numPosicionesImpares = new int[13];//Aumenté el tamaño
    int sumaPar = 0,sumaImpar=0, j=0,k=0;
    
    for (int i = 0; i < cadenaOriginal.length; i++) {
        //asignamos a la cadena El string del codigo de barras
        cadenaOriginal[i] = Integer.parseInt(codBarras.substring(i, i + 1));
        System.out.println(cadenaOriginal[i]);
        if ((i % 2) == 0) {
            numPosicionesPares[j] = cadenaOriginal[i];
            //paso 1
            sumaPar+=numPosicionesPares[j];
            j++;
        } else {
            numPosicionesImpares[k] = cadenaOriginal[i];
            //paso2
            sumaImpar+=numPosicionesImpares[k];
            k++;
        }
    }
    
    int multiSumaImpar= sumaImpar * 3;
    //paso 4
    int sumaParMasMulti = (sumaPar + multiSumaImpar);
    //paso 5
    int sumaParMasMultiRedondeado = (int) Math.ceil(sumaParMasMulti);   
    int digitoDeControl = sumaParMasMultiRedondeado-sumaParMasMulti;
    if(cadenaOriginal[12] == digitoDeControl) {
        codigoValidado = true;
    }
    System.out.println(codigoValidado); //Imprime false

Otra observación es aquí: cadenaOriginal[12] ojo con el límite que como lo tienes con 13 te da error.

Debes tener cuidado cuando tienes arreglos limitados; en tu caso tienes un arreglo con límite 6 que se encuentra en un for que recorre 13 veces; entonces vas a tener un error porque no hay posiciones > 6.

Puedes utilizar índices diferentes para los arreglos con límite [6]. En el siguiente código utilizo j y k para esos arreglos:

numPosicionesPares[j];
numPosicionesImpares[k];

Y sumas en 1 el índice: j++;

Entonces quedaría así:

String codBarras = "1234567890123";
    
    boolean codigoValidado = false;
    int[] cadenaOriginal = new int[13];
    int[] numPosicionesPares = new int[13];//Aumenté el tamaño
    int[] numPosicionesImpares = new int[13];//Aumenté el tamaño
    int sumaPar = 0,sumaImpar=0, j=0,k=0;
    
    for (int i = 0; i < cadenaOriginal.length; i++) {
        //asignamos a la cadena El string del codigo de barras
        cadenaOriginal[i] = Integer.parseInt(codBarras.substring(i, i + 1));
        if ((cadenaOriginal[i] % 2) == 0) {
            numPosicionesPares[j] = cadenaOriginal[i];
            //paso 1
            sumaPar+=numPosicionesPares[j];
            j++;
        } else {
            numPosicionesImpares[k] = cadenaOriginal[i];
            //paso2
            sumaImpar+=numPosicionesImpares[k];
            k++;
        }
    }
    
    int multiSumaImpar= sumaImpar * 3;
    //paso 4
    int sumaParMasMulti = (sumaPar + multiSumaImpar);
    //paso 5
    int sumaParMasMultiRedondeado = (int) Math.ceil(sumaParMasMulti);   
    int digitoDeControl = sumaParMasMultiRedondeado-sumaParMasMulti;
    if(cadenaOriginal[12] == digitoDeControl) {
        codigoValidado = true;
    }
    System.out.println(codigoValidado); //Imprime false

Otra observación es aquí: cadenaOriginal[12] ojo con el límite que como lo tienes con 13 te da error.

Origen Enlace
Bicho
  • 1.9k
  • 4
  • 9
  • 16
Loading