[C con Clase] Solucion calculadora con clase

[Jose Cabrera]

opp menudo fallo no verlo era bastante evidente, pero claro uno va buscando
un pequeño fallo y se deja lo gordo XD

Ahora comparando los dos se ve claramente que creastes unas funciones con
dos argumentos y las usabas con solo uno, ademas de que los valores de
calculadora num1 y num2 eran totalmente inutiles, esto te hace apreciar los
comentarios.

Otra cosa el destructor de la clase es innecesario puesto que no tienes
memoria dinamica entre manos, pero aun asi esta bien que lo pongas es una
buena costumbre, a lo mejor no para clases tan simples como esta pero luego
todo se complica y buenas constumbres como esa te ahorran horas de comerte
la cabeza con un fallo de segmentación.

Un saludo y espero estar mas fino la proxima vez.

El 17 de septiembre de 2009 22:43, Ale Fernandez <
alefernandez_33 en yahoo.com.ar> escribió:

> Gracias a todos los que colaboraron en especial a David fire y Jose Cabrera
> q respondieron, ya encontre los errores y pude solucionar el problema aca se
> los paso por si alguno le interesa tenerla por algun motivo espero que
> colabore algo con el grupo..
>
> #include <iostream>
> using namespace std;
>
> class Calculadora
>     {
>         int val1,
>             val2;
>     public:
>         Calculadora (void);
>
>         void setVal1(int v1);
>         void setVal2(int v2);
>
>         int getVal1(void);
>         int getVal2(void);
>
>         int suma(void);
>         int resta(void);
>         int producto(void);
>         int division(void);
>
>         ~Calculadora (void);
>     };
>
> int main (void)
> {
>     int v1,
>         v2,
>         v,
>         menu,
>         rtdo;
>
>     Calculadora obj1;
>
>     cout << "Ingrese la operacion q desea realizar:"<< endl;
>     cout << "1-SUMA: " << endl;
>     cout << "2-RESTA: " << endl;
>     cout << "3-PRODUCTO: " << endl;
>     cout << "4-DIVISION: " << endl;
>     cin >> menu;
>
>
>     cout << "Ingrese el primer valor: ";
>     cin >> v1;
>     cout << "Ingrese el segundo valor: ";
>     cin >> v2;
>
>
>     obj1.setVal1(v1);
>     obj1.setVal2(v2);
>
>     switch (menu)
>     {
>         case 1:
>             cout << "El resultado es: " << obj1.suma();
>         break;
>         case 2:
>             cout << "El resultado es: " << obj1.resta();
>         break;
>         case 3:
>
>             cout << "El resultado es: " << obj1.producto();
>         break;
>         case 4:
>             cout << "El resultado es: " << obj1.division();
>         break;
>     }
>
>     cin.get();
>
>     return 0;
> }
>
> Calculadora::Calculadora(void)
> {}
>
>
>     int Calculadora::suma (void)
>     {
>         return val1+val2;
>     }
>
>     int Calculadora::resta (void)
>     {
>         return val1-val2;
>     }
>
>     int Calculadora::producto (void)
>     {
>         return val1*val2;
>     }
>
>     int Calculadora::division (void)
>     {
>         return val1/val2;
>     }
>
>     int Calculadora::getVal1 (void)
>     {
>         return val1;
>     }
>
>     int Calculadora::getVal2 (void)
>     {
>         return val2;
>     }
>
>     void Calculadora::setVal1(int v1)
>     {
>         val1=v1;
>     }
>
>     void Calculadora::setVal2(int v2)
>     {
>         val2=v2;
>     }
>
> Calculadora::~Calculadora(void)
> {}
>
>
> // SALU2..
>
> ------------------------------
>
> Encontra las mejores recetas con Yahoo! Cocina.
> http://ar.mujer.yahoo.com/cocina/
>
> _______________________________________________
> Lista de correo Cconclase Cconclase en listas.conclase.net
> http://listas.conclase.net/mailman/listinfo/cconclase_listas.conclase.net
> Bajas: http://listas.conclase.net/index.php?gid=2&mnu=FAQ
>
------------ próxima parte ------------
Se ha borrado un adjunto en formato HTML...
URL: <http://listas.conclase.net/pipermail/cconclase_listas.conclase.net/attachments/20090917/c0db9f40/attachment.html>