Im not getting the expected result

in the part of this code I expect the variable precioFinal ( " precioFinal += costoFinal; " ) to acumulate all the costs of all the buyed items but instead of that it only returns the price of the last buyed item. what am I doing wrong?

Code:

---

//****************( Factura.h )****************
 
#include<string>
using namespace std;
 
class Factura{
 
    public:
 
        Factura();
 
        void establecerDatos( string, string, int, int );
 
        void mostrarMensaje();
 
        void procesar( int, int );
 
        string obtenerNumeroDePieza();
        string obtenerDescripcionDePieza();
        int obtenerCantidadDeArticulos();
        int obtenerPrecioDelArticulo();
        int obtenerPrecioFinal();
        int obtenerCostoFinal();
        int obtenerTotal();
 
    private:
 
        string numeroDePieza;
        string descripcionDePieza;
        int cantidadDeArticulos;
        int precioDeArticulo;
        int precioFinal;
        int costoFinal;
};
 
//****************( end of Factura.h )****************

---

Code:

---

//****************( Factura.cpp )****************
 
#include<iostream>
#include"Factura.h"
using namespace std;
 
Factura::Factura(){
 
    establecerDatos( numeroDePieza, descripcionDePieza, cantidadDeArticulos, precioDeArticulo );
}
 
void Factura::establecerDatos( string numeroPieza, string descripcionPieza, int cantidad, int precio ){
 
    numeroDePieza = numeroPieza;
    descripcionDePieza = descripcionPieza;
    cantidadDeArticulos = cantidad;
    precioDeArticulo = precio;
}
 
void Factura::mostrarMensaje(){
 
    char decidir;
    int cantidadAcomprar;
 
    cout << "Bienvenidos a Ferreteria Don Juan : que es lo que necesita? : \n\n"
         << "Contamos con a)Foco, b)cable, c)pintura, d)martillo, e)cierra, f)tijeras\n"
         << "g)lija, h)solvente, i)desarmador, y j)fusibles\n\nintroduzca una opcion\n\n";
 
    cin  >> decidir;
 
    switch( decidir ){
 
        case 'a':
 
            cout << "Cuantos Focos desea comprar ? : ";
 
            cin  >> cantidadAcomprar;
 
            establecerDatos( "14268596", "Foco ahorrador luz Calida", cantidadAcomprar, 127 );
 
            procesar( cantidadAcomprar, 127 );
 
            cout << "Usted compro codigo : " << obtenerNumeroDePieza() << " " << obtenerDescripcionDePieza() << " cantidad " << obtenerCantidadDeArticulos() << " precio " << obtenerCostoFinal() << "\n\n";
 
        break;
 
        case 'b':
 
            cout << "Cuantos cables desea comprar ? : ";
 
            cin  >> cantidadAcomprar;
 
            establecerDatos( "16589742", "CableCoaxial blanco 3 metros", cantidadAcomprar, 65 );
 
            procesar( cantidadAcomprar, 65 );
 
            cout << "Usted compro codigo : " << obtenerNumeroDePieza() << " " << obtenerDescripcionDePieza() << " cantidad " << obtenerCantidadDeArticulos() << " precio " << obtenerCostoFinal() << "\n\n";
 
         break;
 
         case 'c':
 
            cout << "Cuantos pinturas desea comprar ? : ";
 
            cin  >> cantidadAcomprar;
 
            establecerDatos( "42698816", "Pintura acrilica blanca 1 litro", cantidadAcomprar, 74 );
 
            procesar( cantidadAcomprar, 74 );
 
            cout << "Usted compro codigo : " << obtenerNumeroDePieza() << " " << obtenerDescripcionDePieza() << " cantidad " << obtenerCantidadDeArticulos() << " precio " << obtenerCostoFinal() << "\n\n";
 
         break;
 
         case 'd':
 
            cout << "Cuantos martillos desea comprar ? : ";
 
            cin  >> cantidadAcomprar;
 
            establecerDatos( "72698756", "Martillo metalico chico", cantidadAcomprar, 50 );
 
            procesar( cantidadAcomprar, 50 );
 
            cout << "Usted compro codigo : " << obtenerNumeroDePieza() << " " << obtenerDescripcionDePieza() << " cantidad " << obtenerCantidadDeArticulos() << " precio " << obtenerCostoFinal() << "\n\n";
 
         break;
 
         case 'e':
 
            cout << "Cuantas cierras desea comprar ? : ";
 
            cin  >> cantidadAcomprar;
 
            establecerDatos( "26978543", "Cierra metalica con cegueta", cantidadAcomprar, 83 );
 
            procesar( cantidadAcomprar, 83 );
 
            cout << "Usted compro codigo : " << obtenerNumeroDePieza() << " " << obtenerDescripcionDePieza() << " cantidad " << obtenerCantidadDeArticulos() << " precio " << obtenerCostoFinal() << "\n\n";
 
         break;
 
         case 'f':
 
            cout << "Cuantas tijeras desea comprar ? : ";
 
            cin  >> cantidadAcomprar;
 
            establecerDatos( "98542689", "Tijeras metalicas para electricista", cantidadAcomprar, 59 );
 
            procesar( cantidadAcomprar, 59 );
 
            cout << "Usted compro codigo : " << obtenerNumeroDePieza() << " " << obtenerDescripcionDePieza() << " cantidad " << obtenerCantidadDeArticulos() << " precio " << obtenerCostoFinal() << "\n\n";
 
         break;
 
         case 'g':
 
            cout << "Cuantas lijas desea comprar ? : ";
 
            cin  >> cantidadAcomprar;
 
            establecerDatos( "15829527", "Lija de agua numero 1", cantidadAcomprar, 12 );
 
            procesar( cantidadAcomprar, 12 );
 
            cout << "Usted compro codigo : " << obtenerNumeroDePieza() << " " << obtenerDescripcionDePieza() << " cantidad " << obtenerCantidadDeArticulos() << " precio " << obtenerCostoFinal() << "\n\n";
 
         break;
 
         case 'h':
 
            cout << "Cuantos solventes desea comprar ? : ";
 
            cin  >> cantidadAcomprar;
 
            establecerDatos( "17983366", "Solvente thinner para pintura 1 litro", cantidadAcomprar, 18 );
 
            procesar( cantidadAcomprar, 18 );
 
            cout << "Usted compro codigo : " << obtenerNumeroDePieza() << " " << obtenerDescripcionDePieza() << " cantidad " << obtenerCantidadDeArticulos() << " precio " << obtenerCostoFinal() << "\n\n";
 
         break;
 
         case 'i':
 
            cout << "Cuantos desarmadores desea comprar ? : ";
 
            cin  >> cantidadAcomprar;
 
            establecerDatos( "13263345", "Desarmador de cruz con mango de goma", cantidadAcomprar, 30 );
 
            procesar( cantidadAcomprar, 30 );
 
            cout << "Usted compro codigo : " << obtenerNumeroDePieza() << " " << obtenerDescripcionDePieza() << " cantidad " << obtenerCantidadDeArticulos() << " precio " << obtenerCostoFinal() << "\n\n";
 
         break;
 
         case 'j':
 
            cout << "Cuantos fusibles desea comprar ? : ";
 
            cin  >> cantidadAcomprar;
 
            establecerDatos( "32269564", "Fusible para caja electrica residencial", cantidadAcomprar, 8 );
 
            procesar( cantidadAcomprar, 8 );
 
            cout << "Usted compro codigo : " << obtenerNumeroDePieza() << " " << obtenerDescripcionDePieza() << " cantidad " << obtenerCantidadDeArticulos() << " precio " << obtenerCostoFinal() << "\n\n";
 
         break;
    }
}
 
    void Factura::procesar( int cantidad, int precio ){
 
        precioFinal = 0;
 
        costoFinal = cantidad * precio;
 
        precioFinal += costoFinal;
 
    }
 
    string Factura::obtenerNumeroDePieza(){
 
        return numeroDePieza;
    }
 
    string Factura::obtenerDescripcionDePieza(){
 
        return descripcionDePieza;
    }
 
    int Factura::obtenerCantidadDeArticulos(){
 
        return cantidadDeArticulos;
    }
 
    int Factura::obtenerPrecioDelArticulo(){
 
        return precioDeArticulo;
    }
 
  int Factura::obtenerPrecioFinal(){
 
        return precioFinal;
  }
 
  int Factura::obtenerCostoFinal(){
 
        return costoFinal;
  }
 
