Ayuda! Validacion de Datos en Ordenamiento.

BKsiragon · 1 Febrero 2013, 14:09 PM

Amigos me pueden ayudar con este codigo, lo que pasa es que en la parte donde sale la opcion de "INGRESE UNA OPCION" si pongo una letra o un numero que no es igual a los que estan en el menu sale un bug y empieza a repetir el programa...

quiero ponerle una opcion que diga "Error, opcion invalida" al marcar una letra o un numero incorrecto a la hora de ingresar la opcion del menu.

aqui el codigo:

Código [Seleccionar]

/* 

#include <iostream>
#include <stdio.h>
#include<windows.h>
#define MAX 10 
using namespace std;
 
 
void burbuja();
void seleccion();
void salir();
 
main(void)
 
{
 
system("color 0A"); //COLOR PARA LA CONSOLA
 
int seguir=1;
int x;     
 
cout<<endl<<"\t##################################################################";
cout<<endl<<"\t#                    METODOS DE ORDENAMIENTO                     #";
cout<<endl<<"\t#            METODO DE LA BURBUJA Y SELECCION DIRECTA            #"; //PORTADA DEL PROMAGRA!
cout<<endl<<"\t#                              IS5D-C                            #";           
cout<<endl<<"\t##################################################################"<<endl;
cout<<endl;
cout<<endl;

do // COMIENZO DEL CICLO REPETITIVO DO - WHILE
{
cout<<endl<<"\t\t******************* M E N U ******************";
cout<<endl<<"\t\t*                                            *";
cout<<endl<<"\t\t*     1.- Ordenamiento por Metodo Burbuja    *";
cout<<endl<<"\t\t*                                            *";
cout<<endl<<"\t\t*     2.- Ordenamiento por Metodo Seleccion  *";  //MENU DEL PROGRAMA!
cout<<endl<<"\t\t*                                            *";
cout<<endl<<"\t\t*     3.- Salir                              *";
cout<<endl<<"\t\t**********************************************";
cout<<endl;
cout<<endl;
 
cout<<endl<<">> ELIGE UNA OPCION: ";                              //PEDIR OPCION DEL MENU!
cout<<endl<<"   -------------------> ";
cin>>x;
cout<<endl<<endl;
   
             

 
 
switch (x){
case 1:burbuja();
break;
case 2:seleccion();
break;
case 3:
seguir = 0; 
break;
 
 
default: cout<<"ERROR, OPCION INVALIDA.."<<endl; //OPCION INVALIDA.
}
}
if (seguir!=0)
{
cout <<endl<< "DESEA CONTINUAR??  SI - PRESIONAR 1 ,  NO - PRESIONAR 0 "; 
cin >> seguir;
}
}while (seguir==1);   //FIN DEL BUCLE DO - WHILE
cout<<endl;
system("PAUSE");
}

 
void burbuja()
{
int I,N,Aux,J;
int V[10];
 
 
 
cout<<"Cuantos Numeros va a Ingresar al Vector?, Maximo 10: ";
cin>>N;
cout<<endl;
for(I=0; I<N; I++)
{
cout<<"Ingrese Dato "<<I+1<<": ";
cin>>V[I];
}
for (I=0; I<N-1; I++)
for (J=0; J<(N-1)-I; J++)
if (V[J]>V[J+1])
{
Aux=V[J];
V[J]=V[J+1];
V[J+1]=Aux ;
}
 
cout<<endl<<"[ NUMEROS ORDENADOS MEDIANTE BURBUJA ] "<<endl;
for(I=0; I<N; I++)
cout<<endl<<"\t\t"<<V[I];
cout<<endl<<endl;
 
 
}
 
 
 
void seleccion()
{
int J,I,N,K;
int V[10];
int Aux;
 
cout<<"Cuantos Numeros va a Ingresar al Vector?, Maximo 10: ";
cin>>N;
cout<<endl;
for(I=0; I<N; I++)
 
 
{   
cout<<"Ingrese Dato "<<I+1<<": ";
cin>>V[I];
}
 
for(I=0; I<N-1; I++)
{
K=I;
for (J=I+1; J<N; J++)
if (V[J]<V[K])
K=J;
 
Aux=V[I];
V[I]=V[K];
V[K]=Aux;
}
 
 
cout<<endl<<"[ NUEMROS ORDENADOS MEDIANTE SELECCION ]"<<endl;
for(I=N-1; I>=0; I--)
cout<<endl<<"\t\t"<<V[I];
cout<<endl<<endl;
system("exit");

}

durasno · 1 Febrero 2013, 18:18 PM

Hola! si lo compilas, el compilador te va a decir el error...

Código (c) [Seleccionar]

default: cout<<"ERROR, OPCION INVALIDA.."<<endl; //OPCION INVALIDA.
}
}

hay una llave de cierre que esta demas. Saludos

ThePinkPanther · 1 Febrero 2013, 21:56 PM

switch (x){
case 1:burbuja();
break;
case 2:seleccion();
break;
case 3:
seguir = 0;
break;
default: cout<<"ERROR, OPCION INVALIDA.."<<endl; //OPCION INVALIDA.
break; //te falto el break
}

DickGumshoe · 1 Febrero 2013, 22:22 PM

Cita de: Puredepapas en 1 Febrero 2013, 21:56 PM
switch (x){
case 1:burbuja();
break;
case 2:seleccion();
break;
case 3:
seguir = 0;
break;
default: cout<<"ERROR, OPCION INVALIDA.."<<endl; //OPCION INVALIDA.
break; //te falto el break
}

Si no recuerdo mal, en el 'default' del switch no es necesario añadir "break", puesto que detrás no hay ningún 'case'.

naderST · 1 Febrero 2013, 22:37 PM

Cita de: Puredepapas en 1 Febrero 2013, 21:56 PM
switch (x){
case 1:burbuja();
break;
case 2:seleccion();
break;
case 3:
seguir = 0;
break;
default: cout<<"ERROR, OPCION INVALIDA.."<<endl; //OPCION INVALIDA.
break; //te falto el break
}

Es redundante colocar el break ahí, la aplicación simplemente va a seguir con su ejecución normal, esto se debe a que cuando se cumple uno de los casos dentro del switch la aplicación continua su ejecución secuencial y como podemos observar que el caso "default' es el último no hace falta colocar un break para romper con la estructura de control.

ThePinkPanther · 1 Febrero 2013, 22:53 PM

Cita de: naderST en 1 Febrero 2013, 22:37 PM
Es redundante colocar el break ahí, la aplicación simplemente va a seguir con su ejecución normal, esto se debe a que cuando se cumple uno de los casos dentro del switch la aplicación continua su ejecución secuencial y como podemos observar que el caso "default' es el último no hace falta colocar un break para romper con la estructura de control.

Tienes razón no influye en nada.. perdon por lo anterior..

Así quedaría el codigo arreglado , lo unico que hice fue cambiar la variable x (int) , por una variable tipo char , para asi puedo comprobar los datos ingresados con la tabla ascii y verificar si el dato ingresado es un numero o no.

Codigo :

#include <iostream>
#include <stdio.h>
#include<windows.h>
#define MAX 10
using namespace std;

void burbuja();
void seleccion();
void salir();

main(void)

{

system("color 0A"); //COLOR PARA LA CONSOLA

int seguir=1;
char x;

cout<<endl<<"\t##################################################################";
cout<<endl<<"\t# METODOS DE ORDENAMIENTO #";
cout<<endl<<"\t# METODO DE LA BURBUJA Y SELECCION DIRECTA #"; //PORTADA DEL PROMAGRA!
cout<<endl<<"\t# IS5D-C #";
cout<<endl<<"\t##################################################################"<<endl;
cout<<endl;
cout<<endl;

do // COMIENZO DEL CICLO REPETITIVO DO - WHILE
{
cout<<endl<<"\t\t******************* M E N U ******************";
cout<<endl<<"\t\t* *";
cout<<endl<<"\t\t* 1.- Ordenamiento por Metodo Burbuja *";
cout<<endl<<"\t\t* *";
cout<<endl<<"\t\t* 2.- Ordenamiento por Metodo Seleccion *"; //MENU DEL PROGRAMA!
cout<<endl<<"\t\t* *";
cout<<endl<<"\t\t* 3.- Salir *";
cout<<endl<<"\t\t**********************************************";
cout<<endl;
cout<<endl;

cout<<endl<<">> ELIGE UNA OPCION: "; //PEDIR OPCION DEL MENU!
cout<<endl<<" -------------------> ";
cin>>x;
if(x<48 || x>57) /* si es un caracter y no un numero vuelve al do*/
{
cout<<"Un numero porfavor.."<<endl;
continue;
}
cout<<endl<<endl;

