Ayuda a depurar este algoritmo de multiplicación

Estoy implementando el algoritmo de multiplicación simple para comparar su desempeño con un enfoque de divide y vencerás.

Abandoné mi idea original de hacer esto con aritmética de bytes y opté por convertir los números a través de matrices de caracteres.

Bueno, todo funciona bien en casos simples como 33 x 33, donde el método de depuración se imprime correctamente:

0 9 9
0 9 9

sin embargo, en 34 x 33, obtengo

1 0 3
1 0 2

donde debería estar

1 0 2
1 0 2

¿De dónde viene ese 3?

public static BigInteger simpleMultiply(BigInteger x, BigInteger y){

    char [] longerNum;
    char [] shorterNum;

    if(x.compareTo(y)>=0){ // x is a longer/equal num

        longerNum = x.toString().toCharArray();
        shorterNum = y.toString().toCharArray();

    }

   else { //y is a longer num

       longerNum = y.toString().toCharArray();
       shorterNum = x.toString().toCharArray();

   }


   //shorter num equals the number of rows in partial result
   // longer num + 1 equals the number of columns in partial result


    int [][] partialResult = new int [shorterNum.length][longerNum.length+1];

    int pastCarry=0;
    int result=0;
    int carry=0;

    for (int sIndex=(shorterNum.length-1); sIndex>=0; sIndex--)
        for (int lIndex = (longerNum.length-1); lIndex>=0; lIndex--)
        {
            int sInt = Integer.parseInt(""+shorterNum[sIndex]+"");
            int lInt = Integer.parseInt(""+longerNum[lIndex]+"");

            int product = sInt*lInt;

            if (lIndex==0){

             result  =  (pastCarry+product)% 10;
             carry   = (pastCarry+product) /  10;

             pastCarry = carry;

             partialResult [sIndex][lIndex+1] = result; //one more column element in partialResult

             partialResult[sIndex][lIndex] = carry;


           }

            else {

             result  = (pastCarry+product)% 10;
             carry   = (pastCarry+product) /  10;

             pastCarry = carry;

             partialResult [sIndex][lIndex+1] = result;//one more column element in partialResult


            }

        }

        for (int i=0; i<partialResult.length;i++)
            for (int j=0; j<partialResult[0].length;j++)
            {

                  System.out.print(partialResult[i][j] + " ");
                  if (j==partialResult[0].length-1){System.out.println();}
            }





    return null;
 }

preguntado el 16 de mayo de 11 a las 20:05

1 Respuestas

Es porque no te estás poniendo a cero pastCarry entre iteraciones del bucle exterior.

Cambiar

for (int sIndex=(shorterNum.length-1); sIndex>=0; sIndex--)
    for (int lIndex = (longerNum.length-1); lIndex>=0; lIndex--)
    {
        [snip]
    }

a

for (int sIndex=(shorterNum.length-1); sIndex>=0; sIndex--)
{
    pastCarry = 0;
    for (int lIndex = (longerNum.length-1); lIndex>=0; lIndex--)
    {
        [snip]
    }
}

Y por cierto, el int sInt = .... La línea también debe moverse fuera del bucle interno, ya que no depende de nada dentro del bucle interno.

contestado el 17 de mayo de 11 a las 04:05

No es la respuesta que estás buscando? Examinar otras preguntas etiquetadas or haz tu propia pregunta.