Skip to main content
Commonmark migration
Origen Enlace

Problema.

###Problema. TienesTienes un arreglo estático de tamaño 5 deudores. Al ser el tamaño estático no puedes quitar y poner registros con total libertad si no que debes limitarte a reutilizar las posiciones del arreglo. Esto puede ser problemático si te eliminan posiciones aleatorias y/o no consecutivas del arreglo. ###Solución 1: Usa contenedores stl. Olvida

Solución 1: Usa contenedores stl.

Olvida el arreglo de 5 deudores y usa un contenedor de datos adecuado a la tarea que quieres realizar. En tu caso requieres inserciones y borrados en posiciones aleatorias de tu colección de registros, así que la elección es std::list:

He intentado conservar to código lo máximo posible pero usando funciones y contenedores STL no necesitas limitar los registros a 5. Las inserciones siempre se hacen al final (push_back) y el borrado se puede hacer en cualquier posición sin problemas incluso eliminando más de un registro. ###Solución 2: Guarda un registro de posiciones libres. Si

Solución 2: Guarda un registro de posiciones libres.

Si no quieres olvidarte del arreglo, necesitarás saber en qué posiciones se puede o no pueden escribir datos, para ello usa un arreglo (del mismo tamaño) de posiciones libres, el código podría quedar como:

En esta versión el deudor nunca se borra si no que se marca como "posición disponible", el llevar la cuenta de que posiciones están disponibles es vital para llevar bien los registros porque cabe la posibilidad de que se borren registros alternos. ###Problemas de estilo. Tienes

Problemas de estilo.

Tienes una serie de problemas en tu código que deberías tener en cuenta:

###Problema. Tienes un arreglo estático de tamaño 5 deudores. Al ser el tamaño estático no puedes quitar y poner registros con total libertad si no que debes limitarte a reutilizar las posiciones del arreglo. Esto puede ser problemático si te eliminan posiciones aleatorias y/o no consecutivas del arreglo. ###Solución 1: Usa contenedores stl. Olvida el arreglo de 5 deudores y usa un contenedor de datos adecuado a la tarea que quieres realizar. En tu caso requieres inserciones y borrados en posiciones aleatorias de tu colección de registros, así que la elección es std::list:

He intentado conservar to código lo máximo posible pero usando funciones y contenedores STL no necesitas limitar los registros a 5. Las inserciones siempre se hacen al final (push_back) y el borrado se puede hacer en cualquier posición sin problemas incluso eliminando más de un registro. ###Solución 2: Guarda un registro de posiciones libres. Si no quieres olvidarte del arreglo, necesitarás saber en qué posiciones se puede o no pueden escribir datos, para ello usa un arreglo (del mismo tamaño) de posiciones libres, el código podría quedar como:

En esta versión el deudor nunca se borra si no que se marca como "posición disponible", el llevar la cuenta de que posiciones están disponibles es vital para llevar bien los registros porque cabe la posibilidad de que se borren registros alternos. ###Problemas de estilo. Tienes una serie de problemas en tu código que deberías tener en cuenta:

Problema.

Tienes un arreglo estático de tamaño 5 deudores. Al ser el tamaño estático no puedes quitar y poner registros con total libertad si no que debes limitarte a reutilizar las posiciones del arreglo. Esto puede ser problemático si te eliminan posiciones aleatorias y/o no consecutivas del arreglo.

Solución 1: Usa contenedores stl.

Olvida el arreglo de 5 deudores y usa un contenedor de datos adecuado a la tarea que quieres realizar. En tu caso requieres inserciones y borrados en posiciones aleatorias de tu colección de registros, así que la elección es std::list:

He intentado conservar to código lo máximo posible pero usando funciones y contenedores STL no necesitas limitar los registros a 5. Las inserciones siempre se hacen al final (push_back) y el borrado se puede hacer en cualquier posición sin problemas incluso eliminando más de un registro.

Solución 2: Guarda un registro de posiciones libres.

Si no quieres olvidarte del arreglo, necesitarás saber en qué posiciones se puede o no pueden escribir datos, para ello usa un arreglo (del mismo tamaño) de posiciones libres, el código podría quedar como:

En esta versión el deudor nunca se borra si no que se marca como "posición disponible", el llevar la cuenta de que posiciones están disponibles es vital para llevar bien los registros porque cabe la posibilidad de que se borren registros alternos.

Problemas de estilo.

Tienes una serie de problemas en tu código que deberías tener en cuenta:

Origen Enlace
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86

###Problema. Tienes un arreglo estático de tamaño 5 deudores. Al ser el tamaño estático no puedes quitar y poner registros con total libertad si no que debes limitarte a reutilizar las posiciones del arreglo. Esto puede ser problemático si te eliminan posiciones aleatorias y/o no consecutivas del arreglo. ###Solución 1: Usa contenedores stl. Olvida el arreglo de 5 deudores y usa un contenedor de datos adecuado a la tarea que quieres realizar. En tu caso requieres inserciones y borrados en posiciones aleatorias de tu colección de registros, así que la elección es std::list:

std::list es un contenedor que soporta inserción y borrado rápido de elementos en cualquier parte del contenedor.

El código podría quedar como:

struct deudor{ // Nombre en singular, no en plural.
    int NRR;
    float deuda;
    char nombre[30], domicilio[30], sexo [1];
}; // Ya no es un arreglo

std::list<deudor> deudores; // Contenedor de deudores

void Agregar(){
if (deudores.size() < 5){ // Queda espacio?
    deudor nuevo{}; // Nuevo deudor a anyadir.
    cout<<":::Agregar Deudor:::\n";
    nuevo.NRR = cont + 1;

    cout<<"Nombre: ";    cin>>nuevo.nombre;    cin.ignore();
    cout<<"Domicilio: "; cin>>nuevo.domicilio; cin.ignore();
    cout<<"Sexo(H/M): "; cin>>nuevo.sexo;      cin.ignore();
    cout<<"Deuda: ";     cin>>nuevo.deuda;     cin.ignore();

    ++cont;
    deudores.push_back(nuevo); // <-- Se anyade el nuevo deudor.
    cout<<"\nDeudor registrado correctamente\n\n";
    system("pause");
}
else{
    cout<<"\nNo hay espacio disponible\n\n";
    system ("pause");
  }
}

void Eliminar(){
if (!deudores.empty()){ // Si no esta vacio...
    int dato;

    cout<<"Ingresa el NRR a Eliminar: ";
    cin>>dato;

    // Buscamos en la coleccion...
    auto encontrado = std::find_if(deudores.begin(), deudores.end(),
    [dato](const deudor &d){ // Lambda de busqueda
        return dato == d.NRR;
    });

    if (encontrado != deudores.end()){
        cout<<":::Datos a Eliminar:::\n";
        cout << "========================\n";
        cout<<"NRR: "       <<encontrado->NRR<<endl;
        cout<<"Nombre: "    <<encontrado->nombre<<endl;
        cout<<"Domicilio: " <<encontrado->domicilio<<endl;
        cout<<"Sexo: "      <<encontrado->sexo<<endl;
        cout<<"Deuda: "     <<encontrado->deuda<<endl;
        cout << "========================\n";
        deudores.erase(encontrado); // Borramos el deudor.
        cout << "\nDeudor eliminado correctamente\n\n";

        cont--;
    }
    else{
        cout<<"\nEl deudor ingresado no existe\n\n";
    }
}
else{
  cout<<"\nNo hay deudores registrados\n"<<endl;
}
system ("pause");
}

He intentado conservar to código lo máximo posible pero usando funciones y contenedores STL no necesitas limitar los registros a 5. Las inserciones siempre se hacen al final (push_back) y el borrado se puede hacer en cualquier posición sin problemas incluso eliminando más de un registro. ###Solución 2: Guarda un registro de posiciones libres. Si no quieres olvidarte del arreglo, necesitarás saber en qué posiciones se puede o no pueden escribir datos, para ello usa un arreglo (del mismo tamaño) de posiciones libres, el código podría quedar como:

constexpr unsigned MAX_REGISTROS{5};