switch (x){

case '1':burbuja(); /*recuerda va entre ' ' , estamos hablando de char*/
break;
case '2':seleccion();
break;
case '3':
seguir = 0;
break;

default: cout<<"ERROR, OPCION INVALIDA.."<<endl; //OPCION INVALIDA.
}

if (seguir!=0)
{
cout <<endl<< "DESEA CONTINUAR?? SI - PRESIONAR 1 , NO - PRESIONAR 0 ";
cin >> seguir;
}
}while (seguir==1); //FIN DEL BUCLE DO - WHILE
cout<<endl;
system("PAUSE");
}

void burbuja()
{
int I,N,Aux,J;
int V[10];

cout<<"Cuantos Numeros va a Ingresar al Vector?, Maximo 10: ";
cin>>N;
cout<<endl;
for(I=0; I<N; I++)
{
cout<<"Ingrese Dato "<<I+1<<": ";
cin>>V;
}
for (I=0; I<N-1; I++)
for (J=0; J<(N-1)-I; J++)
if (V[J]>V[J+1])
{
Aux=V[J];
V[J]=V[J+1];
V[J+1]=Aux ;
}

cout<<endl<<"[ NUMEROS ORDENADOS MEDIANTE BURBUJA ] "<<endl;
for(I=0; I<N; I++)
cout<<endl<<"\t\t"<<V;
cout<<endl<<endl;

}

