Un extraño problema...programa en C++

[andres31415]

Hola a todos...soy un estudiante nuevo en el mundo de C++ por lo que tengo poca experiencia. Necesito crear un pequeño programa que calcule el area de ciertas figuras geometricas (cuadrado, triangulo , circulo). Estaba probando con " if " y con "else" pero tengo varios errores..aqui esta el codigo con el que he estado trabajando:


#include <cstdlib>
#include <iostream>
#include <stdio.h>
#include <conio.h>
#include <string.h>

using namespace std;

int main()
{
float figura,cuadrado,triangulo,circulo,lado,base,altura ,radio;
cout<<"digite el nombre de la figura "<<endl;
cin>>figura;
if(figura==cuadrado)
{
cout<<"Digite el valor del lado del cuadrado"<<endl;
cin>>lado;
cout<<"el area del cuadrado es:"<<lado*lado<<endl;
}
else
{
if(figura==triangulo)
{
cout<<"digite el valor de la base del triangulo"<<endl;
cin>>base;
cout<<"digite el valor de la altura del triangulo"<<endl;
cin>>altura;
cout<<"el area del triangulo es:"<<base*altura/2<<endl;
}
else
{
if(figura==circulo)
{
cout<<"digite el valor del radio del circulo"<<endl;
cin>>radio;
cout<<"el area del circulo es:"<<3.1514*radio*radio<<endl;
}
}
}

system("PAUSE");
return EXIT_SUCCESS;
}


Gracias a la persona o a las personas que puedan corregirlo. Existe otra forma de hacer el mismo programa??

[Heimish2000]

Vas comparando figura con otras variables float...a las que no has dado ningún valor y que, en todo caso, serían constantes, no variables.

Mejor que con if-else puedes hacerlo con un switch-case, que es más claro.

A tu pregunta que si hay más maneras de hacerlo te diré que sí, pero como todos los programas, siempre hay muchas maneras de hacer lo mismo.

[andres31415]

Iniciado por Heimish2000

Vas comparando figura con otras variables float...a las que no has dado ningún valor y que, en todo caso, serían constantes, no variables.

Mejor que con if-else puedes hacerlo con un switch-case, que es más claro.

A tu pregunta que si hay más maneras de hacerlo te diré que sí, pero como todos los programas, siempre hay muchas maneras de hacer lo mismo.

Gracias por tu ayuda. Me gustaria saber como utilizar un Switch-case para mi problema en particular...entiendo que debe ser una herramienta poderosa pero ahun no tengo la suficiente preparacion para utilizarla. Tal vez alguien con experiencia como tu pueda guiarme...De nuevo gracias.

[sam90]

Te estas desviando un poco de lo importante. El swicth-case es solo un estructura similar al if. nada mas. Busca en internet la sintaxis y listo....

Tu problema aca es que vos creaste variables del tipo float y por lo que dice el programa estas tratandolas de usar como si fuesen string.

Código C++:

#include <cstdlib>
#include <iostream>
#include <stdio.h>
#include <conio.h>
#include <string>
 
using namespace std;
 
int main()
{
string figura;
float lado,base,altura ,radio;
cout<<"digite el nombre de la figura "<<endl;
cin>>figura;
 
if(figura.compare("cuadrado") == 0)  {
          cout<<"Digite el valor del lado del cuadrado"<<endl;
          cin>>lado;
          cout<<"el area del cuadrado es:"<<lado*lado<<endl;
}
else if(figura.compare("triangulo") == 0 ) {
         cout<<"digite el valor de la base del triangulo"<<endl;
         cin>>base;
         cout<<"digite el valor de la altura del triangulo"<<endl;
         cin>>altura;
         cout<<"el area del triangulo es:"<<base*altura/2<<endl;
} else if(figura.compare("circulo") == 0) {
           cout<<"digite el valor del radio del circulo"<<endl;
           cin>>radio;
           cout<<"el area del circulo es:"<<3.1514*radio*radio<<endl;
}
 
system("PAUSE");
return EXIT_SUCCESS;
}
 

[andres31415]

Iniciado por sam90

Te estas desviando un poco de lo importante. El swicth-case es solo un estructura similar al if. nada mas. Busca en internet la sintaxis y listo....

Tu problema aca es que vos creaste variables del tipo float y por lo que dice el programa estas tratandolas de usar como si fuesen string.

Código C++:

#include <cstdlib>
#include <iostream>
#include <stdio.h>
#include <conio.h>
#include <string>
 
using namespace std;
 
int main()
{
string figura;
float lado,base,altura ,radio;
cout<<"digite el nombre de la figura "<<endl;
cin>>figura;
 
if(figura.compare("cuadrado") == 0)  {
          cout<<"Digite el valor del lado del cuadrado"<<endl;
          cin>>lado;
          cout<<"el area del cuadrado es:"<<lado*lado<<endl;
}
else if(figura.compare("triangulo") == 0 ) {
         cout<<"digite el valor de la base del triangulo"<<endl;
         cin>>base;
         cout<<"digite el valor de la altura del triangulo"<<endl;
         cin>>altura;
         cout<<"el area del triangulo es:"<<base*altura/2<<endl;
} else if(figura.compare("circulo") == 0) {
           cout<<"digite el valor del radio del circulo"<<endl;
           cin>>radio;
           cout<<"el area del circulo es:"<<3.1514*radio*radio<<endl;
}
 
system("PAUSE");
return EXIT_SUCCESS;
 
Gracias Heimish2000 por la ayuda que me has dado. 
}
 

