[SOLUCIONADO] Duda con Switch - Case

[DarkScripter]

Buenas, bueno primero antes que nada permitanme aclararles que en esto del lenguaje C++ como tal estoy viendo lo básico en la universidad que estudio, por lo tanto suelo perderme un poco de acuerdo a los términos técnicos planteados en el problema.

Bien, el problema es el siguiente, una de las actividades fue que debemos hacer un programa en el cual se debe introducir el monto total de una compra, luego de ello escribir el color de una pelota que retiras al realizar la compra y de acuerdo al color que aparezca, obtienes un descuento.

El código que planteó el profesor fue el siguiente:

Código C++:

/*
===================================================================================
En una tienda de descuento se efectúa una promoción en la cual se hace un descuento
sobre el valor de la compra total según el color de la bolita que el cliente
saque al pagar en caja. Si la bolita es de color blanco no se le hará 
descuento alguno, si es verde se le hará un 10% de descuento, 
si es amarilla un 25%, si es azul un 50% y si es roja un 100%. 
Determinar la cantidad final que el cliente deberá pagar por su compra. 
se sabe que solo hay bolitas de los colores mencionados.
===================================================================================
*/
#include <stdio.h>  //  Archivo de la biblioteca estándar de Entrada/Salida.
#include <math.h> // archivo estandar de la biblioteca de matematicas
#include <stdlib.h> // Aritméticas, Números aleatorios; y • Conversión de cadenas.
 
int main(int argc, char *argv[])
{
  char mensaje;
  float Monto, Descuento, TotalPagar;
  const Azul= Azul;
  
  printf("Dime el Monto de la Compra= ");
  scanf("%f", &Monto);  
  printf("=============================================================\n");
  printf("Dime El Color que Recibiste\n");
  scanf("%s",&mensaje);
  printf("=============================================================\n");
  
  switch (mensaje)
  {  
    case 'Blanco':
    case 'blanco':
    case 'BLANCO':
            Descuento= 0;
            printf("El Descuento por la Compra es: %f\n",Descuento);
            break;           
    case 'Verde':  
    case 'verde':
    case 'VERDE':          
            Descuento= ( Monto * 0.10);
            printf("El Descuento por la Compra es: %f\n",Descuento);
            break;             
    case 'Amarillo':
    case 'amarillo':
    case 'AMARILLO':
            Descuento= (Monto * 0.25);
            printf("El Descuento por la Compra es: %f\n",Descuento);
            break;           
    case 'Azul':
    case 'azul':
    case '4':
            Descuento= (Monto * 0.50);
            printf("El Descuento por la Compra es: %f\n",Descuento);
            break;            
    case 'Rojo':
    case 'rojo':
    case 'ROJO':
            (Descuento= Monto);
            printf("El Descuento por la Compra es: %f\n",Descuento);
            break;            
  }
  
  TotalPagar= Monto - Descuento;
  
  printf("El Monto A pagar Total Por la Compra Es= %f\n",TotalPagar);
   
  system("PAUSE");  
  return 0;
}
 

Bueno, ya analizando el código, dio errores de compilación.

Por lo tanto estuve investigando y acomodando el código de acuerdo a que pudiera ejecutar bien el programa, pero me percate de que al introducir el color, este no hace nada, sino que finaliza el programa.

Observen:



Acomodando el case y colocando un solo carácter, en vez de "blanco" colocas la inicial que es "b", este se ejecuta con normalidad.

Captura:



Disculpen tanto las vueltas, pero es para ir explicando lo que estuve haciendo mas o menos para llegar a la conclusión de que el case solo permite un carácter, si colocas 2, no se ejecuta.

Ahora la duda es. ¿Únicamente es posible en los case colocar variables de un solo carácter o este permite ejecutar mas caracteres? De ser así, que estructura o sentencia debo utilizar para que los case se ejecuten con mas de un carácter?.

