Update: memory problem fixed

Author: ctr119

BlackJack/BlackJack.depend

@@ -1,16 +1,16 @@
 # depslib dependency file v1.0
-1447608720 source:c:\users\v�ctor\documents\github\getstartedwithc-\blackjack\main.cpp
+1447611050 source:c:\users\v�ctor\documents\github\getstartedwithc-\blackjack\main.cpp
 	<iostream>
 	"MontonCartas.h"
 	"Player.h"
 
-1447606435 c:\users\v�ctor\documents\github\getstartedwithc-\blackjack\montoncartas.h
+1447610743 c:\users\v�ctor\documents\github\getstartedwithc-\blackjack\montoncartas.h
 	<iostream>
 	<ctime>
 	<cstdlib>
 	"Carta.h"
 
-1447608625 source:c:\users\v�ctor\documents\github\getstartedwithc-\blackjack\montoncartas.cpp
+1447611033 source:c:\users\v�ctor\documents\github\getstartedwithc-\blackjack\montoncartas.cpp
 	"MontonCartas.h"
 
 1447585738 c:\users\v�ctor\documents\github\getstartedwithc-\blackjack\player.h

BlackJack/MontonCartas.cpp

@@ -15,15 +15,14 @@ MontonCartas::MontonCartas(int todas)
     Carta *aux;
     bool primeraCarta = true;
 
-    deleted = 0;
-
     for(int i=0; i<4; i++)
     {
         char a = pal[3-i];
-        aux = new Carta;
 
         for(int j=0; j<12; j++)
         {
+            aux = new Carta;
+
             (*aux).palo = a;
             (*aux).num = 12-j;
 
@@ -35,8 +34,6 @@ MontonCartas::MontonCartas(int todas)
                 (*aux).sig = p;
 
             p = aux;
-
-            aux = new Carta;
         }
     }
 
@@ -45,28 +42,12 @@ MontonCartas::MontonCartas(int todas)
 
 MontonCartas::~MontonCartas()
 {
-    Carta *aux;
+    recursiveDelete(p);
+    recursiveDelete(deleted);
+}
 
-    aux = (*p).sig;
-    while(aux != 0)
-    {
-        delete p;
-        p = aux;
-        aux = (*aux).sig;
-    }
-    delete p;
 
 
-    aux = (*deleted).sig;
-    while(aux != 0)
-    {
-        delete deleted;
-        deleted = aux;
-        aux = (*aux).sig;
-    }
-    delete deleted;
-}
-
 MontonCartas& MontonCartas::operator= (const MontonCartas& orig)
 {
     if(&orig != this)
@@ -119,6 +100,20 @@ MontonCartas& MontonCartas::operator= (const MontonCartas& orig)
     return *this;
 }
 