[dragg04]

una solucion alternativa a la de sam90 podria ser la siguiente

Código C++:

#include <cstdlib>
#include <iostream>
#include <stdio.h>
#include <conio.h>
#include <string>
 
using namespace std;
 
enum enumTipoGeometria{tCuadrado=1, tTriangulo, tCirculo};
 
int main()
{
 
int figura;
float lado,base,altura ,radio;
cout<<"digite el nombre de la figura \n1) Cuadrado \n2)Triangulo \n3)Circulo"<<endl;
cin>>figura;
 
if(figura == ::tCuadrado){
    cout<<"Digite el valor del lado del cuadrado"<<endl;
    cin>>lado;
    cout<<"el area del cuadrado es:"<<lado*lado<<endl;
}
else if(figura == ::tTriangulo ) {
    cout<<"digite el valor de la base del triangulo"<<endl;
    cin>>base;
    cout<<"digite el valor de la altura del triangulo"<<endl;
    cin>>altura;
    cout<<"el area del triangulo es:"<<base*altura/2<<endl;
} else if(figura == ::tCirculo) {
    cout<<"digite el valor del radio del circulo"<<endl;
    cin>>radio;
    cout<<"el area del circulo es:"<<3.1514*radio*radio<<endl;
}
 
system("PAUSE");
return EXIT_SUCCESS;
}
 

[baker1562]

Prueba este, es una tarea mia, cambiale algunos valores y elimina variables que no ocupes

Código C++:

#include <iostream>
#include <iomanip>
#include <cmath>
using namespace std;
 
int main() 
{
 
 string resp;    
 int area_trapecio;
 int base_mayor;
 int base_menor;
 int altura;
 int area_rectangulo;
 int largo;
 int ancho;
 double area_corona;
 double p;
 int radio_mayor;
 int radio_menor;
 int area_rombo;
 int diagonal_mayor;
 int diagonal_menor;
 int opcion;
 
 
 while("resp !=s")  
 {       
      cout<<setw(30)<<"Calculo Geometrico"<<endl;
      cout<<"Para saber el area del trapecio, digite #1:"<<endl;
      cout<<"Para saber el area del rectangulo, digite #2:"<<endl;
      cout<<"Para saber el area del corona circular, digite #3:"<<endl;
      cout<<"Para saber el area del rombo, digite #4:"<<endl;
      cin >> opcion;
      {
         
      if (opcion==1) 
      {           
                    cout <<setw(25)<<"****Area del trapecio!****\n" << endl; 
                    cout << "digite la base mayor: " << endl;
                    cin >> base_mayor;
                    cout << "digite la base menor:" << endl;
                    cin >> base_menor;
                    cout << "digite la altura:" << endl;
                    cin >> altura;
                    area_trapecio = ((base_mayor+base_menor)*altura)/2;
                    cout <<"El area es igual a:"<<" "<<area_trapecio <<  endl;
                    cout << "La formula del area del trapecio es \n(B + b)*h/2\n\n     ******\n    *      *\n   *        *\n  *          *\n"
                    " **************\n" << endl;}
                    
        else                    
        {
                        
          if (opcion==2)
          {           
                       cout <<setw(25)<< "*****Area del rectangulo!*****\n" << endl; 
                       cout << "Digite el largo:" << endl;
                       cin >> largo;
                       cout << "Digite la ancho:" << endl;
                       cin >>ancho;
                       area_rectangulo=largo*ancho;
                       cout <<"El area es igual a:"<<" "<< area_rectangulo<< endl;
                       cout << "La formula del area del rectangulo es: a=l*h\n    ***********\n    *         *\n    *         *\n"
                       "    *         *\n    ***********" << endl;}
         else
          {
                           
           if (opcion==3)
           {           
                       cout <<setw(25)<<"*****Corona Circular!*****\n" << endl; 
                       cout << "Digite el radio mayor:" << endl;
                       cin >> radio_mayor;
                       cout << "Digite el radio menor:" << endl;
                       cin >> radio_menor;
                       p=3.1416;
                       area_corona=p*((radio_mayor*radio_mayor)-(radio_menor*radio_menor));
                       cout <<"El area es:"<<" "<< area_corona << "\n" << endl;
                       cout << "La formula del area de la corona circular es: p(R^2-r^2)\n\n"
                       "\n\n      *    \n   *     *\n  *       *\n  *       *\n   *     *\n      *" << endl;}
                                          
           else
           {
                                
              if (opcion==4)
              {       
                      cout <<setw(25)<<"*****Rombo!*****\n" << endl;
                      cout << "Digite la diagonal mayor:" << endl;
                      cin >> diagonal_mayor;
                      cout << "Digite la diagonal menor" << endl;
                      cin >>diagonal_menor;
                      area_rombo=diagonal_mayor*diagonal_menor/2;
                      cout <<"El area es:"<<" "<<area_rombo<< "\n" << endl;
                      cout << "La formula del area del rombo es: a=D*d/2\n\     *   \n   *   * \n  *     *\n *       *\n"
                      "  *     *\n   *   *\n     *" << endl;}
                      else
                      {
                          if (opcion>4)
                          {
                          cout << "ERROR:Numero no permitido, Valido del 1 al 4" << endl;}
                          }
                          
                       }
                    }
                } 
                cout <<"Desea el usuario realizar otro calculo: S/N?"<< endl;
                cin >> resp;
               
                if ( resp!="s" )
                                             
                   
               return 0; } 
                   
        } 
}