2

Estoy implementando este código el cual es para proyectar las vocales usando una matriz y para identificar si una palabra o frase es palindromo.

Todo iba bien hasta que me marco un error en la opción de la salida

#include <iostream>
#include <stdlib.h>
#include <stdio.h>
using namespace std;

void gvocales(char vocal);
bool palindromo (string frase);

int main() {
    char op;
    cout<<"*** COMPILACION 02 ***"<<endl;
    do{

    cout<<"Elija la opcion: \n 1-Graficar Vocales. \n 2-Analizador de palindromos. \n 3-salir \n --> \n";
    cin>>op;
    system("cls");


    switch (op)
    {
        case '1':
            cout<<"*** Grafica de vocales ***"<<endl;
            char vocal;
            cout<<"ingrese la vocal: ";
            cin>>vocal;
            cout<<endl;
            gvocales(vocal);
        break;
        case '2':
            cout<<"*** Palindromos ***"<<endl;
            string frase;
            cout<<"ingrese la Frase/Palabra: ";
            cin.ignore();
            getline(cin,frase);
            if(palindromo(frase)){  cout<<endl<<"-"<<frase<<"- es palindromo"<<endl;    }
            else{   cout<<endl<<"-"<<frase<<"- no es palindromo"<<endl; }
        break;
        case '3':
            cout<<"saliendo del programa...";
            break;




         }
    system("pause");
         }while  (op!='3');
}



void gvocales(char vocal)
{
    char matriz[5][5];
        for (int i=0;i<5;i++)
        {
            for (int j=0;j<5;j++){ matriz[i][j]='O';}
        }
    switch (vocal){

    case 'a': 
        cout<<"vocal A"<<endl<<endl;
        for (int i=0;i<5;i++)
        {
            for (int j=0;j<5;j++)
            {
                if(i==0 || i==2 || j==0 || j==4){   cout<<matriz[i][j];     }
                else{   cout<<" ";      }
            }
            cout<<endl;
        }
    break;
    case 'e': 
        cout<<"vocal E"<<endl<<endl;
        for (int i=0;i<5;i++)
        {
            for (int j=0;j<5;j++)
            {
                if(i==0 || i==2 || i==4 || j==0){   cout<<matriz[i][j];     }
                else{   cout<<" ";  }
            }
            cout<<endl;
        }
    break; 
    case 'i': 
        cout<<"vocal I"<<endl<<endl;
        for (int i=0;i<5;i++)
        {
            for (int j=0;j<5;j++)
            {
                if(i==0 || i==4 || j==2){   cout<<matriz[i][j];     }
                else{   cout<<" ";  }
            }
            cout<<endl;
        }
    break;
    case 'o': 
        cout<<"vocal O"<<endl<<endl;
        for (int i=0;i<5;i++)
        {
            for (int j=0;j<5;j++)
            {
                if(i==0 || i==4 || j==0|| j==4) {   cout<<matriz[i][j]; }
                else{   cout<<" ";  }
            }
            cout<<endl;
        }
    break;
    case 'u': 
        cout<<"vocal U"<<endl<<endl;
        for (int i=0;i<5;i++)
        {
            for (int j=0;j<5;j++)
            {
                if(i==4 || j==0|| j==4) {   cout<<matriz[i][j]; }
                else{   cout<<" ";  }
            }
            cout<<endl;
        }
    break;
    }
}



bool palindromo (string frase)
{
    int n= frase.length();
    string aux= "";
    for (int i=0;i<n;i++)
    {
        if(frase[i]!=' ' && frase[i]!=',' && frase[i]!='.')
        {
            aux += frase [i];
        }
    }
    frase=aux;
    n= frase.length();
    for (int i=0;i<n/2;i++)
    {
        if(frase[i]!=frase[n-1-i]) return false;
    }
    return true;
}
1

1 respuesta 1

14

Buenas, el problema se debe a la declaración de variables dentro de un case. Si quieres declarar variables en un case tienes que usar las llaves {} para asegurar que el alcance (scope) de estas variables se limita a ese case. Por ejemplo:

switch(op)
{
    case 1: 
    {
       string frase;
       foo();
    }
    break;
    case 2: 
    {
       bar();
    }
    break;
    case 3: 
    {
       exit();
    }
    break;
}    

Por tanto, puedes elegir 2 opciones:

  1. Usar llaves {} en los case donde declares variables (o en todos).
  2. Declarar las variables fuera del switch.

Te recomiendo la opción 1, ya que te permite tener el código más limpio y estructurado.

1
  • Se te olvida una tercera... usar funciones
    – eferion
    Commented el 26 ago. 2017 a las 17:08

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.