resta corregida + un par de asserts

[z.facultad/75.29/dale.git] / src / number.h

diff --git a/src/number.h b/src/number.h
index 578c2b62fd5aa6e45a66f87a94838bccf55d6d82..27d19a848d027b2d59f9d783dcaf8b7b328f7eca 100644 (file)
--- a/src/number.h
+++ b/src/number.h
@@ -50,27 +50,21 @@ struct number
        // Constructores (después de construído, el chunk siempre tiene al
        // menos un elemento).
        // Constructor default (1 'átomo con valor 0)
-       number(): chunk(1, 0) {
-               sign = positive;
-       }
+       number(): chunk(1, 0), sign(positive) {}
 
        // Constructor a partir de buffer (de 'átomos') y tamaño
        // Copia cada elemento del buffer como un 'átomo' del chunk
        // (el átomo menos significativo es el chunk[0] == buf[0])
        number(native_type* buf, size_type len, sign_type s = positive):
-               chunk(buf, buf + len)
+               chunk(buf, buf + len), sign(s)
        {
-               sign = s;
                fix_empty();
        }
 
        // Constructor a partir de un 'átomo' (lo asigna como único elemento
        // del chunk). Copia una vez N en el vector.
        number(native_type n, sign_type s = positive):
-               chunk(1, n)
-       {
-               sign = s;
-       }
+               chunk(1, n), sign(s) {}
 
        number(const std::string& str);
 
@@ -138,19 +132,18 @@ struct number
        // borrow)
        void borrow(size_type i)
        {
-               if (chunk.size() >= i)
+               // para poder pedir prestado debo tener uno a la izquierda
+               assert (chunk.size() >= i);
+               
+               if (chunk[i] == 0)
                {
-                       if (chunk[i] == 0)
-                       {
-                               borrow(i+1); // Overflow, pido prestado
-                               chunk[i] = ~((N)0); //quedo con el valor máximo
-                       }
-                       else
-                       {
-                               --chunk[i]; //tengo para dar, pero pierdo uno yo
-                       }
+                       borrow(i+1); // Overflow, pido prestado
+                       chunk[i] = ~((N)0); //quedo con el valor máximo
+               }
+               else
+               {
+                       --chunk[i]; //tengo para dar, pero pierdo uno yo
                }
-               //else ERROR, están haciendo a-b con a>b
        }
        // Verifica si es un número par
        bool es_impar() const
@@ -332,11 +325,17 @@ number< N, E >& number< N, E >::operator-= (const number< N, E >& n)
                // si no alcanza para restar pido prestado
                if ((minuend.chunk[i] < subtrahend.chunk[i]))
                {
-                       minuend.borrow(i);
+                       // no puedo pedir si soy el más significativo ...
+                       assert (i != fin);
+
+                       // le pido uno al que me sigue
+                       minuend.borrow(i+1);
                }
                
-               // resto el chunk i-ésimo
-               minuend.chunk[i] -= subtrahend.chunk[i];
+               // es como hacer 24-5: el 4 pide prestado al 2 (borrow(i+1)) y después
+               // se hace 4 + (9-5) + 1
+
+               minuend.chunk[i] += (~((N)0) - subtrahend.chunk[i]) + 1; 
        }
 
        //retorno el minuendo ya restado
@@ -403,17 +402,17 @@ number< N, E > operator<< (const number< N, E >& n, typename number< N, E >::siz
        return tmp;
 }
 
-template < typename N, typename E >
-std::ostream& operator<< (std::ostream& os, const number< N, E >& n)
+template < typename NN, typename EE >
+std::ostream& operator<< (std::ostream& os, const number< NN, EE >& n)
 {
        // FIXME sacar una salida bonita en ASCII =)
        if (n.sign == positive)
                os << "+ ";
        else
                os << "- ";
-       for (typename number< N, E >::const_reverse_iterator i = n.chunk.rbegin();
+       for (typename number< NN, EE >::const_reverse_iterator i = n.chunk.rbegin();
                        i != n.chunk.rend(); ++i)
-               os << std::setfill('0') << std::setw(sizeof(N) * 2) << std::hex
+               os << std::setfill('0') << std::setw(sizeof(NN) * 2) << std::hex
                        << *i << " ";
        return os;
 }