void seleccion()
{
int J,I,N,K;
int V[10];
int Aux;

cout<<"Cuantos Numeros va a Ingresar al Vector?, Maximo 10: ";
cin>>N;
cout<<endl;
for(I=0; I<N; I++)

{
cout<<"Ingrese Dato "<<I+1<<": ";
cin>>V;
}

for(I=0; I<N-1; I++)
{
K=I;
for (J=I+1; J<N; J++)
if (V[J]<V[K])
K=J;

Aux=V;
V=V[K];
V[K]=Aux;
}

cout<<endl<<"[ NUEMROS ORDENADOS MEDIANTE SELECCION ]"<<endl;
for(I=N-1; I>=0; I--)
cout<<endl<<"\t\t"<<V;
cout<<endl<<endl;
system("exit");

}

durasno · 1 Febrero 2013, 23:41 PM

Si con el if obligas a hacer un continue, entonces el default del switch no tiene sentido ya que nunca va a entrar ahi

Saludos

ThePinkPanther · 2 Febrero 2013, 01:13 AM

Cita de: durasno en 1 Febrero 2013, 23:41 PM
Si con el if obligas a hacer un continue, entonces el default del switch no tiene sentido ya que nunca va a entrar ahi

Saludos

Si tiene sentido , juega el papel de verificar que el numero de opcion sea el correcto .
El if- continue , solo verifica que aya entrado un dato numerico y no un caracter.
saludos.

durasno · 2 Febrero 2013, 18:48 PM

Tenes razon, nose q estaba viendo. Igual solo con el default alcanzaria

SAludos

leosansan · 2 Febrero 2013, 21:30 PM