struct deudor{ // Nombre en singular, no en plural.
    int NRR;
    float deuda;
    char nombre[30], domicilio[30], sexo [1];
}deudores[MAX_REGISTROS] {};

// Recuerda que posiciones estan disponibles para ser escritas
// inicialmente lo estan todas.
bool posicion_libre[MAX_REGISTROS] {};

void Agregar(){
// Buscamos la primera posicion libre
auto primero_libre = std::find_if(std::begin(posicion_ocupada), std::end(posicion_ocupada),
    [](bool ocupado) { 
        return !ocupado;
    });

    if (primero_libre != std::end(posicion_ocupada)){ // Hay alguna posicion libre?
    // Calculamos la posicion libre
    auto posicion = std::distance(std::begin(posicion_ocupada), primero_libre);
    std::cout << posicion << " esta libre--------------\n";
    cout<<":::Agregar Deudor:::\n";
    // Apuntamos al deudor en la posicion libre
    deudor &nuevo = deudores[posicion];
    nuevo.NRR = cont + 1;

    cout<<"Nombre: ";    cin>>nuevo.nombre;    cin.ignore();
    cout<<"Domicilio: "; cin>>nuevo.domicilio; cin.ignore();
    cout<<"Sexo(H/M): "; cin>>nuevo.sexo;      cin.ignore();
    cout<<"Deuda: ";     cin>>nuevo.deuda;     cin.ignore();

    ++cont;
    // Marcamos la posicion como ocupada
    posicion_ocupada[posicion] = true;
    cout<<"\nDeudor registrado correctamente\n\n";
    system("pause");
}
else{
    cout<<"\nNo hay espacio disponible\n\n";
    system ("pause");
  }
}

void Eliminar()
{
if (cont!=0)
{
    int dato;

    cout<<"Ingresa el NRR a Eliminar: ";
    cin>>dato;

    // Buscamos en la coleccion...
    auto encontrado = std::find_if(std::begin(deudores), std::end(deudores),
    [dato](const deudor &d){ // Lambda de busqueda
        return dato == d.NRR;
    });

    if (encontrado != std::end(deudores))
    {
        // Calculamos la posicion a eliminar
        auto posicion = std::distance(std::begin(deudores), encontrado);
        cout<<":::Datos a Eliminar:::\n";
        cout << "========================\n";
        cout<<"NRR: "       <<encontrado->NRR<<endl;
        cout<<"Nombre: "    <<encontrado->nombre<<endl;
        cout<<"Domicilio: " <<encontrado->domicilio<<endl;
        cout<<"Sexo: "      <<encontrado->sexo<<endl;
        cout<<"Deuda: "     <<encontrado->deuda<<endl;
        cout << "========================\n";
        posicion_ocupada[posicion] = false; // Marcamos la posicion como libre.
        cout << "\nDeudor eliminado correctamente\n\n";

        cont--;
    }
    else{
        cout<<"\nEl deudor ingresado no existe\n\n";
    }
}
else{
        cout<<"\nNo hay deudores registrados\n"<<endl;
    }
}

En esta versión el deudor nunca se borra si no que se marca como "posición disponible", el llevar la cuenta de que posiciones están disponibles es vital para llevar bien los registros porque cabe la posibilidad de que se borren registros alternos. ###Problemas de estilo. Tienes una serie de problemas en tu código que deberías tener en cuenta:

  • No uses el plural para una estructura que contiene datos individuales: Es muy confuso que deudores sea un único deudor mientras que deudor son varios, intercambia los nombres.
  • No uses arreglos de caracteres para guardar textos de longitud variable: No sólo puede ser un desperdicio de espacio (si el usuario se llama Ana desperdicias 27 posiciones de char deudor::nombre[30]) si no que además puede dar lugar a errores en tiempo de ejecución si el nombre es más largo que la longitud del arreglo. Lo mismo para el domicilio. Usa std::string.
  • ¿Un arreglo de tamaño uno? ¿En serio?: Usa un solo char.
  • Usa una constante para definir el tamaño máximo de tu arreglo: De esta manera si necesitas cambiar el tamaño tan sólo deberás modificar la constante no buscar el valor 5 por todo el código.