Aquí está el código que utilicé para que lograra funcionar el programa:

Código C++:

#include <stdio.h> //  Archivo de la biblioteca estándar de Entrada/Salida.
#include <math.h> // archivo estandar de la biblioteca de matematicas
#include <stdlib.h> // Aritméticas, Números aleatorios; y • Conversión de cadenas. 
int main(int argc, char *argv[])
{
  char mensaje;
  float Monto, Descuento, TotalPagar;
  const int Azul= Azul;
  printf("Dime el Monto de la Compra:");
  scanf("%f",&Monto);  
  printf("=============================================================\n");
  printf("Dime El Color que Recibiste:");
  scanf("%s",&mensaje);
  printf("=============================================================\n");
  switch (mensaje) 
  {  
    case 'b':
            TotalPagar = Monto;    
            printf("Lo sentimos, no obtuvo ningun descuento\n");
            printf("El monto total a pagar es:%f\n",TotalPagar);
            break;           
  }
  system("pause");  
  return 0;
}
 

PD: Utilicé la secuencia enum y tampoco funcionó, no tengo conocimiento si esa función se usa en C++ o lo declaré mal.

Bueno, eso es todo y en verdad lamento mucho la molestia de acuerdo a tantas vueltas, espero que me puedan ayudar .

Saludos.

[eferion]

Buenos días.

Si el código que os ha puesto no es más que un ejemplo que tenéis que adaptar para conseguir que funcione puedo decirte que es un poco inutil:

Código C++:

case 'Blanco':
 

Para empezar, las comillas simples sirven, única y exclusivamente, con caracteres sueltos como 'c', 'a', 'X', pero nunca para cadenas de texto. Éstas tienen que ir encerradas con comillas dobles: "Blanco".

El segundo error que tiene esa línea (así como todos los case) es que un case únicamente puede hacer comparaciones con tipos nativos no punteros (bool, int, float, double, char). Y, le pese a quien le pese, la clase "string" no es un tipo nativo de C++, por lo que no puedes usarlo para estos menesteres.

Para salir airoso de una situación así tienes dos opciones:

* Sustituir el switch por una secuencia de if ... else if ... else if ...

* Asignar un índice a cada color: Blanco -> 1, Negro -> 2, ... y usas ese indice en los case. La conversión la puedes hacer de diferentes maneras, pero para no solapar esta solución con la anterior puedes usar un vector o un mapa:

Código C++:

std::map< std::string, int > colores;
colores[ "blanco" ] = 1;
colores[ "negro" ] = 2;
 
// ...
 
// A "colores" le estamos pidiendo el valor asignado a la clave. La clave es el color y el valor es el índice asignado a dicho color
int colorInt = colores[ mensaje ];
switch( colorInt )
{
  case 1: // Blanco
    // ...
  case 2: // Negro
    // ...
  default: // Color desconocido
    // ...
}
 

Un saludo.

[DarkScripter]

Buenas, muchas gracias por responder bro.

Con respecto a lo del código, el profesor siempre lo coloca y nosotros lo copiamos y luego vemos si ejecuta y le mostramos la captura.

En este caso, no sé que le dio que puso esas declaraciones, como si estuviera inventando.

Por eso empezé a investigar y sin respuesta alguna, llegue a este foro a preguntar.

Entonces en resumen, lo que debo buscar en si es una sentencia o función para asignar un indice a cada color?.

¿Tiene nombre técnico o lo puedo buscar asi tal cual?.

Saludos.

[vangodp]

Eso seria el case 'Blanco'....

Código C++:

#include <stdio.h>  //  Archivo de la biblioteca estándar de Entrada/Salida.
#include <math.h> // archivo estandar de la biblioteca de matematicas
#include <stdlib.h> // Aritméticas, Números aleatorios; y • Conversión de cadenas.
 