Nos podemos ahorrar la variable "seguir" y la pregunta "desea continuar" sin más que elegir bien el número, y con etiquetas Code y tabulado queda mejor:

Código (cpp) [Seleccionar]

#include <iostream>
#include<windows.h>
#define MAX 10
using namespace std;

  void burbuja();
  void seleccion();
  
  int main(void){

  system("color 0A"); //COLOR PARA LA CONSOLA
  char x;
  cout<<endl<<"\t##################################################################";
  cout<<endl<<"\t#                    METODOS DE ORDENAMIENTO                     #";
  cout<<endl<<"\t#            METODO DE LA BURBUJA Y SELECCION DIRECTA            #"; //PORTADA DEL PROMAGRA!
  cout<<endl<<"\t#                              IS5D-C                            #";
  cout<<endl<<"\t##################################################################"<<endl;
  cout<<endl;
  cout<<endl;

  cout<<endl<<"\t\t******************* M E N U ******************";
  cout<<endl<<"\t\t*                                            *";
  cout<<endl<<"\t\t*     1.- Ordenamiento por Metodo Burbuja    *";
  cout<<endl<<"\t\t*                                            *";
  cout<<endl<<"\t\t*     2.- Ordenamiento por Metodo Seleccion  *";  //MENU DEL PROGRAMA!
  cout<<endl<<"\t\t*                                            *";
  cout<<endl<<"\t\t*     3.- Salir                              *";
  cout<<endl<<"\t\t**********************************************";
  cout<<endl;
  cout<<endl;

  cout<<endl<<">> ELIGE UNA OPCION: ";                              //PEDIR OPCION DEL MENU!
  cout<<endl<<"   -------------------> ";

  /*********************************************/
  do{
    cout<<"Un numero del 1 al 3."<<endl;
    cin>>x;
    }while ( (x)<49 || (x)>51 );
  /*********************************************/
  switch (x){
    case '1':burbuja();
    break;
    case '2':seleccion();
    break;
    case '3':
    break;
    default: cout<<"ERROR, OPCION INVALIDA.."<<endl; //OPCION INVALIDA.
  }
  return 0;
}


void burbuja() {
  int I,N,Aux,J;
  int V[10];
  cout<<"Cuantos Numeros va a Ingresar al Vector?, Maximo 10: ";
  cin>>N;
  cout<<endl;
  for(I=0; I<N; I++) {
    cout<<"Ingrese Dato "<<I+1<<": ";
    cin>>V[I];
  }
  for (I=0; I<N-1; I++)
    for (J=0; J<(N-1)-I; J++)
      if (V[J]>V[J+1]){
        Aux=V[J];
        V[J]=V[J+1];
        V[J+1]=Aux ;
    }
  cout<<endl<<"[ NUMEROS ORDENADOS MEDIANTE BURBUJA ] "<<endl;
  for(I=0; I<N; I++)
    cout<<endl<<"\t\t"<<V[I];
  cout<<endl<<endl;
}

void seleccion() {
  int J,I,N,K;
  int V[10];
  int Aux;
  cout<<"Cuantos Numeros va a Ingresar al Vector?, Maximo 10: ";
  cin>>N;
  cout<<endl;
  for(I=0; I<N; I++){
    cout<<"Ingrese Dato "<<I+1<<": ";
    cin>>V[I];
  }
  for(I=0; I<N-1; I++){
    K=I;
    for (J=I+1; J<N; J++)
    if (V[J]<V[K])
      K=J;
    Aux=V[I];
    V[I]=V[K];
    V[K]=Aux;
  }
  cout<<endl<<"[ NUEMROS ORDENADOS MEDIANTE SELECCION ]"<<endl;
  for(I=N-1; I>=0; I--)
    cout<<endl<<"\t\t"<<V[I];
  cout<<endl<<endl;
  system("exit");
}

Saluditos!.
P.D: ¿Puredepapas?, yo creo que estaría mejor mojitopicón o papitasarrugadas.