//****************( end of Factura.cpp )****************

---

Code:

---

//****************( main.cpp )****************
 
using namespace std;
 
int main(){
 
    int seleccionar;
    Factura factura1;
    factura1.mostrarMensaje();
    cout << "\n\nDesea comprar mas cosas? : 1)Si  2)No : ";
    cin  >> seleccionar;
 
        while( seleccionar == 1 ){
 
            factura1.mostrarMensaje();
            cout << "\n\nDesea comprar mas cosas? : 1)Si  2)No : ";
            cin  >> seleccionar;
        }
 
        if( seleccionar == 2 ){
 
            cout << "Gracias por su compra\n\nEl total a pagar es : " << factura1.obtenerPrecioFinal();
        }
 
    system("PAUSE");
    return 0;
}
 
//****************( end of main.cpp )****************

---

Quote:

---

Originally Posted by sauerplayer

in the part of this code I expect the variable precioFinal ( " precioFinal += costoFinal; " ) to acumulate all the costs of all the buyed items

---

Are you talking about the code in Factura::procesar()?
Why would you expect that to hold a sum?
You are setting the variable to 0 on every call.

Cheers,
_

Quote:

---

Originally Posted by anda_skoa

Are you talking about the code in Factura::procesar()?
Why would you expect that to hold a sum?
You are setting the variable to 0 on every call.

Cheers,
_

---

so If I modify this code will it work?

Code:

---

void Factura::procesar( int cantidad, int precio ){
 
        costoFinal = cantidad * precio;
 
        precioFinal += costoFinal;
 
    }

---

Quote:

---

Originally Posted by sauerplayer

so If I modify this code will it work?

---

You are not serioulsy asking that instead of trying it, right?

Cheers,
_

Quote:

---

Originally Posted by anda_skoa

You are not serioulsy asking that instead of trying it, right?

Cheers,
_

---

I found the and fixed the problem, now it works correctly, the problem was that I did not initialize PrecioFinal before using it, but it had to be done inside the constructor...

Factura::Factura(){

precioFinal = 0;

establecerDatos( numeroDePieza = "", descripcionDePieza = "", cantidadDeArticulos = 0, precioDeArticulo = 0 );
}

still I do not undesrtand why, if I try to initialize precioFinal to 0 inside Factura.h it fails to do so...

the error message I get for this is : C:\Users\eyel\Desktop\CBlocks\Clase9\Factura.h|30|warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11|

Quote:

---

Originally Posted by sauerplayer

still I do not undesrtand why, if I try to initialize precioFinal to 0 inside Factura.h it fails to do so...

the error message I get for this is : C:\Users\eyel\Desktop\CBlocks\Clase9\Factura.h|30|warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11|

---

As the compiler hints that is available from the C++11 standard onwards.
If you are building for an older standard then you need to use assignment in code or the constructor's initialization list.

Cheers,
_