int main ( int argc, char *argv[] ) {
    char mensaje[100];
    float Monto, Descuento, TotalPagar;
    
    
    printf ( "Dime el Monto de la Compra= " );
    scanf ( "%f", &Monto );
    printf ( "=============================================================\n" );
    printf ( "Dime El Color que Recibiste\n" );
    scanf ( "%s", mensaje );
    printf ( "=============================================================\n" );
    
    //pelota blanca
    if ( mensaje == "Blanco" || mensaje == "blanco" || mensaje == "BLANCO" ) {
        Descuento = 0;
        printf ( "El Descuento por la Compra es: %f\n", Descuento );
    }
    
    //poner aqui pelota roja, azul... Lo que sea XD
    
    
    TotalPagar = Monto - Descuento;
    
    printf ( "El Monto A pagar Total Por la Compra Es= %f\n", TotalPagar );
    
    getchar();
    return 0;
}
 

Es adaptarlo para cada color ;)

[DarkScripter]

Jajajajajajajaja, muchas gracias bro, pero en si lo que busco es adaptarlo con el switch y case, es decir buscar que haga las instrucciones cuando se escriba blanco,rojo y todo lo demas, que de a eso venía mi duda si los case se pueden colocar con más de un caracter.

porque puedo adaptarlo a un menu sencillo y meter las opciones, pero quiero aprender sobre esas situaciones.

En verdad aprecio tu ayuda, y si me puedes guiar de acuerdo como hacer con esos case para que se escriban completo en verdad apreciaría la ayuda.

Nuevamente gracias y disculpa, saludos.

[vangodp]

Código C++:

#include <stdio.h>  
#include <math.h>   
#include <stdlib.h> 
#include <map>      //incluya el map y esto es C++
 
int main ( int argc, char *argv[] ) {
    char mensaje[100];
    float Monto, Descuento, TotalPagar;
    std::map< std::string, int > colores;
    //Variantes para blanco
    colores[ "blanco" ] = 1;
    colores[ "Blanco" ] = 1;
    colores[ "BLANCO" ] = 1;
    
    //Declarar otros colores...
    colores[ "Verde" ] = 2;
    
    
    printf ( "Dime el Monto de la Compra= " );
    scanf ( "%f", &Monto );
    printf ( "=============================================================\n" );
    printf ( "Dime El Color que Recibiste\n" );
    scanf ( "%s", mensaje );
    printf ( "=============================================================\n" );
    
    
    int colorInt = colores[ mensaje ];
    switch ( colorInt ) {
        case 1: // Blanco, Blanco o BLANCO
            Descuento= 0;
            printf("El Descuento por la Compra es: %f\n",Descuento);            
            break;
        
        //caso2....    
            
        default:
            printf("\nEl color no color no existe\n");
            break;
    }
    
    
    TotalPagar = Monto - Descuento;
    
    printf ( "El Monto A pagar Total Por la Compra Es= %f\n", TotalPagar );
    
    getchar();
    return 0;
}
 

Venga... tu a lo más difícil XD.

Por cierto... Si en tu universidad echan a tu profes, diles que estoy sin trabajo. Calculo que no le queda mucho si programa cosas como esta XDDD

[eferion]

Iniciado por vangodp

Venga... tu a lo más difícil XD.

También puedes pasar la cadena a minúsculas y te evitas el tener varios elementos con el mismo índice:

Código C++:

// ...
 
  // Convertimos a minúsculas, opción 1
  std::locale loc;
  for( auto& c : mensaje ) c = std::tolower( c, loc );
 
  // opción 2:
  for( auto& c : mensaje ) c = tolower( c );
 
  int colorInt = colores[ mensaje ];
  // ...
 

[Malenko]

No entiendo como os complicáis tanto.

- Lo principal es hacer lo que ha comentado @eferion y trabajar con minúsculas o mayúsculas porque sino si alguien pone por error "BLanco" el programa fallará.

