1

Ayuda, el programa no reconoce el dato Ingresado y sigue repitiendo el ciclo:

intro#include <iostream>
using namespace std;
int main(){
int contM=0,contJ=0,contP=0,contN=0;
char opcion;
do{
    cout<<" Candidatos:  \n"
        <<" M-Maria \n"
        <<" J-Juan \n"
        <<" P-Pedro \n"
        <<" S-Salir \n\t";
    cin>>opcion;
        switch (opcion){
            case 'm':
            case 'M':
                contM++;
                break;
            case 'j':
            case 'J':
                contJ++;
                break;
            case 'p':
            case 'P':
                contP++;
            default : contN++;
            break;          
        }
}while (opcion !='s')||( opcion!='S');
cout<<"\n";
cout<<" Resultados Maria: "<<contM;
cout<<"\n";
cout<<" Resultados Juan: "<<contJ;
cout<<"\n";
cout<<" Resultados Pedro: "<<contP;
cout<<"\n";
cout<<" Resultados Votos Nulos: "<<contN;
}
4
  • podrias agregar tu codigo completo Commented el 21 jun. 2018 a las 19:50
  • deberías leer algo de lógica proposicional, los if las manipulan bastante Commented el 21 jun. 2018 a las 21:18
  • Te recomiendo, de cara a un futuro, que formatees tu código a la hora de publicarlo en SOes. Principalmente, porque eso nos ayuda a nosotros a leerlo, y a ti, a aprender a escribir mejor tu código. Así localizaremos tu error con más rapidez, y al mismo tiempo, te indicaríamos más detalles de tu código que no indicas (1/2) Commented el 17 oct. 2018 a las 6:43
  • (2/2) Tal como no indicar cuando haces un fall through como haces en esos cases, no usar el using namespace std;, o declarar main sin argumentos y sin especificar qué argumentos de entrada tiene int main (void), int main(int argc, const char **argv), int main(int argc, const char **argv, const char **envp), entre otros. El espaciado también es importante, y te habrás fijado que los tabuladores no son muy bien aceptados por el sistema, así que usar espacios será una idea para identar tu código. Todo sea para que podamos leerlo mejor y ayudarte a mejorar. Commented el 17 oct. 2018 a las 6:45

2 respuestas 2

1

El problema es que tu condicion esta mal. Supongamos que el usuario ingresa S, entonces la condicion opcion != 's' es verdadera. Al usar el comparador OR, toda la condicion se hace verdadera. Lo que necesitas es usar AND en lugar de OR asi:

do{
cout<<" Candidatos:  \n"
    <<" M-Maria \n"
    <<" J-Juan \n"
    <<" P-Pedro \n"
    <<" S-Salir \n\t";
cin>>opcion;
    switch (opcion){
        case 'm':
        case 'M':
            contM++;
            break;
        case 'j':
        case 'J':
            contJ++;
            break;
        case 'p':
        case 'P':
            contP++;
        default : contN++;
        break;          
    }
}while ((opcion !='s')&&( opcion!='S'));
2
  • Gracias, esta es la respuesta.
    – Dave
    Commented el 21 jun. 2018 a las 20:47
  • De nada @Dave Saludos
    – alanfcm
    Commented el 21 jun. 2018 a las 20:48
0

¿Podrías probar a añadir unos paréntesis en la condición del while?

do {
  // ...
} while((opcion != 's') && (opcion != 'S'));
1
  • Lo Mismo no funciona
    – Dave
    Commented el 21 jun. 2018 a las 20:39

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.