Tengo este algoritmo: Duda con condicional

[juandobleme]

Quisiera saber como puedo cambiar en los condicionales el numero que estoy asignando por una palabra, por ejemplo que escriba ROJA y me imprima un tipo de descuento.. Ya estuve realizando varias pruebas y solo me funciona con números.



#include <iostream>

using namespace std;
int main()
{
float Bola,PrecioDes,CantidadAP;

PrecioDes=0;
CantidadAP=0;

cout<<("\nIntroduzca el precio total: ");

cin>> CantidadAP;

cout<<("\nCual es el codigo de la bolita: ");

cin>> Bola;

if(Bola==1) {
cout<<("\nNo posee descuento") << endl;
}

else if (Bola==2) {
cout<<("\nPosee un descuento de 10%") << endl;

PrecioDes=(CantidadAP-((CantidadAP*10)/100));

cout <<"\nTotal a pagar es: " << PrecioDes << endl;
}

else if (Bola==3){
cout<<("\nPosee un descuento de 25%");

PrecioDes=(CantidadAP-((CantidadAP*25)/100));

cout <<"\nTotal a pagar es: " << PrecioDes << endl;
}

else if(Bola==4){
cout<<("\nPosee un descuento de 50%") << endl;

PrecioDes=(CantidadAP-((CantidadAP*50)/100));

cout <<"\nTotal a pagar es: " << PrecioDes << endl;
}

else {

cout<< "\nLa compra es gratis" << endl;

}
system("PAUSE");
return(0);

}

[eferion]

Lo primero antes de nada. Si Bola únicamente almacena enteros debería ser int en vez de float. Si no quieres errores raros en tu código acostúmbrate a usar el tipo correcto para cada variable.

Y otro detalle más, std::cout no necesita paréntesis para sacar el contenido por la pantalla.

Lo que pretendes puedes hacerlo de varias formas, pero básicamente todas pasan por comparar números o cadenas.

Para hacer la versión sencilla, es decir, comparando palabras, tienes que tener en cuenta que no es posible comparar dos char* usando el operador de comparación ==, sino que has una función del tipo strcmp. Una opción más sencilla es usar la clase std::string:

Código C++:

std::string bola;
cout<<("\nCual es el color de la bolita: ");
 
cin>> bola;
 
if(Bola==" negro") {
  cout<<("\nNo posee descuento") << endl;
}
 
else if (Bola=="rojo") {
  cout<<("\nPosee un descuento de 10%") << endl;
}
 

Por cierto, nota que el código es sensible al uso de mayúsculas.

Otra opción pasa por comparar códigos. En este caso recuperamos un string y calculamos el código que tiene asociado. En este caso, por claridad, se puede usar la clase std::map para guardar la relación entre un color y su código:

Código C++:

std::map<std::string,int> mapa;
mapa["negro"]=1;
mapa["rojo"]=2;
 
std::string bola;
cout<<("\nCual es el color de la bolita: ");
 
cin>> bola;
int codigo=mapa[bola];
 
 
if(codigo==1) {
  cout<<("\nNo posee descuento") << endl;
}
 
else if (codigo==2) {
  cout<<("\nPosee un descuento de 10%") << endl;
}
 

Aunque quizás, ya puestos, lo suyo sería que el mapa almacenase directamente el descuento, lo que nos permitiría ahorrarnos código repetido:

Código C++:

std::map<std::string,int> mapa;
mapa["negro"]=0;
mapa["rojo"]=10;
// ...
 
cout<<("\nCual es el color de la bolita: ");
 
std::string bola;
cin>> bola;
int descuento=mapa[bola];
 