- Segundo, que utilidad tiene tener un array para que te devuelva un int y uses ese int en un switch para calcular el descuento? Es mucho más sencillo y óptimo tener un array que consultes por el color, como estás haciendo ahora, y en lugar de devolver un "int" para simular un enumerado, que te devuelva directamente el descuento. Si el color no se encuentra en el array pues simplemente tendrás un descuento de CERO.

Con estos dos cambios, el tener que agregar un nuevo color y su correspondiente descuento simplemente sería añadir un valor más en la inicialización del array. El resto del programa no se tendría que tocar con lo que sería más mantenible. Compara eso con todo lo que tendrías que añadir si lo haces como hasta ahora con el switch...

[eferion]

Iniciado por Malenko

- Segundo, que utilidad tiene tener un array para que te devuelva un int y uses ese int en un switch para calcular el descuento? Es mucho más sencillo y óptimo tener un array que consultes por el color, como estás haciendo ahora, y en lugar de devolver un "int" para simular un enumerado, que te devuelva directamente el descuento. Si el color no se encuentra en el array pues simplemente tendrás un descuento de CERO.

A nivel práctico seguramente sí, pero cuando estás aprendiendo muchas veces te toca hacer ejercicios "inútiles" o de formas poco ortodoxas símplemente porque necesitas practicar las diferentes características del lenguaje.

Si una imposición es usar un "switch"... pues es un requisito que no puedes evitar y lo mejor que puedes hacer es adaptarte... si me dices que tienes libertad absoluta entonces la cosa cambia, pero mientras tanto...

Un saludo.

[YorchX]

Totalmente de acuerdo... Cambia a minúsculas ... Tolower :)

[vangodp]

Es que no se como un maestro que hace case 'blanco': tiene trabajo. >_<

[DarkScripter]

Buenas, gracias a todos por sus respuestas, y con respecto a lo que dice malenko, entiendo pero no puedo modificar el código a mi manera porque para mi es mas tedioso hacerlo con el if y else, pero me es mas sencillo hacerlo asi o con el switch y case con menú y la "lección" trata sobre el switch y case.

Pero como quiero aprender como dice vangodp "a lo dificil" xD, por eso me detuve aquí a preguntar y pedir ayuda.

Con respecto a los códigos que me han dado, disculpen mi ignorancia o falta de conocimiento, sería mas o menos así?:

Código C++:

#include <stdio.h> //  Archivo de la biblioteca estándar de Entrada/Salida.
#include <math.h> // archivo estandar de la biblioteca de matematicas
#include <stdlib.h> // Aritméticas, Números aleatorios; y • Conversión de cadenas. 
#include <map>
int main(int argc, char *argv[])
{
  char mensaje[100];
  float Monto, Descuento, TotalPagar;
  std::map< std::string, int > colores;
   colores["blanco"]=1;
   colores["verde"]=2;
   colores["amarillo"]=3;
   colores["azul"]=4;
   colores["rojo"]=5;
  printf("Dime el Monto de la Compra:");
  scanf("%f",&Monto);  
  printf("=============================================================\n");
  printf("Dime El Color que Recibiste:");
  scanf("%s",&mensaje);
  printf("=============================================================\n");
  for( auto& c : mensaje ) c = tolower( c );
  int colorInt = colores[ mensaje ];
  switch ( colorInt ) 
  {  
    case 1:
            TotalPagar = Monto;    
            printf("Lo sentimos, no obtuvo ningun descuento\n");
            printf("El monto total a pagar es:%f\n",TotalPagar);
            break;           
  }
  system("pause");  
  return 0;
}
 