+
+void MontonCartas::recursiveDelete(Carta* &pointer)
+{
+    if(pointer != 0)
+    {
+        if(pointer->sig == 0)
+        {
+            delete pointer;
+        }else{
+            recursiveDelete(pointer->sig);
+        }
+    }
+}
+
 void MontonCartas::intercambiar(int pos1, int pos2)
 {
     Carta* aux1;

BlackJack/MontonCartas.h

@@ -9,10 +9,11 @@ using namespace std;
 class MontonCartas
 {
     Carta *p; // this is our stack, a simple pointer. Yeah, amazing xD
-    Carta *deleted ;
+    Carta *deleted = 0;
 
     void intercambiar(int pos1, int pos2);
     void eliminarCarta(int pos);
+    void recursiveDelete (Carta* &pointer);
 
     public:
         MontonCartas();          // constructor that builds an empty Spanish card stack

BlackJack/bin/Debug/BlackJack.exe

@@ -12,130 +12,6 @@ void on ()
     cout << "\t3 - Exit" <<endl;
 }
 
-void test1()
-{
-    MontonCartas monton (1);
-
-    monton.printMonton();
-
-    cout << endl << "Sacamos una carta: ";
-    Carta* carta = monton.sacarCarta(12);
-
-    cout << (*carta).num << (*carta).palo << endl << endl;
-    if((*carta).sig != 0)
-        cout << "Apunta a otra carta" << endl << endl;
-    else
-        cout << "No apunta a ninguna carta" << endl << endl;
-
-
-    monton.printMonton();
-}
-
-void test2 ()
-{
-    MontonCartas monton (1);
-
-    monton.printMonton();
-
-    int pos = 13;
-    cout << endl << "Posicion: " << pos << endl;
-
-    cout << endl << "Sacamos una carta: ";
-    Carta* carta = monton.sacarCarta( pos );
-
-    cout << (*carta).num << (*carta).palo << endl << endl;
-
-    Player player (1, "Victor");
-    cout << "Se la damos a " << player.getNombre() << endl;
-    double puntuacion = player.addCarta(carta);
-
-    cout<< endl << "Mostramos la mano de " << player.getNombre() << endl;
-    player.showHand();
-
-    cout << endl << "Mostramos la puntuacion de " << player.getNombre() << endl;
-    cout << puntuacion << " punto/s" << endl << endl;
-
-    cout << "Volvemos a mirar el monton" << endl;
-    monton.printMonton();
-}
-
-
-void test3()
-{
-    MontonCartas monton (1);
-
-    monton.printMonton();
-
-    int pos = 13;
-    cout << endl << "Posicion: " << pos << endl;
-
-    cout << endl << "Sacamos dos cartas: " << endl;
-    Carta* carta = monton.sacarCarta( pos );
-    Carta* carta2 = monton.sacarCarta( pos );
-
-    cout << "Carta 1 -> " << (*carta).num << (*carta).palo << endl << endl;
-    cout << "Carta 2 -> " << (*carta2).num << (*carta2).palo << endl << endl;
-
-    Player player (1, "Victor");
-    cout << "Se las damos a " << player.getNombre() << endl;
-    double puntuacion = player.addCarta(carta);
-    puntuacion = player.addCarta(carta2);
-
-    cout<< endl << "Mostramos la mano de " << player.getNombre() << endl;
-    player.showHand();
-
-    cout << endl << "Mostramos la puntuacion de " << player.getNombre() << endl;
-    cout << puntuacion << " punto/s" << endl << endl;
-
-    cout << "Volvemos a mirar el monton" << endl;
-    monton.printMonton();
-}
-
-
-void test4()
-{
-    cout << "Test 4" << endl << endl;
-
-    MontonCartas monton (1);
-
-    cout << "Monton ordenado" << endl;
-    monton.printMonton();
-
-    cout << "Monton mezclado" << endl;
-    monton.mezclar();
-    monton.printMonton();
-
-    int pos = 13;
-    cout << endl << "Posicion: " << pos << endl;
-
-    cout << endl << "Sacamos dos cartas: " << endl;
-    Carta* carta = monton.sacarCarta( pos );
-    Carta* carta2 = monton.sacarCarta( pos );
-
-    cout << "Carta 1 -> " << (*carta).num << (*carta).palo << endl ;
-    cout << "Carta 2 -> " << (*carta2).num << (*carta2).palo << endl << endl;
-
-    Player player (1, "Victor");
-    cout << "Se las damos a " << player.getNombre() << endl;
-    double puntuacion = player.addCarta(carta);
-    puntuacion = player.addCarta(carta2);
-
-    cout<< endl << "Mostramos la mano de " << player.getNombre() << endl;
-    player.showHand();
-
-    cout << endl << "Mostramos la puntuacion de " << player.getNombre() << endl;
-    cout << puntuacion << " punto/s" << endl << endl;
-
-    cout << "Volvemos a mirar el monton" << endl;
-    monton.printMonton();
-
-
-    cout << endl << "Limpiamos la mano de " << player.getNombre() << endl;
-    player.limpiarMano();
-    player.showHand();
-}
-
-
 void test5()
 {
     cout << "Test 5" << endl << endl;
@@ -176,7 +52,6 @@ void test6()
     cout << "Test 6" << endl << endl;
 
     MontonCartas monton (1);
-
     monton.printMonton();
 }