C++ programa calculadora

samilobo · 12 de dezembro de 2010

Fiz esse programa , mas não sei onde esta o erro.

#include <cstdlib>
#include <iostream>

using namespace std;

int main(int argc, char *argv[])
{
    int op;
    float n1, n2;
    char resp,n,s;


    cout << " deseja calcular " << endl;
    cin >> resp;

    if (resp == s)
    {

    cout << " digite o primeiro numero " << endl;
    cin >> n1;
    cout << " digite o segundo numero " << endl;
    cin >> n2;

    cout << " 1 = SOMA " <<endl;
    cout << " 2 = SUBTRACAO " << endl;
    cout << " 3 = MULTIPLICACAO "<< endl;
    cout << " 4 = DIVISAO " << endl;

    cout << " digite sua opcao " << endl;
    cin >> op;

     switch (op)
    {
           case '1':
           cout << n1 + n2 << endl;
           break;

           case '2':
           cout <<  n1 - n2 << endl;
           break;

           case '3':
           cout <<  n1 * n2 << endl;
           break;

           case '4':
           cout <<  n1 / n2 << endl;
           break;

           default:
           cout << " Opcao invalida " << endl;
           break;
    }
}
 else
 {
     cout << " você escolheu sair " << endl;
     }


    system("PAUSE");
    return EXIT_SUCCESS;
}

E esta dando esse resultado:

Deseja calcular

S

você escolheu sair

Pressione qla tecla para continuar....

Alguem pode me ajudar.Agraço desde já.

D13g0 · 12 de dezembro de 2010

O erro acontece pois você está comparando o conteúdo da variável resposta com o conteúdo da variável s no teste do if, e como a variável s não foi inicializada, ela guarda lixo de memória.

Para corrigir é só inicializar a variável da seguinte maneira:


char s = 's';

Espero ter ajudado.

editoso · 13 de dezembro de 2010

Bom, primeiro você tem que declarar uma constante pra S, da seguinte forma.

char s = 's', como o nosso amigo D13g0 já comentou. O segundo passo é tirar os valores dos cases de dentro dos parênteses. Dai vai funcionar direitinho.

Exemplo:

switch (op)

{

case 1:

cout << n1 + n2;

break;

case 2:

cout << n1 - n2;

break;

//Dai segue com o restante nesse padrão...ok?

}