[SOLUCIONADO] Por que mi codigo multiplica en lugar de sumar?

[godeffuffeff]

buenas, soy nuevo programando y tengo un problema con mi codigo, se supone que deberia sumar pero multiplica el resultado, a la hora de escribir el primer numero tendria que asignarle el valor escrito a la variable tal como el usuario lo escribio, pero por alguna razon la cual soy incapaz de comprender el primer numero se multiplica por dos y es hasta la segunda vez que se ingresa un valor cuando suma.

agradezco su ayuda

Código C++:

#include<iostream>
using namespace std;
 
class ClaseCuenta
{
public:
    int obtenerSaldo ()
    {
        return saldoTotal;
    }
 
    void agregarSaldo (int saldo)
    {
        if(saldoTotal == 0)
        saldoTotal =  saldo;
 
        if(saldoTotal > 0)
        saldoTotal = saldo + saldoTotal;
    }
 
    void retirarSaldo (int saldo)
    {
        saldoTotal = saldoTotal - saldo;
    }
 
private:
    int saldoTotal = 0;
};
 
int main()
{
    ClaseCuenta Cuenta1;
    int valor = 0;
 
    cout<<"el saldo es : ";
    Cuenta1.obtenerSaldo();
 
    cout<<"\ncuanto desea agreagar : ";
    cin>>valor;
    Cuenta1.agregarSaldo(valor);
    cout<<"\nel saldo actual es : ";
    cout<<Cuenta1.obtenerSaldo();
 
 
    cout<<"\ncuanto desea agreagar : ";
    cin>>valor;
    Cuenta1.agregarSaldo(valor);
    cout<<"\nel saldo actual es : ";
    cout<<Cuenta1.obtenerSaldo();
 
}
 

[dehm]

Hola, esto te pasa porque has puesto dos if en la función agregarSaldo(float) que no son excluyentes. Es decir, cuando entras en esa función se evalúan los dos if.
La primera vez, cuando no hay saldo, esto es lo que sucede:

Código C++:

if(saldoTotal == 0)
        saldoTotal =  saldo;
 

Como saldoTotal es 0, se añade saldo al total....pero se evalúa el siguiente if:

Código C++:

if(saldoTotal > 0)
            saldoTotal = saldo + saldoTotal;
 

Como ahora saldoTotal ya es >0 por la sentencia anterior, se le vuelve a añadir el valor de saldo

En las sucesivas llamadas a la función ya no se cumple e primer if, porque saldoTotal ya es mayor que 0, luego sólo se ejecuta el segundo.

Para solucionarlo has de hacer que los dos if sean excluyentes :

Código C++:

if{}
else if{}
 

Aún así no le veo mucha lógica a la función propiamente dicha. Se debería limitar a sumar la cantidad deseada a saldoTotal, sin hacer distinciones si saldoTotal es = a 0 o mayor.

[godeffuffeff]

entonces crees que seria mas eficiente poner el codigo de la siguiente manera?

otra cosa, me podrias explicar de una forma no muy tecnica pero concisa, que es un if excluyente?

Código C++:

void agregarSaldo (int saldo)
    {
        saldoTotal = saldo + saldoTotal;
    }
 

[dehm]

Hola:

Respecto a lo primero, pues si, no tiene mucho sentido distinguir si el saldoTotal es 0 o no.
Porque esto:

Código C++:

saldoTotal =  saldo;
 

y esto:

Código C++:

saldoTotal = saldo + saldoTotal;//saldoTotal = saldo + 0
 

Es lo mismo cuando saldoTotal es = a 0, luego no ha lugar la distinción.

Respecto a los if excluyentes....bueno, no sé si es muy correcto llamarlos así , pero más o menos:

Esta forma:

Código C++:

if(condicion)
{
    haz una cosa
}
if(otra condicion)
{
    haz otra cosa
}
 

implica que lo que sea va a ser evaluado dos veces, primero para ver si cumple "condicion" y luego para ver si cumple "otra condicion". Es el caso de tu código.

Por otro lado, si hago así:

Código C++:

if(condicion)
{
    haz una cosa
}
else if(otra condicion)
{
    haz otra cosa
}
 

La segunda condición no se evaluará si se ha cumplido la primera.
Y queda:

Código C++:

if(condicion)
{
    haz una cosa
}
else
{
    haz otra cosa
}
 

En este caso se ejecutará el bloque después del else si no se cumple la primera condición.

Mira este ejemplo:

Código C++:

int main()
    {
        int num=100;
        if (num==100)
        {
            cout<<"numero = 100"<<endl;
        }
        if (num>50)
        {
            cout<<"numero > 50"<<endl;
        }
    }
 

En este caso la variable num cumple la doble condición, es = 100 y es > 50
Tal como está el código va a evaluar las dos condiciones y si son ciertas va a ejecutar el bloque que hay dentro.
Pero si cambias la línea:
if (num>50)
por
else if (num>50)
verás que aún siendo cierta la segunda condición no se evalúa (y por tanto no se ejecuta) ya que al ser cierta la primera condición no se sigue evaluando.