Foro de elhacker.net

Os cuento la situación para situaros , solo me dejan usar la stdio.h , por lo tanto cualquier cosa o ocurrencia para solucionar el problema que no se resuelva con esa libreria no me valdra

El tema es es siguente yo hago un switch dentro de un do//while para que se vaya repitiendo todo va perfecto , funciona , tiene los cases bien puestos el default por si acaso introduce el numero incorrecto etc etc , pero justo en el numero esta mi problema, me explico si por casualidad al usuario se le pasa por la cabeza poner una 'a' pues se fastidio el programa porque empieza a hacer un bucle infinito no se como solucionarlo :s

es algo asi por si no lo veis

scanf("%d", &cosa);
do{
switch(cosa)
     case 1:
     break
     .

     . 

     .
     default printf("opcion incorrecta!);

}while(cosa != 6)

Simplemente haz un break:

#include<stdio.h>
 
int main()
{
    int cosa;
 
    scanf("%i", &cosa);
 
    switch(cosa)
    {
        case 1:
            //Primera opcion
        case 2:
            //Segunda opcion
        default:
            //Lo que ocurre si no se eligieron las anteriores
            break;
     }
 
     return 0;
}
 

Gracias, pero no no funciona tampoco lo del break :S, no se que hacer la verdad

podes poner el codigo completo?

mmm.. bueno es que es basicamente lo que he puesto xD no se pero bueno si venga pongo un ejemplo tal y como es el mio

#include <stdio.h>

int main()
{
//menu con tres casos
int opcion;


do{

    printf("Introduzca la opcion que desea \n");
    scanf("%d", &opcion);

    switch(opcion)
  
    case 1: printf("hola");
    break;

    case 2: printf("adios");
    break;

    case 3:
    break; //para salir

default:
printf("Opcion incorrecta");
break;

}while(opcion != 3);

}

lo que he hecho ahora es una tonteria pero es en escencia lo que tengo , el problema viene cuando en vez de poner por ejemplo la opcion 2 se me ocurre poner una a, al principio pense en hacerlo con if's pero hay montones de posibilidades y en este caso no seria muy dificil pero en un menu con 15 opciones por ejemplo seria morirse

// modificado tienes razon esque lo copie mal perdon

¿No ves que la variable opcion siempre va tener el mismo valor dentro del bucle? La entrada de datos está fuera. :rolleyes:

Si el problema es que el usuario ingrese una letra, es logico que va a saltar un error porque la variable opcion es un int.
Intenta cambiarlo por un char. Y en vez de poner case 1, case 2, le pones case '1', case '2'.
Ah y te faltan las llaves { } para el switch.

GUAU!! muchas gracias asi si que funciona , bueno saltan unos warnings al compilar que dicen algo de multi-character constant pero si el programa funciona

Lh: No hagas doble post. Utiliza el botón modificar.

Pero ahora la duda que tengo es porque me sale lo del multi-character constant en el momento de compilar con el gcc, esque claro preferiria que compilara limpio sabeis sin warnings de por medio, aunque le doy las gracias a qily y pablomi ;D por solucionar el problema

Eso salta cuando haces algo asi por ejemplo:

switch( op )
{
   case 'aa': break;
}
 

y sabes si hay alguna forma de resolverlo ? es decir yo hago exactamente lo que dices y vale si el programa tira bien pero el problema viene porque la compilación no es digamosle "limpia", si hay alguna forma que alguien sepa para solucionar lo del dichoso multi-character constat le estaria agradecido

O estás poniendo más de un caracter (como ya dijeron) o es un caracter acentuado o por el estilo.

Esto también daría warnings:

case 'Á':
case 'Ã:

Y si pones más de un caracter, nunca vas a llegar a esa parte del case porque char solo almacena un caracter.

Buenas!
Porqué solo stdio?
Es decir,¿ no podés usar funciones del estilo atoi o itoa?.
si no me equivoco definidas en stdlib.h, una librería estándar.
Lo que haría yo es:
.Guardar la entrada en una cadena.
.Comprovar que es un número válido.
.Usar la función atoi para pasar de char* a int.
.Usar enumeraciones para el switch.
En c++ con boost todo es mas simple ^^.
Saludos

.
Espero te sirva alguno de estos ejemplos con Switch y otro con if, no me gusta mucho usar otras librerias si no nesesito muchas de ellas, por ende me creo algunas como el el segundo ejemplo, donde me ahorre String.h.

 
#include<iostream>
 
using namespace std;
 
int main()
{
    int opcion;
    do {
        cout << "Introduzca la opcion que desea"<<endl;
        cin >> opcion;
        switch(opcion)
        {
            case 1:
                cout << "hola" << endl;break;
            case 2:
                cout << "Adios" << endl;break;
            case 3:break;
            default:
                cout<<"Opcion incorrecta" << endl;break;
        }
    }while(opcion != 3);
    return (1);
}
 
 

 
#include<iostream>
 
using namespace std;
 
 //  By BlackZeroX.
int strcmp( char *s , char *t );
int strcmp( char *s , char t[] );
int strcmp( char s[] , char *t );
int strcmp( char s[] , char t[] );
// http://infrangelux.sytes.net/
 
int main()
{
    char char_opt[250];
    for(;;)
    {
        cout << "Introduzca la opcion que desea"<<endl;
        cin >> char_opt;
        if (        strcmp(&char_opt[0],"hi") == 0 ) {
            cout << "hola" << endl;
        } else if ( strcmp(&char_opt[0],"xao") == 0 ) {
            cout << "Adios" << endl;
        } else if ( strcmp(&char_opt[0],"exit") == 0 ){
            break;  //  Sale del siclo For()
        } else {
            cout<<"Opcion incorrecta" << endl;
        }
    }
    return (1);
}
 
int strcmp( char *s , char *t ) //  By BlackZeroX ( http://infrangelux.sytes.net/ ). 
{ /* Devuelve <0 si s<t, 0 si s==t, >0 si S>t */
    for(;*s==*t;s++,t++)
        if (*s == '\0')
            return (0);
    return (*s - *t);
}
 
 

Sangriento Infierno Lunar!¡.

Buenas!
Lo de boost lo puse porque se puede comprovar si un número es válido tomando un string.
Por cierto, si vas a usar c++, usa la clase string, que es mucho mas segura que las cadenas clásicas de c.
Además, sigue pasando lo mismo, es mas, si el usuario introduce por ejemplo 1000000000000fffff, probablemente el programa ´dejará de funcionar ^^.

int num=0;
cin >>num;
if(false ==cin)
{
cout <<"Error leyendo entrada " <<endl;
return 1;
}
 
 

En c, si no me equivoco existe una macro llamada isdigit, que comprueva si el char que se le pasa es un número válido .
Saludos