if(descuento==0)
  cout< ("\nNo posee descuento" << endl;
else if (descuento==100)
  cout<< "\nLa compra es gratis" << endl;
else
{
  cout<<"\nPosee un descuento de " << descuento << "%" << endl;
 
  float PrecioDes=(CantidadAP-((CantidadAP*descuento)/100));
 
  cout <<"\nTotal a pagar es: " << PrecioDes << endl;
}
 

Hay más formas de hacerlo pero las expuestas bastan para darte una idea sobre cómo resolver el problema.

Por cierto, copiar y pegar el código no te va a servir de mucho a la hora del examen. Es recomendable que entiendas lo que estás poniendo aunque la solución propuesta no sea la mejor de todas.

Un saludo.

[juandobleme]

GRacias amigo! Me sirvio mucho tu ayuda.. Tambien encontre otra forma utilizando el string.. Ya que necesitaba imprimir era caracteres

[aguml]

Me surge una duda acerca de lo que haces con std::map. Si tienes:

Código C++:

std::map<std::string,int> mapa;
mapa["negro"]=0;
mapa["rojo"]=10;
 
cout<<("\nCual es el color de la bolita: ");
 
std::string bola;
cin>> bola;
int descuento=mapa[bola];
 

Y bola vale por ejemplo "verde" ¿que comportamiento tendría? Es que nunca traté con ello.

[Instru]

Y bola vale por ejemplo "verde" ¿que comportamiento tendría? Es que nunca traté con ello.

Según el estándar. El hacer mapa["Verde"], literalmente se crea una nueva entrada para esa llave.

Lo que sucede a continuación es que al crearse el nuevo elemento, y el valor ser de tipo int, se inicializa a 0.
Al final, se regresa una referencia a a ese int inicializado en 0.

Sin embargo, si el valor fueran arreglos o estructuras mas complejas, no hay inicialización, por lo que podrías obtener basura en los diferentes campos.

Saludos

[eferion]

Iniciado por Instru

Sin embargo, si el valor fueran arreglos o estructuras mas complejas, no hay inicialización, por lo que podrías obtener basura en los diferentes campos.

Si el campo "valor" del mapa únicamente puede devolver basura si se trata de un objeto y su constructor por defecto no está correctamente implementado.

El motivo es que std::map, al pedirle una clave que no existe, crea una nueva entrada en su lista interna y el campo "valor" se inicializa llamando a su constructor por defecto (si lo hubiese) o asignando el valor 0, el cual es aceptado por todos los tipos nativos (int, float, double, char, punteros...)

Lo que sí que puede presentar basura (o incluso un casque en la aplicación) es, en el caso de que el campo valor sea un puntero, intentar acceder al contenido del mismo. A estas alturas de la vida todos entendemos que acceder al contenido de un puntero nulo no es nunca una buena idea.

Lo comentado anteriormente explica por qué podemos tener problemas al intentar meter en el campo valor de un mapa una clase con su constructor por defecto deshabilitado (por ejemplo declarado en la parte private) o borrado (usando delete en C++11 en adelante). La razón que subyace detrás es que en el caso de los templates el compilador únicamente compila las funciones que se usan realmente. En este caso el operador[], de usarlo, trata de llamar al constructor por defecto y eso tira un error de compilación... pero si se evita su uso el mapa no tendrá inconvenientes en aceptar objetos sin constructor por defecto.

Un saludo.

[aguml]

Entonces para el ejercicio en cuestión no es tan buena idea usar std::map sobre todo si el valor 0es un valor válido ¿no? Y suponiendo que 0 no es un valor válido y se ha creado ¿como corriges eso para quitarlo del mapa?

[eferion]

Hay formas de saber si la clave existe o no en el mapa:

Código C++:

int main()
{
  std::map<int,int> mapa;
  mapa.insert(std::make_pair(1,11));
  mapa.insert(std::make_pair(4,67));
  mapa.insert(std::make_pair(9,18));
  mapa[5] = 22; 
 
  for( auto i=0; i<10; i++ )
  {
    if( mapa.count(i) == 0 )
      std::cout << "La clave " << i << " no existe en el mapa\n";
    else
      std::cout << "mapa[" << i << "] = " << mapa[i] << "\n";
  }
}
 

Para evitar lanzar dos búsquedas por cada clave que existe en el mapa se pueden usar también iteradores:

Código C++:

int main()
{
  std::map<int,int> mapa;
  mapa.insert(std::make_pair(1,11));
  mapa.insert(std::make_pair(4,67));
  mapa.insert(std::make_pair(9,18));
  mapa[5] = 22; 
 
  for( auto i=0; i<10; i++ )
  {
    auto it = mapa.find(i);
    if( it == mapa.end() )
      std::cout << "La clave " << i << " no existe en el mapa\n";
    else
      std::cout << "mapa[" << i /* tambien se puede usar it->first */ << "] = " << it->second << "\n";
  }
}