[SOLUCIONADO] Reinicio del programa

[HelThunk]

Buenas, he intentado hacer que mi programa le pregunte al usuario si desea reinciar la aplicación y que este, contestando si o no, haga que se cierre o se reinicie el programa.. podrían echarme una mano (soy muy nuevo, lo siento);

Código C++:

#include "stdafx.h"
#include <iostream>
using namespace std;
 
 
int main(){
    system("cls");
 
    bool exit = false;
 
    do{
    int num1, num2;
    //Escanear digitos;
    cout << "Escribe dos numeros;" << endl;
    cin >> num1 >> num2;
    //Ordenar de mayor a menor;
    if (num1 > num2){
        cout << "" << num1 << " es mayor que " <<num2<<endl;
    }
    else if (num1 < num2){
        cout << "" << num1 << " es menor que " << num2<<endl;
    }
    //Es igual;
    else cout << "" << num1 << " es igual a " << num2 << endl;
    //Repetición
 
    cout << "¿Desea usted continuar?" << endl;
    char cont;
    cin >> cont;
    switch(cont){
        case 'Si':
        case 'si':
        case 'SI':
        case 'sI': return 1;
            break;
        case 'No':
        case 'nO':
        case 'NO':
        case 'no': break;
        defaul:cout << "Por favor, escriba 'si' o 'no'" << endl;
    }
    
    } while (!exit);
 
    system("pause");
    return 0;
}
 

[razpeitia]

Esa cosa ni siquiera compila!

Por favor danos algo que al menos compile.

[HelThunk]

Iniciado por razpeitia

Esa cosa ni siquiera compila!

Por favor danos algo que al menos compile.

A mi si me compila :s, el problema es que después de la pregunta de si deseas continuar salta el error.

Un saludo

[vangodp]

Los errores son:
-No puedes guardar mas de una letra en un char 'si' es ilegal, legal es 's' por ejemplo.
-Lo mismo pasa con switch. No puedes poner 'Si' solo puedes poner 'a' o un entero, float etc switch no compara strings, para eso use if que va tener mismo efecto.

Una forma de arreglarlo seria:

Código C++:

#include <iostream>
#include <windows.h>
using namespace std;
 
 
int main() {
    bool exit = false;
    
    do {
        int num1, num2;
        //Escanear digitos;
        cout << "Escribe dos numeros;" << endl;
        cin >> num1 >> num2;
        
        //Ordenar de mayor a menor;
        if ( num1 > num2 ) {
            cout << "" << num1 << " es mayor que " << num2 << endl;
        } else if ( num1 < num2 ) {
            cout << "" << num1 << " es menor que " << num2 << endl;
        }
        //Es igual;
        else cout << "" << num1 << " es igual a " << num2 << endl;
        
        //Repetición
        bool salir = false;
        
        while ( !salir ) {
            cout << "¿Desea usted continuar?" << endl;
            string opcion;
            cin >> opcion;
            
            if ( opcion == "si" || opcion == "SI" || opcion == "Si" || opcion == "sI" ) {
                salir = true;
                system ( "cls" );
            } else if ( opcion == "no" || opcion == "NO" || opcion == "No" || opcion == "nO" ) {
                salir = true;
                exit = true;
                system ( "cls" );
                //al decir no saliremos de este bucle interno y tambien del externo
            } else {
                system ( "cls" );
                cout << "No existe esta opcion. ";
            }
        }
    } while ( !exit );
    
    system ( "pause" );
    return 0;
}
 

Hay muchas formas de salir. Esta no es la mejor ni la más efectiva, pero cumple lo que propone.

PD: Sobre lo de no compilar es solo quitar el archivo ese header stdafx, ese archivo es de visual studio y en codeblocks no hace falta.
Añadir cstdlib o windows.h para el system("cls"); y el system("pause");. Lo mejor seria no usar nunca system ya que no es una solución portable.

[HelThunk]

Iniciado por vangodp

Los errores son:
-No puedes guardar mas de una letra en un char 'si' es ilegal, legal es 's' por ejemplo.
-Lo mismo pasa con switch. No puedes poner 'Si' solo puedes poner 'a' o un entero, float etc switch no compara strings, para eso use if que va tener mismo efecto.

Una forma de arreglarlo seria:

Código C++:

#include <iostream>
#include <windows.h>
using namespace std;
 
 
int main() {
    bool exit = false;
    
    do {
        int num1, num2;
        //Escanear digitos;
        cout << "Escribe dos numeros;" << endl;
        cin >> num1 >> num2;
        
        //Ordenar de mayor a menor;
        if ( num1 > num2 ) {
            cout << "" << num1 << " es mayor que " << num2 << endl;
        } else if ( num1 < num2 ) {
            cout << "" << num1 << " es menor que " << num2 << endl;
        }
        //Es igual;
        else cout << "" << num1 << " es igual a " << num2 << endl;
        
        //Repetición
        bool salir = false;
        
        while ( !salir ) {
            cout << "¿Desea usted continuar?" << endl;
            string opcion;
            cin >> opcion;
            
            if ( opcion == "si" || opcion == "SI" || opcion == "Si" || opcion == "sI" ) {
                salir = true;
                system ( "cls" );
            } else if ( opcion == "no" || opcion == "NO" || opcion == "No" || opcion == "nO" ) {
                salir = true;
                exit = true;
                system ( "cls" );
                //al decir no saliremos de este bucle interno y tambien del externo
            } else {
                system ( "cls" );
                cout << "No existe esta opcion. ";
            }
        }
    } while ( !exit );
    
    system ( "pause" );
    return 0;
}
 