La verdad en parte a veces me cuesta entender los ejemplos, ya que los debo ver con un programa como tal :(.

Mil disculpas nuevamente, saludos.

PD: Vangodp, si eres de Venezuela postúlate xD

[vangodp]

El unico error es que no debes poner el & en la linea: scanf ( "%s", mensaje ); delante de mensaje. Eso es por que el proprio nombre se considera un puntero al primer elemento de la cadena.

A efectos mensaje es lo mismo que &mensaje[0](un puntero a la primera posición de la cadena). Como ves ya lleva el & y por eso no debes poner.

Cambia scanf ( "%s", &mensaje ); por scanf ( "%s", mensaje );.

Suerte ;)

[DarkScripter]

Gracias bro, bueno lo compilé y me salieron varios pero varios errores, uso el compilador dev c++ ya que ese es el que usa el profe :(.

Los errores son los siguientes, no vienen tanto al código sino a otros archivos:

Este me aparece con respecto a los colores:

Código C++:

no match for 'operator[]' in 'colores["blanco"]'
 

asi con verde, amarillo, etc.

Despues de ese error aparece este en cada uno de los colores:

Código C++:

candidates are: _Tp& std::map<_Key, _Tp, _Compare, _Alloc>::operator[](const _Key&) [with _Key = std::string, _Tp = int, _Compare = std::less<std::string>, _Alloc = std::allocator<std::pair<const std::string, int> >]
 

Línea 32 del código que es donde esta el for, me sale esto:

Código C++:

[expected primary-expression before "auto" ] y expected ';' before "auto"
 

Línea 33:

Código C++:

no match for 'operator[]' in 'colores[mensaje]'.
 

Línea 34:

Código C++:

expected primary-expression before "switch"
 

Código C++:

expected `)' before "switch"
 

Código C++:

at global scope 34 e In instantiation of `std::pair<const std::string, int>':
 

Ya los errores a partir de alli, me lleva a otros archivos, este error aparece en el archivo stl_tree.h:

Código C++:

instantiated from `std::_Rb_tree_node<std::pair<const std::string, int> >' 
Línea 135: _Val _M_value_field;
Línea 565: { _M_erase(_M_begin()); }
Línea 1064: _M_erase(_S_right(__x));
 

Este aparece en el archivo stl_pair.h:

Código C++:

`std::pair<_T1, _T2>::first' has incomplete type 
Línea 73: _T1 first;
 

Y este ultimo me aparece en el archivo stringfwd.h

Código C++:

declaration of `const struct std::string' 
Línea 56: class basic_string;
 

Bueno, estaré investigando sobre todo estos errores para ir indagando mas sobre este tipo de situaciones.

Muchas gracias a todos por su ayuda, cualquier cosa estaré al tanto aca a ver si conocen solución a esos problemas :(

[eferion]

Iniciado por DarkScripter

Este me aparece con respecto a los colores:

Código C++:

no match for 'operator[]' in 'colores["blanco"]'
 

asi con verde, amarillo, etc.

Pega tu código completo... o te falta algo o estás usando un compilador con errores.

Iniciado por DarkScripter

Línea 32 del código que es donde esta el for, me sale esto:

Código C++:

[expected primary-expression before "auto" ] y expected ';' before "auto"
 

"auto" es una palabra que está disponible si compilas usando el estándar C++11 o superior (es bastante reciente). Si usas un compilador antiguo no tendrás acceso a este estándar.

Una variante usando sintaxis más antigua sería:

for( size_t i=0; i<mensaje.length(); ++i ) mensaje[i] = tolower(mensaje[i]);

El resto de errores están relacionados con el uso de map... no les des más importancia.

Un saludo

[vangodp]

dejame que te haga una pregunta: ¿Es tu dev C++ el 4.9.2, o algo así? La 4 vamos, por que si lo es será por eso.

Deberías usar dev C++ pero la version orwell. Busque "dev C++ Orwell" en google.

Lo que pasa es que ahora me tiene dudas... ¿Si tu maestro usa el 4 tu puedes usar el 5? Lo digo por que seria lo mismo que usar otro. XD

El devC++ 4 esta obsoleto ya que esta descontinuado, el compilador es antiguo, y el IDE no recibe más actualizaciones por parte de su creador. Es este señor llamado Orwell el que está actualizando de forma regular, de ahí que le llamen Dev c++ Orwell.

Piensa lo y dependiendo de lo que veas tendrás que modificar el programa por enésima vez. O bien podrías sugerir actualizar el compilador de la uni. jajaj.

Por cierto. yo uso codeblocks naugtly build, con mingw W-64 4.9.2 que es casi lo ultimo.
La cosa consiste en tener mingw o gcc medianamente actualizado.
Si no me equivoco los de linux ya van por gcc 5. XD

Lo que quiero decir es que no se si le va hacer gracia a tu maestro si llevas un programa diferente al que sabes que es lo ultimo y que funciona, pero que a tu profes no le guste, por que no le va compilar por ser su compilador obsoleto.

Tú no tienes la culpa vamos... Aún hay centros de formación que usan borland con getch() jajaja

Suerte!

[alexismdash]

hola amigo

lo que yo haria para este programa seria a cada color darle un numero

ejemplo

blanco =1

entonces en el case pones

case 1;

el proceso que haga cada color

espero y te sirva

[DarkScripter]

vangodp
Respuesta: Duda con Switch - Case:
dejame que te haga una pregunta: ¿Es tu dev C++ el 4.9.2, o algo así? La 4 vamos, por que si lo es será por eso.

Si bro el 4.9.9.2, instalé el 5.10 que me dijiste que es de la versión más reciente de Orwell y me sale esto ahora:

Ya, verificando los problemas, solo me encuentro con estos 2:

Línea 33:

Código C++:

for( auto& c : mensaje ) c = tolower( c );
 

[Error] ISO C++ forbids declaration of 'c' with no type [-fpermissive]
Y
[Error] range-based 'for' loops are not allowed in C++98 mode

Tenía otros errores de acuerdo al string, pero ya lo solucioné.

Solo había que agregar la librería string en la cabecera al inicio del programa.

Estaré al tanto y seguiré buscando solución al problema para que por fín pueda ejecutarme :(.

Saludos.

[eferion]

Iniciado por DarkScripter

Estaré al tanto y seguiré buscando solución al problema para que por fín pueda ejecutarme :(.

Si miras mis respuestas ya te di solución para ese error:

Iniciado por eferion

Una variante usando sintaxis más antigua sería:

Código C++:

for( size_t i=0; i<mensaje.length(); ++i ) mensaje[i] = tolower(mensaje[i]);
 

Un saludo

[DarkScripter]

Iniciado por eferion

Si miras mis respuestas ya te di solución para ese error:

Código C++:

for( size_t i=0; i<mensaje.length(); ++i ) mensaje[i] = tolower(mensaje[i]);
 

Un saludo

De igual manera da error, de otro tipo, dice esto:

[Error] request for member 'length' in 'mensaje', which is of non-class type 'char [100]'

Buscaré ese error tambn a ver si hay solución.

Saludos y mcuhas gracias por la ayuda eferion.

[eferion]

Iniciado por DarkScripter

De igual manera da error, de otro tipo, dice esto:

[Error] request for member 'length' in 'mensaje', which is of non-class type 'char [100]'

Buscaré ese error tambn a ver si hay solución.

Saludos y mcuhas gracias por la ayuda eferion.

Cambia mensaje... de char[100] a std::string

[vangodp]

Te recomiendo lo que te ha dicho eferion, usar strings. Pero una posible solución es:

/*Siendo mensaje una cadena de caracter estilo C. Ejemplo char cString [100];*/

Código C++:

int tam = sizeof(mensaje); 
for( size_t i=0; i<tam; ++i ) mensaje[i] = tolower(mensaje[i]);
 

...pero si usas string no tienes por que preocuparte por tamaños. ;)

También deberías ir al menú que esta al lado(CodeWarnings) y activar los avisos (-Wall). La ruta es:
menu: Herramientas/Configuración/Code Warnings: Active la opción Show most warnings(-Wall)
Esta opción te va mostrar mas que avisos consejos sobre algunas cosas, muy útil ;).

Por ejemplo si intentas hacer eso en C++:char* p = "hola mundo"; no te esta diciendo que es un [error] si no aviso: [Warning] deprecated conversion from string constant to 'char*' [-Wwrite-strings]. Puedes hacerlo si quieres, simplemente que se desaconseja por completo, los motivos ya los investigas tu. XD

Asi que recuerda:
activar C++11:
menu: Herramientas/Configuración/Generación de Código: Elija en la opción Language standart(-std) la opción ISO C++11.

Avisos o warnings (-Wall)
menu: Herramientas/Configuración/Code Warnings: Active la opción Show most warnings(-Wall)

Con eso ya deberías tener todo ok.

[DarkScripter]

Buenas, ya ejecutó pero cuando pongo el color, se queda hasta allí.

Observen:



Jooo, ya me había emocionado y todo :(.

Saludos :(.

[eferion]

Iniciado por DarkScripter

Buenas, ya ejecutó pero cuando pongo el color, se queda hasta allí.

Observen:



Jooo, ya me había emocionado y todo :(.

Saludos :(.

Pon el codigo... Pero tiene pinta a que te has comido alguna condición en el switch

[vangodp]

Código C++:

#include <stdio.h> //  Archivo de la biblioteca estándar de Entrada/Salida.
#include <math.h> // archivo estandar de la biblioteca de matematicas
#include <stdlib.h> // Aritméticas, Números aleatorios; y • Conversión de cadenas.
#include <map>
int main(int argc, char *argv[]) {
    char mensaje[100];
    float Monto, Descuento, TotalPagar;
    std::map< std::string, int > colores;
    colores["blanco"]=1;
    colores["verde"]=2;
    colores["amarillo"]=3;
    colores["azul"]=4;
    colores["rojo"]=5;
    printf("Dime el Monto de la Compra:");
    scanf("%f",&Monto);
    printf("=============================================================\n");
    printf("Dime El Color que Recibiste:");
    scanf("%s", mensaje);
    printf("=============================================================\n");
    for( auto& c : mensaje ) c = tolower( c );
    int colorInt = colores[ mensaje ];
    switch ( colorInt ) {
        case 1:
            TotalPagar = Monto;
            printf("Lo sentimos, no obtuvo ningun descuento\n");
            printf("El monto total a pagar es:%f\n",TotalPagar);
            break;
        case 2:
            //verde
            break;
        case 3:
            //amarillo
            break;
        case 4:
            //azul
            break;
        case 5:
            //rojo
            break;
        default:
            std::cout << "No existe ese caso" << std::endl
            break;                                              
    }
     
    system("pause");
    return 0;
}
 

Este es el código que me compila. Si tienes otro diferente ponlo aquí. No te olvides de sacar la & delante de mensaje a la hora de leer. Lo compile en Dev C++ igual que tu.

[DarkScripter]

Este es el código que tengo bro:

Código C++:

#include <stdio.h> //  Archivo de la biblioteca estándar de Entrada/Salida.
#include <math.h> // archivo estandar de la biblioteca de matematicas
#include <stdlib.h> // Aritméticas, Números aleatorios; y • Conversión de cadenas. 
#include <map>
#include <string>
int main(int argc, char *argv[])
{
  std::string mensaje;
  float Monto, Descuento, TotalPagar;
  std::map< std::string, int > colores;
   colores["blanco"]=1;
   colores["verde"]=2;
   colores["amarillo"]=3;
   colores["azul"]=4;
   colores["rojo"]=5;
    printf("Introduce el monto de la compra:");
    scanf("%f",&Monto);  
    printf("=============================================================\n\n");
    printf("Introduce el color que has recibido:");
    scanf("%s",&mensaje);
    printf("=============================================================\n\n");
    for( size_t i=0; i<mensaje.length(); ++i ) mensaje[i] = tolower(mensaje[i]);
    int colorInt = colores[ mensaje ];
    switch ( colorInt ) 
    {  
        case 1:
            Descuento = 0;    
            printf("Lo sentimos, no obtuvo ningun descuento\n\n");
            break;      
        case 2:    
            Descuento= (Monto*0.10);
            printf("El Descuento por la Compra es: %f\n\n",Descuento);
            break;
        case 3:
            Descuento= (Monto * 0.25);
            printf("El Descuento por la Compra es: %f\n\n",Descuento);
            break;
        case 4:
            Descuento= (Monto * 0.50);
            printf("El Descuento por la Compra es: %f\n\n",Descuento);
            break;
        case 5:
            (Descuento= Monto);
            printf("El Descuento por la Compra es: %f\n\n",Descuento);
            break;                          
    }
  TotalPagar= Monto - Descuento;
  printf("El monto total a pagar es= %f\n\n",TotalPagar);
  system("pause");  
  return 0;
}
 

El ejecuta normalmente si le quito el for y hace sus sentencias normal, pero ahora no sé que pasó que me dio ese error de no hacer los case.

Cuando el destino te dice que no eres bueno en algo :(.

Saludos

[eferion]

Estás programando en C++. Para gestionar la entrada (teclado) / salida (consola) en C++ es casi más recomendable usar las clases cin y cout, las cuales se encuentran en la librería iostream. La ventaja de usar esta librería es que es compatible con la clase "string" de C++:

Código C++:

std::cout << "Introduce el monto de la compra:" ;
std::cin >> Monto;
std::cout << "=============================================================\n\n";
std::cout << "Introduce el color que has recibido:";
std::cin >> mensaje;
std::cout << "=============================================================\n\n";
 

Con esto ya debería funcionar tu programa. Sin embargo me gustaría destacar un pequeño detalle... si el usuario introduce un color no válido la aplicación no va a mostrar ningún mensaje... en mi opinión, lo que puedes hacer, es añadir la opción "default" al final del switch y mostrar ahí un mensaje de error:

Código C++:

switch( )
{
  // ...
 
  default:
  {
    // Mensaje de error
  }
}
 

Un saludo

[DarkScripter]

Iniciado por eferion

Estás programando en C++. Para gestionar la entrada (teclado) / salida (consola) en C++ es casi más recomendable usar las clases cin y cout, las cuales se encuentran en la librería iostream. La ventaja de usar esta librería es que es compatible con la clase "string" de C++:

Código C++:

std::cout << "Introduce el monto de la compra:" ;
std::cin >> Monto;
std::cout << "=============================================================\n\n";
std::cout << "Introduce el color que has recibido:";
std::cin >> mensaje;
std::cout << "=============================================================\n\n";
 

Con esto ya debería funcionar tu programa. Sin embargo me gustaría destacar un pequeño detalle... si el usuario introduce un color no válido la aplicación no va a mostrar ningún mensaje... en mi opinión, lo que puedes hacer, es añadir la opción "default" al final del switch y mostrar ahí un mensaje de error:

Código C++:

switch( )
{
  // ...
 
  default:
  {
    // Mensaje de error
  }
}
 

Un saludo

Así es bro, funciona al toque, muchas gracias en verdad por toda la ayuda y lamento mucho que haya sido tan novato y haber dado muchos problemas de acuerdo a esto. También le doy las gracias a Vangodp porque ambos me enseñaron mucho y me tuvieron paciencia xD.

PD: Mañana veo esa clase, tiempo de abofetear al profe por inventor :(.

Saludos y nuevamente gracias :), cierro tema.

[vangodp]

Programar es eso compañero. Es leer y releer hasta que quedas con la copla ;)
No desista nunca.

Suerte.