2
#include<iostream>
#include<string.h>
using namespace std;
class jugador{
    char *nombre;
    float arg;
    char *equipo;
    int aux;

    public:
        jugador(char *, char *, float);
       ~jugador();
        jugador(jugador &j);
        char *getnombre();
        char *getequipo();
        float getarg();
        void setnombre(char *n);
        void setequipo(char *e);
        void setarg(float a);
        void operator=(jugador &p)
        {
            cout<<endl;
            cout<<"sobrecarga del operador"<<endl;
            aux=p.aux;
            nombre=new char[aux+1];
            strcpy(nombre,p.nombre);
            aux=p.aux;
            equipo=new char[aux+1];
            strcpy(equipo,p.equipo);
            arg=p.arg;
            }
            void mostrar_jugador();
};
jugador::jugador(char *n,char *e, float a){
    aux= ::strlen(n);
    nombre=new char[aux+1];
    strcpy(nombre, n);
    aux= ::strlen(n);
    equipo=new char[aux+1];
    strcpy( equipo, e);
    arg=a;
}
jugador::jugador(jugador &j){
    cout<<endl;
    cout<<"constructor copia"<<endl;
    aux=j.aux;
    nombre=new char[aux+1];
    strcpy(nombre,j.nombre);
    cout<<"se creo la copia de la clase para "<<nombre<<endl;
    aux=j.aux;
    equipo=new char[aux+1];
    strcpy(equipo,j.equipo);
    cout<<"se creo la copia de la clase para "<<equipo<<endl;
    arg=j.arg;
}
void jugador::mostrar_jugador(){
    cout<<endl;
    cout<<"Nombre: "<<nombre<<endl;
    cout<<"Equipo: "<<equipo<<endl;
    cout<<"Arg: "<<arg<<endl;
    }
jugador::~jugador(){
}
main(){ 
    int op;
    cout<<"1.Ingreso de 3 jugadores"<<endl;
    cout<<"2.Crear copia de los 3 jugadores"<<endl;
    cout<<"3.Mostrar a los jugadores"<<endl;
    cout<<"4.Mostrar mejor jugador"<<endl;
    cout<<"5.salir"<<endl;
    cout<<"----Elija una opcion----"<<endl;
    cin>>op;
switch(op){
    case 1:
    jugador p((char*)"Edwin", (char*)"Tigres", 20 );
    jugador p2((char*)"Maria", (char*)"Rapidos", 21 );
    jugador p3((char*)"Julio", (char*)"Triunfadores", 30 );
        break;
    case 2://aquí empieza el problema no se porque
    jugador x1(p);
        x1.mostrar_jugador();
    jugador x2(p2);
        x2.mostrar_jugador();
    jugador x3(p3);
        x3.mostrar_jugador();
        break;
    case 3:
        break;
    case 4:
        break;
    default:
        cout<<"Opcion invalida"<<endl;
}   
    return 0;
}

1 respuesta 1

4

Los bloques switch son diferentes a los bloques if-else por varios motivos:

  • El compilador tiene mayor margen de libertad para aplicar optimizaciones.
  • Las etiquetas case no tienen ámbito propio.

Para evitar la aparición de incongruencias y errores difíciles de encontrar debido a la segunda característica, dentro de cada case no se deben declarar variables. Estas podrían colisionar con las variables de otro case y su ciclo de vida no es sencillo de acotar.

Para poder declarar variables dentro de un case es necesario crear un ámbito, lo cual se consigue con las llaves { }. Ahora bien, si revisamos tu código:

switch(op){
  case 1:
    jugador p((char*)"Edwin", (char*)"Tigres", 20 );
    jugador p2((char*)"Maria", (char*)"Rapidos", 21 );
    jugador p3((char*)"Julio", (char*)"Triunfadores", 30 );
    break;
  case 2://aquí empieza el problema no se porque
    jugador x1(p);
    x1.mostrar_jugador();
    jugador x2(p2);
    x2.mostrar_jugador();
    jugador x3(p3);
    x3.mostrar_jugador();
    break;

Vemos que estás declarando variables en el case 1 que luego intentas reutilizar en el case 2... pero claro si op==2 es imposible que op==1. Dicho de otra manera: si se ejecuta el case 2 no se va a ejecutar el case 1, luego las variables p, p2 y p3 no van a estar ni declaradas ni inicializadas y eso no se arregla ni tan siquiera con las llaves.

Para verlo mejor vamos a refactorizar ligeramente el código:

void func1()
{
  // variables locales
  jugador p((char*)"Edwin", (char*)"Tigres", 20 );
  jugador p2((char*)"Maria", (char*)"Rapidos", 21 );
  jugador p3((char*)"Julio", (char*)"Triunfadores", 30 );
}

void func2()
{
  jugador x1(p); // ¿Que es p?
  x1.mostrar_jugador();
  jugador x2(p2); // ¿Que es p2?
  x2.mostrar_jugador();
  jugador x3(p3); // ¿Que es p3?
  x3.mostrar_jugador();
}

// ...

if( op == 1 )
  func1();
else if( op == 2 )
  func2();

Tu programa tiene que pedir los datos de 3 jugadores en la opción 1... y esos datos deben seguir vivos en sucesivas opciones, luego deberías mover su declaración al nivel del main.

Llegados a este punto tienes un problema y es que el constructor por defecto no está implementado. Esto te impide crear objetos sin valores específicos. Lo puedes solucionar usando punteros o habilitando el constructor por defecto. Lo más sencillo es habilitar el constructor por defecto (considera usar std::string en vez de char* al programar en C++).

class jugador
{
  std::string nombre;

public:
  // C++11 en adelante
  jugador() = default; 

  // Standares antiguos C++
  jugador()
  { }
};

int main(){ // <-- el tipo de retorno es obligatorio en C++
  jugador p, p2, p3;

Aunque, como hemos dicho, también se pueden usar punteros e ignorar el constructor por defecto:

  • opción C++11, C++14, C++17

    int main() {
    
      std::unique_ptr<jugador> p, p1, p2;
    
      // ...
      switch(op)
      {
        case 1:
          // C++11 en adelante
          p.reset(new jugador("Edwin","Tigres",20));
          p2.reset(new jugador("Maria","Rapidos",21));
          p2.reset(new jugador("Julio","Triunfadores",30));
    
          // C++14 o C++17
          p = std::make_unique<jugador>("Edwin","Tigres",20);
          p2 = std::make_unique<jugador>("Maria","Rapidos",21);
          p2 = std::make_unique<jugador>("Julio","Triunfadores",30);
    
          break;
    
        case 2:
        { // iniciamos ambito para declarar variables
          // Hay que comprobar que los punteros estan inicializados
          if( p && p2 && p3 )
          {
            jugador x1(*p); // p es un puntero
            jugador x2(*p2);
            jugador x3(*p3);
    
            x1.mostrar_jugador();
            x2.mostrar_jugador();
            x3.mostrar_jugador();
          }
          else
            std::cout << "No se han introducido datos\n";
    
          break;
        }
      }
    
      // ...
      // No es necesario borrar los punteros, se borran solos
    }
    
  • opcion C++98, C++03 (versiones antiguas)

    int main() {
    
      jugador *p = 0, *p1 = 0, *p2 = 0;
    
      // ...
      switch(op)
      {
        case 1:
          p = new jugador("Edwin","Tigres",20);
          p2 = new jugador("Maria","Rapidos",21);
          p2 = new jugador("Julio","Triunfadores",30);
    
          break;
    
        case 2:
        {
          // Hay que comprobar que los punteros estan inicializados
          if( p && p2 && p3 )
          {
            jugador x1(*p);
            jugador x2(*p2);
            jugador x3(*p3);
    
            x1.mostrar_jugador();
            x2.mostrar_jugador();
            x3.mostrar_jugador();
          }
          else
            std::cout << "No se han introducido datos\n";
    
          break;
        }
      }
    
      // ...
    
      delete p;
      delete p2;
      delete p3;
    }
    

Nota final

Te he comentado que es preferible sustituir char* por std::string. Te lo digo porque el uso que haces de los punteros en C++ es incorrecto:

jugador p((char*)"Edwin", (char*)"Tigres", 20 );
//        ^^^^^^^         ^^^^^^^

Esas dos conversiones te están chivando que estás intentando hacer algo peligroso en C++. Las cadenas escritas a pelo son, por definición, constantes. Modificar una de estas cadenas puede resultar en un comportamiento indefinido de la aplicación (puede no pasar nada o que las cadenas se almacenen en memoria de solo lectura y el Sistema Operativo mate tu aplicación), eso sin contar con que puedes acabar pisando memoria de otras variables:

char* cad = (char*)"a"; // cad apunta a un array de 2 bytes
strcpy(cad,"ABCDE");    // ¿Donde metemos los 3 bytes de mas?

Las soluciones estilo C pasarían entonces por usar memoria dinámica para gestionar las cadenas o usar un array de tamaño fijo (nombre[100]) y pegarte con la gestión de punteros y la copia de cadenas a mano con strcpy y compañía.

Sin embargo la solución C++ pasa por usar la clase std::string:

std::string cadena = "a"; // ok
cadena = "ABCDE"; // ok, la clase se encarga de gestionar la memoria del 
1
  • gracias es una respuesta muy útil a decir verdad es una respuesta muy buena que no esperaba muchas gracias por tus consejos Commented el 21 sept. 2017 a las 22:55

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.