Hay muchas formas de salir. Esta no es la mejor ni la más efectiva, pero cumple lo que propone.

PD: Sobre lo de no compilar es solo quitar el archivo ese header stdafx, ese archivo es de visual studio y en codeblocks no hace falta.
Añadir cstdlib o windows.h para el system("cls"); y el system("pause");. Lo mejor seria no usar nunca system ya que no es una solución portable.

Perdón por seguir molestando, he comprendido tu explicación, pero al probarla en la linea 30;

Código C++:

string opcion;
cin >> opcion;
 

no me lee ese cin>>opcion, me sale un error si intento ejecutar el programa y me marca los signos de 'mayor que' en rojo.

[vangodp]

poner como std::cin o incluya el using namespace std; luego debajo de haber incluido iostream arriba del todo.

A mi si me compila con cero errores. :/

Usas visual studio?

Pruebe si no poner string opcion; antes del bucle arriba al principio de main.

[HelThunk]

Iniciado por vangodp

poner como std::cin o incluya el using namespace std; luego debajo de haber incluido iostream arriba del todo.

A mi si me compila con cero errores. :/

Usas visual studio?

Pruebe si no poner string opcion; antes del bucle arriba al principio de main.

Así es, uso visual studio, he probado el std:: y tengo puesta la biblioteca <iostream> (solo me pasa con ese cin, los demás funcionan bien), si a ti te funciona no comprendo que puede ser (si cambio el interior del cin por otra variable del tipo entero, char.. si funciona, solo sale el error cuando es string)

[vangodp]

prueba una cosa...
Despues de cada cin, luego abajo ponga cin.sync(); a ver

Guardas el archivo como .cpp?

vuelva a poner el stdafx a ver si se soluciona. Quizas para VS sea imprecindible.

[HelThunk]

Iniciado por vangodp

prueba una cosa...
Despues de cada cin, luego abajo ponga cin.sync(); a ver

Guardas el archivo como .cpp?

vuelva a poner el stdafx a ver si se soluciona. Quizas para VS sea imprecindible.

Si lo guardo como ccp, he probado a poner cin.sync(); y tampoco..

Adjunto imágenes para intentar facilitar las cosas;





Un saludo!

[vangodp]

pues ya ves el error tonto que es... Es un simple string :S

No se por que te tira error el programa. sustituya el cin ese por: std::getline (std::cin,opcion); a ver si te funciona. Si no compila ponga el include <cstring>
Quita los cin.sync

y has dicho ccp ¿No guardas como CPP? :O

[HelThunk]

Iniciado por vangodp

pues ya ves el error tonto que es... Es un simple string :S

No se por que te tira error el programa. sustituya el cin ese por: std::getline (std::cin,opcion); a ver si te funciona. Si no compila ponga el include <cstring>
Quita los cin.sync

y has dicho ccp ¿No guardas como CPP? :O

También me tira error con std::getline (std::cin,opcion); , añadiendo la biblioteca <cstring>, etc.. creo que esto no vamos a poder solucionarlo :s

*Me he confundido, guardo en cpp.

[vangodp]

pero a ver? Como lees tu un string en visual studio?

[HelThunk]

Iniciado por vangodp

pero a ver? Como lees tu un string en visual studio?

Lo acabo de solucionar, me dijiste antes que incluyese #include <cstring>, he probado a incluir #include <string> y ahora si funciona, muchas gracias por tu ayuda, ahora va perfectamente.

[vangodp]

De nada.
Es que de visual studio estoy pillado. Como nunca lo use =(

[eferion]

Iniciado por vangodp

Guardas el archivo como .cpp

Una pequeña aclaración... la extensión que se le ponga a los ficheros es totalmente irrelevante... puedes, si quieres, hacerlo todo al reves y tener la implementación en los .h y la declaración en los .cpp (recuerda actualizar los includes y que ahora únicamente debes compilar los .h)... o en la extensión que te de la gana y la compilación no se verá afectada... al compilador únicamente le estás diciendo que compile una serie de archivos... el los coge y los procesa y no se preocupa para nada por la extensión de los mismos.

El motivo de que las cabeceras sean .h y los archivos de código .cpp es simplemente por estandarización... así cualquiera que coja ese proyecto puede hacerse una idea acerca de cómo está organizado el código... si cada uno usase la extensión que le diese la gana podría ser traumático usar librerías externas...

Iniciado por HelThunk

También me tira error con std::getline (std::cin,opcion); , añadiendo la biblioteca <cstring>, etc.. creo que esto no vamos a poder solucionarlo

Las librerías que empiezan con "c" suelen ser simples "wrappers" de las antiguas librerías de C. Así, en C tienes la librería "string.h" que en C++ pasa a ser "cstring"... o la librería de C "math.h" que pasa a ser "cmath". En teoría puedes hacer un include a cualquiera de las dos versiones. Aunque ambas versiones son compatibles, no tienen por qué ser exactamente iguales. Un ejemplo:

• La función memchr(const void*, int, size_t) de "string.h" es reemplazada en "cstring" por dos funciones diferentes:
  • const void* memchr(const void* s, int c, size_t n);
  • void* memchr( void* s, int c, size_t n);

Un saludo