headers, carpetas y menúes c++

[detective_jd]

Hola a todos, quería hacer una consulta sobre c++, antes que nada uso netbeans 8.0.2 y linuz mint 17.1, mi problema es que intento simplificar el código para que no quede demasiado largo, y hasta ahí bien pero cuando ejecuto el menú no me funcionan la parte para acceder a las modificaciones y las bajas, pongo el código:

Header Files:

Persona.h

Código C++:

#ifndef PERSONA_H
#define PERSONA_H
using namespace std;
struct persona{
    int id;
    string nom;
    string ape;
};
#endif  /* PERSONA_H */
 

ManPersonas.h

Código C++:

#include "Persona.h"
#ifndef CONTRPERSONAS_H
#define CONTRPERSONAS_H
using namespace std;    
    struct persona lista[20];
    int maxIdP = 0;
    int lastP = 0;
    int ind = 0;
    int getMaxIdP(){
        return maxIdP+1;
    }
    void upMaxIdP(){
        maxIdP++;
    }
    bool check(persona p){
        for (int i=0; i< lastP; i++){
            if(lista[i].nom == p.nom){
                ind = i;
                return true;
            }
        }
        return false;
    }
    persona getRow(string xnom){
        persona p;
        for (int i=0; i< lastP; i++){
            if(lista[i].nom == xnom){
                p = lista[i];
                break;
            }
        }
        return p;
    }
    persona getRow(int id){
        persona p;
        for (int i=0; i< lastP; i++){
            if(lista[i].id == id){
                p = lista[i];
                break;
            }
        }
        return p;
    }
    bool create(persona p){
        if(check(p)){            
            return false;
        }
        else{
            lista[lastP] = p;
            lastP++;
            upMaxIdP();
            return true;
        }
    }
    bool update(persona p){
        persona p1 = getRow(p.id);
        if(p1.nom != p.nom){
            if(check(p)){
                return false;                
            }            
        }        
        lista[ind] = p;
        ind = 0;
        return true;
    }
    void remove(persona p){
        if(check(p)){
            for(int i=ind; i<lastP-1; i++){
                lista[i] = lista[i+1];
            }
            lastP--;
            ind = 0;
        }
    }
    persona * read(){
        return lista;        
    }
    
#endif  /* CONTRPERSONAS_H */
 

crud_per.h

Código C++:

#include <iostream>
#include "Util.h"
#include "ManPersonas.h"
#ifndef CRUD_PER_H
#define CRUD_PER_H
using namespace std;
void created(){
    int op;
    cout << "\n ¿Desea crear una nueva persona? Aceptar=1/Cancelar=2 : ";
    cin >> op;
    if(op == 1) {
        persona p;
        p.id = getMaxIdP();
        p.nom = addString("Nombre");
        p.ape = addString("Apelido");
        if(create(p)){
            cout << "\n persona creada :)" << endl;            
        }
        else {
            cout << "\n la persona ya existe :(" << endl;
        }                        
    }
    else if(op == 2) {        
        pause(2);
    }
    else {
        cout << "\n error :(" << endl;
    }
}
void updated(){   
    int op;
    string xnom;
    persona p;
    cout << "\n Ingrese nombre: ";
    cin >> xnom;
    p = getRow(xnom);
    cout << "\n Persona: " << p.id << " Nombre: " << p.nom << " Apellido: " << p.ape;
    cout << "\n ¿Desea algún dato? Aceptar=1/Cancelar=2 :";
    cin >> op;
    if(op == 1){
        cout << "\n ¿Desea editar el nombre ("<< p.nom <<") actual? Aceptar=1/Cancelar=2 :";
        cin >> op;
        if(op == 1){
            p.nom = addString("Nombre");
        }
        else {
            pause(2);
        }
        cout << "\n ¿Desea editar el apellido ("<< p.ape <<") actual? Aceptar=1/Cancelar=2 :";
        cin >> op;
        if(op == 1){
            p.ape = addString("Apellido");
        }
        else {
            pause(2);
        }
        if(update(p)){            
            cout << "\n persona editada :)" << endl;            
        }
        else {
            cout << "\n la persona ya existe :(" << endl;
        }
    }
    else if(op == 2) {        
        pause(2);
    }
    else {
        cout << "\n error :(" << endl;
    }
}
void deleted(){
    int op;
    string xnom;
    persona p;
    cout << "\n Ingrese nombre: ";
    cin >> xnom;
    p = getRow(xnom);
    cout << "\n Persona: " << p.id << " Nombre: " << p.nom << " Apellido: " << p.ape;
    cout << "\n ¿Desea eliminar la persona seleccionada? Aceptar=1/Cancelar=2 :";
    cin >> op;
    if(op == 1){
        remove(p);
        cout << "\n persona eliminada :)" << endl;
    }
    else if(op == 2) {        
        pause(2);
    }
    else {
        cout << "\n error :(" << endl;
    }
}
void list(){
    cout << "\n ---Mostrar Personas--- \n";        
    cout << "Persona\t  Nombre\t  Apellido \n";    
    for (int i=0; i< lastP; i++){
        cout << read()[i].id << "\t  " << read()[i].nom << "\t  " << read()[i].ape << "\n";                        
    }
}
#endif  /* CRUD_PER_H */
 

menu_per.h

Código C++:

#include "crud_per.h"
#ifndef MENU_PER_H
#define MENU_PER_H
void menu(){
    int op = 0;
    cout << "\n ---CRUD Personas---\n";
    cout << "\n 1. Crear Persona";
    cout << "\n 2. Editar Persona";
    cout << "\n 3. Borrar Persona";
    cout << "\n 4. Mostrar Personas";
    cout << "\n 5. Salir \n";
    cout << "\n Ingrese Opción: ";
    cin >> op;    
    switch(op){
        case 1:
            created();
            break;  
        case 2:
            updated();
            break; 
        case 3:
            deleted();
            break;     
        case 4:
            list();
            break; 
        case 5:
            cout << "\n Gracias por usarme ;) \n";
            exit(0);
            break;
        default:
            cout << "\n Error :( \n";
            break;    
    }
    menu();
}
#endif  /* MENU_PER_H */
 

Util.h

Código C++:

#ifndef UTIL_H
#define UTIL_H
using namespace std;
bool checkInt(int s){
    return s >= 48 and s <= 57;
}
bool checkStr(string s){
    for(int i=0; i < s.size(); i++){
        if((s[i] >= 65 and s[i]<= 90) or (s[i] >= 90 and s[i]<= 122)){
            return true;
        }
    }
    return false;
}
void pause(int dur) {
    int temp = time(NULL) + dur;
    while(temp > time(NULL));
}
string addString(string type){
    int intento = 1;
    string nom = "";    
    while(!checkStr(nom)){
        cout << "\n intento nro "<< intento << " Ingrese " << type <<": ";
        cin >> nom;
        intento++;
    }
    return nom;
}
#endif  /* UTIL_H */
 

Sources Files:

main.cpp

Código C++:

#include <iostream>
#include <stdlib.h>
#include "menu_per.h"
using namespace std;
int main(int argc, char** argv) {
    menu();
    exit(2);
}
 

Espero sus respuestas y saludos

[eferion]

Código C++:

struct persona lista[20];
    int maxIdP = 0;
    int lastP = 0;
    int ind = 0;
 

¿Variables globales? Evítalas salvo que sea imperativo usarlas... y en un programa de 200 líneas dudo mucho que sea imprescindible tirar de variables globales.

Una forma de eliminar algunas variables globales:

Código C++:

int getMaxIdP(){
  static int maxIdp = 0;
  return ++maxIdP;
}
 

Además, No tiene demasiado sentido que esperes a que el nuevo registro se añada definitamente a la lista para actualizar maxIdp... int, para el uso particular que tienen estos identificadores, tiene un rango lo suficientemente amplio como para poder desaprovechar valores.

Aunque también, puestos a simplificar el código, yo pasaría de los arreglos tipo C y consideraría el uso de contenedores. Como ventaja principal que no te tienes que preocupar del número de elementos en el contenedor. Hablando del número de elementos... ¿Compruebas en algún punto que no se intentan meter más de 20 elementos en la lista?

Código C++:

std::vector<persona> lista;
 

Por cierto, nota que en C++ no hace falta poner "struct" a la hora de declarar elementos de tipo "persona". Aunque compile no es necesario en C++... al no ser necesario podemos decir que no aporta absolutamente nada al código, luego lo mejor es eliminarlo para tener un código lo más limpio posible. Otro detalle respecto a este punto es que en C++, como norma general, las clases y estructuras se suelen nombrar con la primera letra en maýusculas. Es una pequeña pijotada pero ayuda a leer mejor el código.

Código C++:

void menu(){
  // ...
  menu();
}
 

¿De verdad es necesario que menu se llame a sí misma de forma recursiva? ¿No conoces los bucles tipo while o do-while?

Código C++:

void menu(){
  while(true)
  {
    // ...
  }
}
 

Y pasando un poco a la chicha del programa:

Código C++:

persona getRow(std::string xnom){
    persona p;
    for (int i=0; i< lastP; i++){
        if(lista[i].nom == xnom){
            p = lista[i];
            break;
        }
    }
    return p;
}
 
void updated(){  
  // ...
  p = getRow(xnom);
    std::cout << "\n Persona: " << p.id << " Nombre: " << p.nom << " Apellido: " << p.ape;
    std::cout << "\n ¿Desea algún dato? Aceptar=1/Cancelar=2 :";
}
 

¿Por qué asumes que el usuario va a introducir un nombre válido? ¿No puede equivocarse?

Por otro lado... si tu idea es simplificar el código... ¿Por qué no dejas a la STL hacer su trabajo?

Código C++:

// Lo suyo sería devolver un puntero... si el puntero es nulo es que no se ha encontrado el registro.
persona* getRow(std::string xnom){
    persona* toReturn = nullptr; // inicialización de puntero nulo
 
    auto it = std::find_if(std::begin(lista),std::end(lista),
                                     [&xnom](const presona& p)
                                     { return p.nom == xnom; });
    
    if( it != std::end(lista) )
      toReturn = &(*it); // it
    return p;
}
 

Aun así tendrías que lidiar con situaciones como... ¿Qué sucede si el usuario únicamente quiere cambiar el apellido? Ahora mismo el programa te va a decir que la persona ya existe y no va a actualizarlo.

Para borrar elementos de la lista tienes problemas similares que deberías corregir.

Más posibles mejoras:

Código C++:

bool check(const persona& p){
  int num_reg = std::count_if(std::begin(lista),std::end(lista),
                              [&p](const persona& p2)
                              { return p2.nom == p.nom; });
 
  return num_reg > 0;
}
 

Código C++:

bool create(const persona& p){
  bool toReturn = false;
 
  if(!check(p)){
      // Esta línea...
      lista[lastP] = p;
      // ...con std::vector podría quedar tal que
      lista.push_back(p);
 
      lastP++; // Esto con contenedores no es necesario
      upMaxIdP(); // Si haces caso de lo que te he dicho antes esta línea también sobra
      toReturn = true;
  }
 
  return toReturn;
}
 

Y bueno aquí tienes algunas guías para mejorar tu código, el resto, dado que hay que tomar decisiones respecto al diseño, es cosa tuya.

PD.: find_if, count_if forman parte de la librería algorithm.

Un saludo.

[detective_jd]

Hola eferion, gracias por responder, verás hice lo que me dijiste y me marcan errores y no sé xq razón, pongo el código, no sé si puedes pasar por privado tu correo, ya que esto no sólo es para mí y no sé cómo arreglarlo ya que c++ no lo dominio bien.

Saludos

[eferion]

No deberias tener problemas para poner aquí tu código actual para que te sigamos ayudando

[detective_jd]

Hola, gracias por responder, verás te hice caso y adapté lo que me dijiste pero me da error en todos los headers y no sé porque:

ManPersonas.h

Código C++:

#include <list>
#include <algorithm>
#include "Persona.h"
#ifndef CONTRPERSONAS_H
#define CONTRPERSONAS_H
using namespace std;    
    std::list<Persona>lista;
    static int maxIdP = 0;
    int getMaxIdP(){
        return ++maxIdP;
    }
    bool check(const Persona& p){
        int num_reg = std::count_if(lista.begin(),lista.end(),[&p](const Persona& p2){ 
            return p2.nom == p.nom; 
        });
        return num_reg > 0;
    }
    Persona* getRow(std::string xnom){
        Persona* toReturn = NULL; // inicialización de puntero nulo
        auto it = std::find_if(lista.begin,lista.end,[&xnom](const Persona& p){ 
            return p.nom == xnom; 
        });    
        if( it != lista.end())
            toReturn = &(it); // it
        return toReturn;
    }
    Persona* getRow(int xid){
        Persona* toReturn = nullptr; // inicialización de puntero nulo
        auto it = std::find_if(lista.begin,lista.end,[&xid](const Persona& p){ 
            return p.id == xid; 
        });    
        if( it != lista.end())
            toReturn = &(it); // it
        return toReturn;
    }
    bool create(Persona p){
        if(check(p)){            
            return false;
        }
        else{
            lista.push_back(p);
            return true;
        }
    }
    bool update(Persona p){
        /*
        Persona p1 = getRow(p.id);
        if(p1.nom != p.nom){
            if(check(p)){
                return false;                
            }            
        }        
        lista[ind] = p;
        ind = 0;
        return true; */
    }
    void remove(Persona p){
        if(check(p)){
            lista.remove(p);
        }
    }
    std::list<Persona> read(){
        return lista;        
    }
    
#endif  /* CONTRPERSONAS_H */
 

Persona.h

Código C++:

#ifndef PERSONA_H
#define PERSONA_H
using namespace std;
struct Persona{
    int id;
    string nom;
    string ape;
};
#endif  /* PERSONA_H */
 

Util.h

Código C++:

#ifndef UTIL_H
#define UTIL_H
using namespace std;
bool checkInt(int s){
    return s >= 48 and s <= 57;
}
bool checkStr(string s){
    for(int i=0; i < s.size(); i++){
        if((s[i] >= 65 and s[i]<= 90) or (s[i] >= 90 and s[i]<= 122)){
            return true;
        }
    }
    return false;
}
void pause(int dur) {
    int temp = time(NULL) + dur;
    while(temp > time(NULL));
}
string addString(string type){
    int intento = 1;
    string nom = "";    
    while(!checkStr(nom)){
        cout << "\n intento nro "<< intento << " Ingrese " << type <<": ";
        cin >> nom;
        intento++;
    }
    return nom;
}
int addInt(string type){
    int intento = 1;
    int nro;    
    while(!checkInt(nro)){
        cout << "\n intento nro "<< intento << " Ingrese " << type <<": ";
        cin >> nro;
        intento++;
    }
    return nro;
}
#endif  /* UTIL_H */
 

crud_per.h

Código C++:

#include <iostream>
#include "Util.h"
#include "ManPersonas.h"
#ifndef CRUD_PER_H
#define CRUD_PER_H
using namespace std;
void created(){
    int op;
    cout << "\n ¿Desea crear una nueva persona? Aceptar=1/Cancelar=2 : ";
    cin >> op;
    if(op == 1) {
        Persona p;
        p.id = getMaxIdP();
        p.nom = addString("Nombre");
        p.ape = addString("Apelido");
        if(create(p)){
            cout << "\n persona creada :)" << endl;            
        }
        else {
            cout << "\n la persona ya existe :(" << endl;
        }                        
    }
    else if(op == 2) {        
        pause(2);
    }
    else {
        cout << "\n error :(" << endl;
    }
}
void updated(){   
    int op;
    string xnom;
    Persona p;
    xnom = addString("Nombre para la búsqueda");
    p = getRow(xnom);
    cout << "\n Persona: " << p.id << " Nombre: " << p.nom << " Apellido: " << p.ape;
    cout << "\n ¿Desea algún dato? Aceptar=1/Cancelar=2 :";
    cin >> op;
    if(op == 1){
        cout << "\n ¿Desea editar el nombre ("<< p.nom <<") actual? Aceptar=1/Cancelar=2 :";
        cin >> op;
        if(op == 1){
            p.nom = addString("Nombre");
        }
        else {
            pause(2);
        }
        cout << "\n ¿Desea editar el apellido ("<< p.ape <<") actual? Aceptar=1/Cancelar=2 :";
        cin >> op;
        if(op == 1){
            p.ape = addString("Apellido");
        }
        else {
            pause(2);
        }
        if(update(p)){            
            cout << "\n persona editada :)" << endl;            
        }
        else {
            cout << "\n la persona ya existe :(" << endl;
        }
    }
    else if(op == 2) {        
        pause(2);
    }
    else {
        cout << "\n error :(" << endl;
    }
}
void deleted(){
    int op;
    string xnom;
    Persona p;
    xnom = addString("Nombre para la búsqueda");
    p = getRow(xnom);
    cout << "\n Persona: " << p.id << " Nombre: " << p.nom << " Apellido: " << p.ape;
    cout << "\n ¿Desea eliminar la persona seleccionada? Aceptar=1/Cancelar=2 :";
    cin >> op;
    if(op == 1){
        remove(p);
        cout << "\n persona eliminada :)" << endl;
    }
    else if(op == 2) {        
        pause(2);
    }
    else {
        cout << "\n error :(" << endl;
    }
}
void listed(){
    cout << "\n ---Mostrar Personas--- \n";        
    cout << "Persona\t  Nombre\t  Apellido \n"; 
    for(std::list<Persona>::iterator it = read().begin(); it != read().end(); it++) {
        cout << (it)->id << "\t  " << (it)->nom << "\t  " << (it)->ape << "\n";
    }
}
#endif  /* CRUD_PER_H */
 

menu_per.h

Código C++:

#include "Util.h"
#include "crud_per.h"
#ifndef MENU_PER_H
#define MENU_PER_H
void menu(){
    int op;
    cout << "\n ---CRUD Personas---\n";
    cout << "\n 1. Crear Persona";
    cout << "\n 2. Editar Persona";
    cout << "\n 3. Borrar Persona";
    cout << "\n 4. Mostrar Personas";
    cout << "\n 5. Salir \n";
    op = addInt("Opción");
    while (op != 5){
        switch(op){
            case 1:
                created();
                break;  
            case 2:
                updated();
                break; 
            case 3:
                deleted();
                break;     
            case 4:
                listed();
                break; 
            default:
                cout << "\n Error :( \n";
                break;    
        }
    }
    cout << "\n Gracias por usarme ;) \n";
    pause(2);
}
#endif  /* MENU_PER_H */
 

Espero sus respuestas y saludos.

[eferion]

Las funciones, con paréntesis

Código C++:

Persona* getRow(std::string xnom){
  Persona* toReturn = NULL; // inicialización de puntero nulo
  auto it = std::find_if(lista.begin,lista.end,[&xnom](const Persona& p){
    return p.nom == xnom;
  });
  if( it != lista.end())
    toReturn = &(it); // it
  return toReturn;
}
 

begin y end son métodos de lista, luego deben llevar paréntesis.

Los includes son obligatorios

Si usas time para simular una pausa debes añadir el include donde se declara dicha función.

Cuidado con las conversiones

Código C++:

Persona* getRow(std::string xnom){
  Persona* toReturn = NULL; // inicialización de puntero nulo
  auto it = std::find_if(lista.begin,lista.end,[&xnom](const Persona& p){
    return p.nom == xnom;
  });
  if( it != lista.end())
    toReturn = &(it); // it
  return toReturn;
}
 

En el código que yo te puse la penúltima línea no era exactamente así, sino más bien:

Código C++:

toReturn = &(*it);
 

Está claro que si trabajamos con tipos nativos & y * son complementarios y se anulan entre sí, pero cuando trabajamos con clases (con las que se pueden sobrecargar dichos operadores) la cosa cambia:

• *it te devuelve el elemento apuntado por el iterador, en este caso un objeto de tipo Persona por valor.
• &(*it) coge el objeto por valor anterior y obtiene su dirección de memoria, que es lo que almacenas en el puntero toReturn

Cuidado con las variables no inicializadas

Las variables se han de inicializar SIEMPRE, y si no fíjate en el siguiente fragmento:

Código C++:

bool checkInt(int s){
  return s >= 48 and s <= 57;
}
 
int addInt(string type){
  int intento = 1;
  int nro;
  while(!checkInt(nro)){
    cout << "\n intento nro "<< intento << " Ingrese " << type <<": ";
    cin >> nro;
    intento++;
  }
  return nro;
}
 

En teoría, addInt va a pedir al usuario que introduzca un número, y va a repetir la petición hasta que se introduzca un número válido. La realidad dice que nro va a tener inicialmente un valor aleatorio. Si resulta que dicho valor se encuentra en el rango (48,57) va a tomar ese valor como bueno y no va a mostrar el mensaje...

¿Soluciones? Aparte de inicializar la variable se puede usar un do-while para aumentar la protección.

La función tiene otros problemas:

• Dado que nro es de tipo int, cin va a intentar convertir la entrada del usuario a entero. En consecuencia checkInt va a tomar como válidos números mayores de 47 y menores de 58.
• Si se introduce un valor no numérico, p.ej "prueba", cin no va a ser capaz de convertir el dato a tipo int y activará el flag de error que no compruebas... tu programa no se va a comportar como esperas.

La solución pasaría por recoger la entrada del usuario en un string e intentar convertir dicho string a int. Si la conversión es satisfactoria el dato introducido por el usuario se toma como bueno.

Los tipos son muy importantes en C++

Código C++:

Persona* getRow(int xid){
  // ...
}
 
Persona p;
p = getRow(xnom);
 

Si getRow devuelve un dato de tipo Persona* no puedes pretender almacenar ese retorno en una variable de tipo Persona. La gracia de devolver un puntero era para poder identificar cuándo estabamos intentando localizar un registro que no se encuentra en la lista. Puedes ahorrarte el puntero si utilizas alguna convención, por ejemplo devolver un objeto de tipo Persona con 'id=-1' en el caso de registros no válidos.

Prestar atención a los errores

Uno de los errores te está indicando que no se encuentra la sobrecarga del operador de comparación para la clase Persona. En concreto dicho operador se usa aquí:

Código C++:

void remove(Persona p){
  if(check(p)){
    lista.remove(p); // <<<--- AQUI!!!!
  }
}
 

Y se usa porque es la forma en la que remove puede encontrar un elemento equivalente al que le estás pasando como parámetro. Si quieres usar esta función tienes que implementar dicho operador en la clase Persona.

Y con esto el código ya debería compilarte... sigue teniendo algún error más, pero depurando el código deberíais ser capaces de localizarlos.

Un saludo.

[detective_jd]

Hola eferion, gracias por responder, verás, hice lo que me dijiste pero sigue dando error, todo apunta a ManPersonas, te pongo el código y en dónde marcan los errores:

Código C++:

#include <list> //aqui marca error
#include <algorithm>
#include "Persona.h"
#ifndef CONTRPERSONAS_H
#define CONTRPERSONAS_H
using namespace std;    
    std::list<Persona>lista;
    static int maxIdP = 0;
    int getMaxIdP(){
        return ++maxIdP;
    }
    bool check(const Persona& p){
        int num_reg = std::count_if(lista.begin(),lista.end(),[&p](const Persona& p2){ 
            return p2.nom == p.nom; 
        }); //aqui marca error
        return num_reg > 0;
    }
    Persona* getRow(std::string xnom){
        Persona* toReturn = NULL; // inicialización de puntero nulo
        auto it = std::find_if(lista.begin(),lista.end(),[&xnom](const Persona& p){  //aqui marca error
            return p.nom == xnom; 
        });    
        if(it != lista.end()) //aqui marca error
            toReturn = &(it); // it
        return toReturn;
    }
    Persona* getRow(int xid){
        Persona* toReturn = NULL; // inicialización de puntero nulo
        auto it = std::find_if(lista.begin(),lista.end(),[&xid](const Persona& p){ //aqui marca error 
            return p.id == xid; 
        });    
        if( it != lista.end()) //aqui marca error
            toReturn = &(*it); 
        return toReturn;
    }
    bool create(Persona p){
        if(check(p)){            
            return false;
        }
        else{
            lista.push_back(p);
            return true;
        }
    }
    bool update(Persona p){ }
    void remove(Persona p){
        if(check(p)){
            lista.remove(p); //aqui marca error
        }
    }
    std::list<Persona> read(){
        return lista;        
    }   
#endif  /* CONTRPERSONAS_H */
 

Pero cuando ejecuto, los errores que me saltan son estos:

In file included from crud_per.h:3:0,
from menu_per.h:2,
from main.cpp:3:
ManPersonas.h: In function ‘bool check(const Persona&)’:
ManPersonas.h:15:9: warning: lambda expressions only available with -std=c++11 or -std=gnu++11 [enabled by default]
});
^
ManPersonas.h:15:10: error: no matching function for call to ‘count_if(std::list<Persona>::iterator, std::list<Persona>::iterator, check(const Persona&)::__lambda0)’
});
^
ManPersonas.h:15:10: note: candidate is:
In file included from /usr/include/c++/4.8/algorithm:62:0,
from ManPersonas.h:2,
from crud_per.h:3,
from menu_per.h:2,
from main.cpp:3:
/usr/include/c++/4.8/bits/stl_algo.h:4647:5: note: template<class _IIter, class _Predicate> typename std::iterator_traits<_Iterator>::difference_type std::count_if(_IIter, _IIter, _Predicate)
count_if(_InputIterator __first, _InputIterator __last, _Predicate __pred)
^
/usr/include/c++/4.8/bits/stl_algo.h:4647:5: note: template argument deduction/substitution failed:
In file included from crud_per.h:3:0,
from menu_per.h:2,
from main.cpp:3:
ManPersonas.h: In substitution of ‘template<class _IIter, class _Predicate> typename std::iterator_traits<_Iterator>::difference_type std::count_if(_IIter, _IIter, _Predicate) [with _IIter = std::_List_iterator<Persona>; _Predicate = check(const Persona&)::__lambda0]’:
ManPersonas.h:15:10: required from here
ManPersonas.h:15:10: error: template argument for ‘template<class _IIter, class _Predicate> typename std::iterator_traits<_Iterator>::difference_type std::count_if(_IIter, _IIter, _Predicate)’ uses local type ‘check(const Persona&)::__lambda0’
});
^
ManPersonas.h:15:10: error: trying to instantiate ‘template<class _IIter, class _Predicate> typename std::iterator_traits<_Iterator>::difference_type std::count_if(_IIter, _IIter, _Predicate)’
ManPersonas.h: In function ‘Persona* getRow(std::string)’:
ManPersonas.h:20:14: error: ‘it’ does not name a type
auto it = std::find_if(lista.begin(),lista.end(),[&xnom](const Persona& p){
^
ManPersonas.h:22:10: error: expected primary-expression before ‘)’ token
});
^
ManPersonas.h:22:10: error: expected ‘;’ before ‘)’ token
ManPersonas.h:23:12: error: ‘it’ was not declared in this scope
if(it != lista.end())
^
ManPersonas.h: In function ‘Persona* getRow(int)’:
ManPersonas.h:29:14: error: ‘it’ does not name a type
auto it = std::find_if(lista.begin(),lista.end(),[&xid](const Persona& p){
^
ManPersonas.h:31:10: error: expected primary-expression before ‘)’ token
});
^
ManPersonas.h:31:10: error: expected ‘;’ before ‘)’ token
ManPersonas.h:32:13: error: ‘it’ was not declared in this scope
if( it != lista.end())
^
In file included from menu_per.h:2:0,
from main.cpp:3:
crud_per.h: In function ‘void updated()’:
crud_per.h:35:7: error: no match for ‘operator=’ (operand types are ‘Persona’ and ‘Persona*’)
p = getRow(xnom);
^
crud_per.h:35:7: note: candidate is:
In file included from ManPersonas.h:3:0,
from crud_per.h:3,
from menu_per.h:2,
from main.cpp:3:
Persona.h:4:8: note: Persona& Persona::operator=(const Persona&)
struct Persona{
^
Persona.h:4:8: note: no known conversion for argument 1 from ‘Persona*’ to ‘const Persona&’
In file included from menu_per.h:2:0,
from main.cpp:3:
crud_per.h: In function ‘void deleted()’:
crud_per.h:75:7: error: no match for ‘operator=’ (operand types are ‘Persona’ and ‘Persona*’)
p = getRow(xnom);
^
crud_per.h:75:7: note: candidate is:
In file included from ManPersonas.h:3:0,
from crud_per.h:3,
from menu_per.h:2,
from main.cpp:3:
Persona.h:4:8: note: Persona& Persona::operator=(const Persona&)
struct Persona{
^
Persona.h:4:8: note: no known conversion for argument 1 from ‘Persona*’ to ‘const Persona&’
In file included from /usr/include/c++/4.8/list:64:0,
from ManPersonas.h:1,
from crud_per.h:3,
from menu_per.h:2,
from main.cpp:3:
/usr/include/c++/4.8/bits/list.tcc: In instantiation of ‘void std::list<_Tp, _Alloc>::remove(const value_type&) [with _Tp = Persona; _Alloc = std::allocator<Persona>; std::list<_Tp, _Alloc>::value_type = Persona]’:
ManPersonas.h:48:27: required from here
/usr/include/c++/4.8/bits/list.tcc:248:17: error: no match for ‘operator==’ (operand types are ‘Persona’ and ‘const value_type {aka const Persona}’)
if (*__first == __value)
^
/usr/include/c++/4.8/bits/list.tcc:248:17: note: candidates are:
In file included from /usr/include/c++/4.8/iosfwd:40:0,
from /usr/include/c++/4.8/ios:38,
from /usr/include/c++/4.8/ostream:38,
from /usr/include/c++/4.8/iostream:39,
from main.cpp:1:
/usr/include/c++/4.8/bits/postypes.h:216:5: note: template<class _StateT> bool std::operator==(const std::fpos<_StateT>&, const std::fpos<_StateT>&)
operator==(const fpos<_StateT>& __lhs, const fpos<_StateT>& __rhs)
^
Entre otros errores, pero esos son los que me llaman más la atención

¿y ahora? espero sus respuestas y saludos

[eferion]

Existe una herramienta muy útil en este mundo que es google.

Una de las responsabilidades a asumir cuando programas es saber leer los mensajes de error... Al menos la mayoría.

Los errores te están diciendo que el compilador está montando el código conforme al estándar C++99 y que, sin embargo, está encontrando elementos propios del estándar C++11, que para poder compilar el programa tienes que activar el estándar C++11 o C++14 usando los flags correspondientes.

Si no quieres/no puedes hacer uso de dichos estándares en clase (lo cual sería motivo para recriminar al profesor puesto que son estándares más modernos y actuales) tendrás que adaptar el código para que sea compatible con el antiguo estándar.

Un saludo

[detective_jd]

Hola eferion, gracias por responder, verás, estuve viendo en google y parece que tengo que editar en Makefile pero intenté código del páginas y ningún cambio hace que habilite el C++11 o C++14.

¿Me puedes dar una idea?

Espero sus respuestas y saludos.

[eferion]

No puedo estar allí para probar tu caso concreto pero el compilador ya te está dando pistas:

warning: lambda expressions only available with -std=c++11 or -std=gnu++11

Tienes que poner uno de estos dos flags cada vez que compilas un fichero. Tampoco has puesto tu makefile por lo que mucho más no se puede hacer.

[detective_jd]

Mirá hasta intenté ponder esos flags que dicen pero tampoco funciona, te pongo el Makefile:

-------------------------------------------------------------------------------------------------------------------
#
# There exist several targets which are by default empty and which can be
# used for execution of your targets. These targets are usually executed
# before and after some main targets. They are:
#
# .build-pre: called before 'build' target
# .build-post: called after 'build' target
# .clean-pre: called before 'clean' target
# .clean-post: called after 'clean' target
# .clobber-pre: called before 'clobber' target
# .clobber-post: called after 'clobber' target
# .all-pre: called before 'all' target
# .all-post: called after 'all' target
# .help-pre: called before 'help' target
# .help-post: called after 'help' target
#
# Targets beginning with '.' are not intended to be called on their own.
#
# Main targets can be executed directly, and they are:
#
# build build a specific configuration
# clean remove built files from a configuration
# clobber remove all built files
# all build all configurations
# help print help mesage
#
# Targets .build-impl, .clean-impl, .clobber-impl, .all-impl, and
# .help-impl are implemented in nbproject/makefile-impl.mk.
#
# Available make variables:
#
# CND_BASEDIR base directory for relative paths
# CND_DISTDIR default top distribution directory (build artifacts)
# CND_BUILDDIR default top build directory (object files, ...)
# CONF name of current configuration
# CND_PLATFORM_${CONF} platform name (current configuration)
# CND_ARTIFACT_DIR_${CONF} directory of build artifact (current configuration)
# CND_ARTIFACT_NAME_${CONF} name of build artifact (current configuration)
# CND_ARTIFACT_PATH_${CONF} path to build artifact (current configuration)
# CND_PACKAGE_DIR_${CONF} directory of package (current configuration)
# CND_PACKAGE_NAME_${CONF} name of package (current configuration)
# CND_PACKAGE_PATH_${CONF} path to package (current configuration)
#
# NOCDDL

# Environment
MKDIR=mkdir
CP=cp
CCADMIN=CCadmin


# build
build: .build-post

.build-pre:
# Add your pre 'build' code here...

.build-post: .build-impl
# Add your post 'build' code here...


# clean
clean: .clean-post

.clean-pre:
# Add your pre 'clean' code here...

.clean-post: .clean-impl
# Add your post 'clean' code here...


# clobber
clobber: .clobber-post

.clobber-pre:
# Add your pre 'clobber' code here...

.clobber-post: .clobber-impl
# Add your post 'clobber' code here...


# all
all: .all-post

.all-pre:
# Add your pre 'all' code here...

.all-post: .all-impl
# Add your post 'all' code here...


# build tests
build-tests: .build-tests-post

.build-tests-pre:
# Add your pre 'build-tests' code here...

.build-tests-post: .build-tests-impl
# Add your post 'build-tests' code here...


# run tests
test: .test-post

.test-pre: build-tests
# Add your pre 'test' code here...

.test-post: .test-impl
# Add your post 'test' code here...


# help
help: .help-post

.help-pre:
# Add your pre 'help' code here...

.help-post: .help-impl
# Add your post 'help' code here...



# include project implementation makefile
include nbproject/Makefile-impl.mk

# include project make variables
include nbproject/Makefile-variables.mk

Espero sus respuestas y saludos

[eferion]

Ese makefile no lo has hecho tu. No te habrías molestado en hacer un makefile tan personalizable para un trabajo de clase.

¿Estás compilando con code::blocks u otro IDE? Si es así la configuración has de establecerla en el IDE, ya que es él el que se encarga de dar las órdenes al compilador.

[detective_jd]

Hola eferion, gracias por responder, verás pude con el problema de c++11, lo que tenía que hacer era en propiedades del proyecto, luego a c++ compiler y en c++ standard pongo c++11 y solucionado, sólo q en ManPersonas.h muestra pocos errores:

Código C++:

#include <algorithm>
#include <list> // error - In file included from
#include "Persona.h"
#ifndef CONTRPERSONAS_H
#define CONTRPERSONAS_H
using namespace std;    
    std::list<Persona>lista;
    static int maxIdP = 0;
    int getMaxIdP(){
        return ++maxIdP;
    }
    bool check(const Persona& p){
        int num_reg = std::count_if(lista.begin(),lista.end(),[&p](const Persona& p2){ 
            return p2.nom == p.nom; 
        });
        return num_reg > 0;
    }
    Persona* getRow(std::string xnom){
        Persona* toReturn = NULL; // inicialización de puntero nulo
        auto it = std::find_if(lista.begin,lista.end,[&xnom](const Persona& p){
            return p.nom == xnom;
        }); // error - no matching function for call to find_if
        if( it != lista.end())
            toReturn = &(it); // it
        return toReturn;
    }
    Persona* getRow(int xid){
        Persona* toReturn = NULL; // inicialización de puntero nulo
        auto it = std::find_if(lista.begin,lista.end,[&xid](const Persona& p){
            return p.id == xid;
        }); // error - no matching function for call to find_if
        if( it != lista.end())
            toReturn = &(it); // it
        return toReturn;
    }
    bool create(Persona p){
        if(check(p)){            
            return false;
        }
        else{
            lista.push_back(p);
            return true;
        }
    }
    bool update(Persona p){ }
    void remove(Persona p){
        if(check(p)){
            lista.remove(p); // error - in file included from
        }
    }
    std::list<Persona> read(){
        return lista;        
    }   
#endif  /* CONTRPERSONAS_H */
 

Espero sus respuestas y saludos

[eferion]

Iniciado por eferion

Prestar atención a los errores

Uno de los errores te está indicando que no se encuentra la sobrecarga del operador de comparación para la clase Persona. En concreto dicho operador se usa aquí:

Código C++:

void remove(Persona p){
  if(check(p)){
    lista.remove(p); // <<<--- AQUI!!!!
  }
}
 

Y se usa porque es la forma en la que remove puede encontrar un elemento equivalente al que le estás pasando como parámetro. Si quieres usar esta función tienes que implementar dicho operador en la clase Persona.

Y con esto el código ya debería compilarte... sigue teniendo algún error más, pero depurando el código deberíais ser capaces de localizarlos.

Un saludo.

Dicho de otra forma, tienes que implementar

Código C++:

bool operator==(const Persona& p1, const Persona& p2);
 

De tal forma que retorne true cuando ambas instancias sean iguales y false en caso contrario.

Un saludo.

[detective_jd]

Hola eferion, gracias por responder verás usé un remove_if por lo que me dijiste, pero ahora estoy un poco confundido:

Código C++:

#include <algorithm>
#include <list>
#include "Persona.h"
#ifndef CONTRPERSONAS_H
#define CONTRPERSONAS_H
using namespace std;    
    std::list<Persona>lista;
    static int maxIdP = 0;
    int getMaxIdP(){
        return ++maxIdP;
    }
    bool check(const Persona& p){
        int num_reg = std::count_if(lista.begin(),lista.end(),[&p](const Persona& p2){ 
            return p2.nom == p.nom; 
        });
        return num_reg > 0;
    }
    Persona* getRow(std::string xnom){
        Persona* toReturn = NULL; // inicialización de puntero nulo
        auto it = std::find_if(lista.begin,lista.end,[&xnom](const Persona& p){
            return p.nom == xnom;
        }); //esta línea da error - no matching function for call to ....
        if( it != lista.end())
            toReturn = &(it); // it
        return toReturn;
    }
    Persona* getRow(int xid){
        Persona* toReturn = NULL; // inicialización de puntero nulo
        auto it = std::find_if(lista.begin,lista.end,[&xid](const Persona& p){
            return p.id == xid;
        }); //esta línea da error - no matching function for call to ....
        if( it != lista.end())
            toReturn = &(it); // it
        return toReturn;
    }
    bool create(Persona p){
        if(check(p)){            
            return false;
        }
        else{
            lista.push_back(p);
            return true;
        }
    }
    bool update(Persona p){ }
    void deleted(Persona p){
        if(check(p)){
            lista.remove_if(lista.begin(),lista.end(),[&p](const Persona& p2){
                return p.id == p2.id;
            }); //esta línea da error - no matching function for call to ....
        }
    }
    std::list<Persona> read(){
        return lista;        
    }
 

Lo raro es que da error cuando cierra paréntesis y llave, otra pregunta ¿hay otra forma para obtener elementos que no sea usando punteros?

Espero sus respuestas y respuestas.

[eferion]

A ver, lo que tienes que hacer es, simple y llanamente, sobrecargar el operador de comparación para que el contenedor sea capaz de comparar elementos de tipo Persona para encontrar el elemento a eliminar.

La función a usar es remove, no remove_if. Sobrecarga el operador (y que funcione correctamente) y el código debería funcionar.

No hagas experimentos raros. Simplemente necesitas decirle a tu programa cómo se comparan dos elementos de tipo Persona.

Por otro lado, sí que es posible devolver elementos sin usar punteros, puedes devolver los objetos tal cual, pero en ese caso tendrías que pensar en algún mecanismo para identificar "objetos no válidos" (por ejemplo si la función tiene que devolverte los datos de "Pedro" pero no hay ningún registro con ese nombre). Una opción puede ser asignar un id negativo o que el nombre esté en blanco... eso ya es decisión tuya.

Un saludo

[detective_jd]

Hola eferion ¿cómo estás?, gracias por responder, tenías razón con el remove me costó darme cuenta lo que querías decirme, hice lo que me dijiste y ya no da error el remove

Persona.h

Código C++:

#ifndef PERSONA_H
#define PERSONA_H
using namespace std;
struct Persona{
    int id;
    string nom;
    string ape;
    bool operator == (const Persona &p)
    {
        return p.id == id;
    }
};
#endif  /* PERSONA_H */
 

ManPersonas.h

Código C++:

#include <list>
#include "Persona.h"
#ifndef MANPERSONAS_H
#define MANPERSONAS_H
using namespace std;    
    std::list<Persona>lista;
    static int maxIdP = 0;
    int getMaxIdP(){
        return ++maxIdP;
    }
    bool check(Persona p){
        for(Persona per : lista){
            if(per.id == p.id){
                return true;
            }
        }
        return false;
    }
    Persona getRow(std::string xnom){
        for(Persona p : lista){
            if(p.nom == xnom){
                return p;
            }
        }
        return NULL; // acá me da error, pero cómo le puedo decir que es nulo cuando no trato con punteros??
    }
    Persona getRow(int xid){
        for(Persona p : lista){
            if(p.id == xid){
                return p;
            }
        }
        return NULL; // acá me da error, pero cómo le puedo decir que es nulo cuando no trato con punteros??
    }
    bool create(Persona p){
        if(check(p)){            
            return false;
        }
        else{
            lista.push_back(p);
            return true;
        }
    }
    bool update(Persona p){ 
        
    }
    void deleted(Persona p){
        lista.remove(p);        
    }
    std::list<Persona> read(){
        return lista;        
    }   
#endif  /* MANPERSONAS_H */
 

Los null me están dando problemas ahora, me queda eso hacer el update para las modificaciones, buscador y paginación, pero los null ahora son mi prioridad.

Espero sus respuestas y saludos.

[eferion]

Deberías leerte las respuestas al completo... no solo intuir lo que pone, sino comprender lo que realmente dicen las respuestas.

Iniciado por eferion

Por otro lado, sí que es posible devolver elementos sin usar punteros, puedes devolver los objetos tal cual, pero en ese caso tendrías que pensar en algún mecanismo para identificar "objetos no válidos" (por ejemplo si la función tiene que devolverte los datos de "Pedro" pero no hay ningún registro con ese nombre). Una opción puede ser asignar un id negativo o que el nombre esté en blanco... eso ya es decisión tuya.

El caso es que tienes que definir una característica que identifique objetos válidos de objetos no válidos y aplicar esa regla en todo tu programa.

[detective_jd]

Hola eferion, al final no me da más error con los getRow, lamentablemente tuve que usar los punteros para eso:

Código C++:

#include <list>
#include "Persona.h"
#ifndef MANPERSONAS_H
#define MANPERSONAS_H
using namespace std;    
    std::list<Persona*>lista;
    static int maxIdP = 0;
    int getMaxIdP(){
        return ++maxIdP;
    }
    bool check(Persona* p){
        for(Persona* per : lista){
            if(per->id == p->id){
                return true;
            }
        }
        return false;
    }
    Persona* getRow(std::string xnom){
        for(Persona* p : lista){
            if(p->nom == xnom){
                return p;
            }
        }
        return NULL;
    }
    Persona* getRow(int xid){
        for(Persona* p : lista){
            if(p->id == xid){
                return p;
            }
        }
        return NULL;
    }
    bool create(Persona* p){
        if(check(p)){            
            return false;
        }
        else{
            lista.push_back(p);
            return true;
        }
    }
    bool update(Persona* p){ 
        
    }
    void deleted(Persona* p){
        lista.remove(p);        
    }
    std::list<Persona*> read(){
        return lista;        
    }   
#endif  /* MANPERSONAS_H */
 

Pero tengo ahora 2 problemas y es que el checkInt y el addInt no me andan ya q lo que quiero es que me validen de si lo que ingreso es númerico o no:

Código C++:

#include <ctime>
#ifndef UTIL_H
#define UTIL_H
using namespace std;
bool checkInt(int s){
    return isdigit(s);
}
bool checkStr(string s){
    for(int i=0; i < s.size(); i++){
        if((s[i] >= 65 and s[i]<= 90) or (s[i] >= 90 and s[i]<= 122)){
            return true;
        }
    }
    return false;
}
void pause(int dur) {
    int temp = time(NULL) + dur;
    while(temp > time(NULL));
}
string addString(string type){
    int intento = 1;
    string nom = "";    
    while(!checkStr(nom)){
        cout << "\n intento nro "<< intento << " Ingrese " << type <<": ";
        cin >> nom;
        intento++;
    }
    return nom;
}
int addInt(string type){
    int intento = 1;
    int nro =0;    
    while(!checkInt(nro)){
        cout << "\n intento nro "<< intento << " Ingrese " << type <<": ";
        cin >> nro;
        intento++;
    }
    return nro;
}
#endif  /* UTIL_H */
 

Por otro lado cuando quiero agregar un nuevo elemento a mi lista al crear me da el error de: Segmentation Failed.

¿sugerencias?

Espero sus respuestas y saludos

[Instru]

Piensa. Que pasa si a create le pasas un puntero invalido?
Cuando se ejecute p->id que crees que suceda?

Por otro lado. Al decidir manejar punteros directamente, aunque uses list tienes que hacerte cargo de devolver la memoria de los objetos. El destructor de list va a destruir los punteros no el contendio. Es una sugerencia.


Saludos

[detective_jd]

Hola eferion, resolví el problema pero tuve que instanciar el objeto:

te pongo sólo el código del create del crud

Código C++:

void created(){
    int op;
    cout << "\n ¿Desea crear una nueva persona? Aceptar=1/Cancelar=2 : ";
    cin >> op;
    if(op == 1) {
        Persona *p = new Persona(); // así me anda
        p->id = getMaxIdP();
        p->nom = addString("Nombre");
        p->ape = addString("Apelido");
        if(create(p)){
            cout << "\n persona creada :)" << endl;            
        }
        else {
            cout << "\n la persona ya existe :(" << endl;
        }                        
    }
    else if(op == 2) {        
        pause(2);
    }
    else {
        cout << "\n error :(" << endl;
    }
}
 

pero el check que me asegura de si la persona existe por medio el nombre no funciona:

ManPersonas (parte del check y create)

Código C++:

bool check(Persona* p){        
        for(Persona* per : lista){
            if(per->nom == p->nom){
                return true;
            }
        }
        return false;         
    }
bool create(Persona* p){
        if(check(p)){            
            return false;
        }
        else{
            lista.push_back(p);
            return true;
        }
    }
 

Pensar que en java y php no tengo los problemas que estoy teniendo en c++ y me da bronca, que sugerencia me das con el check y con la validación de numeros????

Espero sus respuestas y saludos

[eferion]

Código C++:

bool check(Persona* p){        
    for(Persona* per : lista){
        if(per->nom == p->nom){
            return true;
        }
    }
    return false;         
}
 

Si esa función recibe un puntero lo suyo es asegurarse de que el puntero es válido antes de hacer ninguna operación sobre el... salvo que quieras arriesgarte a que tu programa sufra una violación de segmento.

Código C++:

bool check(Persona* p){
    if( nullptr != p ){
      for(Persona* per : lista){
          if(per->nom == p->nom){
              return true;
          }
      }
    }
    return false;         
}
 

Y esta norma es aplicable para toda función que reciba un puntero. Si la función nunca va a recibir un puntero no válido entonces lo suyo es que reciba una referencia o una referencia constante (dependiendo de si el objeto puede ser modificado dentro de la función o no).

Código C++:

bool check(Persona& p);
 
bool check(const Persona& p);
 

Iniciado por detective_jd

Pensar que en java y php no tengo los problemas que estoy teniendo en c++ y me da bronca

Java y PHP manejan únicamente punteros solo que en vez de manejarlos con las manos descubiertas usan un recubrimiento especial. El mismo efecto puedes conseguir en C++ si usas smart pointers (shared_ptr, unique_ptr y weak_ptr). En concreto podríamos decir que tanto JAVA como PHP usan un pariente lejano de shared_ptr.

Iniciado por detective_jd

que sugerencia me das con el check y con la validación de numeros????

Lo que te comenté anteriormente. Lee una cadena, después verifica que la cadena únicamente contiene números y finalmente convierte dicha cadena en un entero. Si el proceso falla (lo cual únicamente debería pasar en la verificación) entonces le vuelves a pedir el dato al usuario.

No hay que olvidar que C++ es un lenguaje multiusos no especializado en nada en particular. Además nació en una época muy anterior a la de los lenguajes de moda que ya vienen con librerías tremendamente completas. En C++ cosas tan tontas como crear una interfaz mínimamente seria en una consola puede obligarte a hacer más malabares que en el circo... pero gran parte de la culpa la tiene, yo creo, el momento en el que nació (y, por supuesto, que no tiene ninguna gran empresa detrás metiendo pasta como si no hubiera un mañana).

Un saludo.

[detective_jd]

Hola eferion, funcionó lo que me dijiste:

Con respecto al ManPersonas.h finalmente me anduvo la validación, tuve que limpiar y generar de nuevo el código, estoy teniendo problemas con el update porque no sé cómo cambiar el elemento cambiado a la lista, así me quedó, pongo la parte en lo que respecta al create y update:

Código C++:

bool check(Persona* p){ 
        if(p != nullptr){
            for(Persona* per : lista){
                if(per->nom == p->nom){
                    return true;
                }
            }
        }
        return false;         
    }
    Persona* getRow(int xid){
        for(Persona* p : lista){
            if(p->id == xid){
                return p;
            }
        }
        return NULL;
    }
    bool create(Persona* p){
        if(check(p)){            
            return false;
        }
        else{
            lista.push_back(p);
            return true;
        }
    }
    bool update(Persona* p){ 
        Persona* per = getRow(p->id);
        if(per->nom != p->nom){
            if(check(p)){
                return false;
            }
        }
        //pero acá no sé cómo editar el elemento en la lista
        return true;
    }
 

Por otro lado te hice caso y pude hacer validación con respecto al numero, pero tanto para validar strings como enteros sucede que si ingreso en ambos una letra y un numero no me funciona la validación, no sé q hacer, te pongo el código de los check y add:

Código C++:

bool checkInt(string s){
    for(int i=0; i < s.size(); i++){
        if(s[i] >= 48 and s[i]<= 57) {
            return true;
        }
    }
    return false;
}
bool checkStr(string s){
    for(int i=0; i < s.size(); i++){
        if((s[i] >= 65 and s[i]<= 90) or (s[i] >= 90 and s[i]<= 122)){
            return true;
        }
    }
    return false;
}
string addString(string type){
    int intento = 1;
    string nom = "";    
    while(!checkStr(nom)){
        cout << "\n intento nro "<< intento << " Ingrese " << type <<": ";
        cin >> nom;
        intento++;
    }
    return nom;
}
 
int addInt(string type){
    int intento = 1;    
    string numero;
    while(!checkInt(numero)){
        cout << "\n intento nro "<< intento << " Ingrese " << type <<": ";
        cin >> numero;
        intento++;
    }
    return std::stoi(numero);
}
 

Agradezco tu ayuda y sugerencias, gracias a tí estoy entiendo un poco más c++.

Espero sus respuestas y saludos

[Instru]

Debo decir que por un momento me sentí ignorado. Gracias eferion por mencionar la recomendación nuevamente.
Y ahora haciendo eco a su recomendación. En vez de pasar punteros como parámetros, deberías usar referencias. Con esto evitarías algunos problemas.

Aunque ignores la recomendación, la vuelvo a hacer. Piensa en a donde van a parar todas las instancias que creaste con new. Espero que las borres y no estes cultivando un mal hábito (leaks de memoria). Creeme, estas cosas, aunque no lo parezcan, son cruciales a la hora de trabajar en sistemas mayores....

[eferion]

A este paso nos vais a tener que citar en la memoria de vuestro ejercicio. Deberías también invertir tiempo en investigar por tu cuenta los problemas ya que estamos entrando en una dinámica peligrosa: pruebo lo que me dicen, compilo, falla, reporto los errores y a la cama.

Deberías aprender a usar el depurador de código para localizar errores en tiempo de ejecución. Es una herramienta imprescindible.

Un par de pistas a tu problema:

• ¿Qué sucede con el ciclo de ejecución de un bucle si aparece un return?
• Cuando calidad un dato se presupone que la validación no sabes si es correcta hasta que no has hecho TODAS las comprobaciones... Mientras tanto lo máximo que puedes llegar a tener claro es que la validación no es correcta... Tu lo estás intentando hacer al revés.

Un saludo.

[detective_jd]

Hola ¿cómo están?, gracias por responder, créanme que si no me mareara tanto los punteros y los valores por referencia no pediría ayuda, además hice lo que me dijeron y me dan más errores, cómo dije anteriormente tanto en java como php no tengo este tipo de problemas pero con esto de los punteros y los parámetros por referencia sí, pongo el código:

Código C++:

#include <list>
#include "Persona.h"
#ifndef MANPERSONAS_H
#define MANPERSONAS_H
using namespace std;    
    std::list<Persona*>lista;
    static int maxIdP = 0;
    int getMaxIdP(){
        return ++maxIdP;
    }
    bool check(Persona& p){ 
        if(p != nullptr){  // acá me da error por el objeto pasado por parámetro
            for(Persona* per : lista){
                if(per->nom == p.nom){
                    return true;
                }
            }
        }
        return false;         
    }
    Persona* getRow(std::string xnom){
        for(Persona* p : lista){
            if(p->nom == xnom){
                return p;
            }
        }
        return NULL;
    }
    Persona* getRow(int xid){
        for(Persona* p : lista){
            if(p->id == xid){
                return p;
            }
        }
        return NULL;
    }
    bool create(Persona& p){  // acá me da error por el objeto pasado por parámetro
        if(check(p)){            
            return false;
        }
        else{
            lista.push_back(p);  // acá me da error por el objeto pasado por parámetro
            return true;
        }
    }
    bool update(Persona& p){   // acá me da error por el objeto pasado por parámetro
        Persona* per = getRow(p.id);
        if(per->nom != &p.nom){  // acá me da error por mal uso de lo que me mencionan
            if(check(p)){
                return false;
            }
        }
        return true;
    }
    void deleted(Persona& p){
        lista.remove(p);  // acá me da error por el objeto pasado por parámetro      
    }
    std::list<Persona*> read(){
        return lista;        
    }   
#endif  /* MANPERSONAS_H */
 

Tampoco es la idea que me hagan la tarea, ya que esto es para mi primo que está haciendo ingeniería en electrotecnia, y me pide ayuda........
Lo de las validaciones lo tendré q ver bien cómo arreglármelas..

Espero sus respuestas y saludos.

[eferion]

Iniciado por detective_jd

Tampoco es la idea que me hagan la tarea, ya que esto es para mi primo que está haciendo ingeniería en electrotecnia, y me pide ayuda..

Lo único que te puedo decir a este respecto es que le haces un flaco favor... Con esto le estás haciendo los deberes... Pero piensa que al examen se va a enfrentar el solito.

Lo suyo en tal caso sería que fuese él el que se pasase por aquí a preguntar dudas y el que se rompiese los cuernos sacando el programa adelante.

Lo que está sucediendo ahora es que tu estás aprendiendo el C++ que él necesita para pasar la asignatura y no tengo muy claro que ese conocimiento se le vaya a trasvasar a él.

Lo mismo estoy equivocado, pero tengo casos más que suficientes para pensar así.

Un saludo.

[Instru]

Iniciado por detective_jd

Hola ¿cómo están?, gracias por responder, créanme que si no me mareara tanto los punteros y los valores por referencia no pediría ayuda, además hice lo que me dijeron y me dan más errores, cómo dije anteriormente tanto en java como php no tengo este tipo de problemas pero con esto de los punteros y los parámetros por referencia sí, pongo el código:

Código C++:

#include <list>
#include "Persona.h"
#ifndef MANPERSONAS_H
#define MANPERSONAS_H
using namespace std;    
    std::list<Persona*>lista;
    static int maxIdP = 0;
    int getMaxIdP(){
        return ++maxIdP;
    }
    bool check(Persona& p){ 
        if(p != nullptr){  // acá me da error por el objeto pasado por parámetro
            for(Persona* per : lista){
                if(per->nom == p.nom){
                    return true;
                }
            }
        }
        return false;         
    }
    Persona* getRow(std::string xnom){
        for(Persona* p : lista){
            if(p->nom == xnom){
                return p;
            }
        }
        return NULL;
    }
    Persona* getRow(int xid){
        for(Persona* p : lista){
            if(p->id == xid){
                return p;
            }
        }
        return NULL;
    }
    bool create(Persona& p){  // acá me da error por el objeto pasado por parámetro
        if(check(p)){            
            return false;
        }
        else{
            lista.push_back(p);  // acá me da error por el objeto pasado por parámetro
            return true;
        }
    }
    bool update(Persona& p){   // acá me da error por el objeto pasado por parámetro
        Persona* per = getRow(p.id);
        if(per->nom != &p.nom){  // acá me da error por mal uso de lo que me mencionan
            if(check(p)){
                return false;
            }
        }
        return true;
    }
    void deleted(Persona& p){
        lista.remove(p);  // acá me da error por el objeto pasado por parámetro      
    }
    std::list<Persona*> read(){
        return lista;        
    }   
#endif  /* MANPERSONAS_H */
 

Tampoco es la idea que me hagan la tarea, ya que esto es para mi primo que está haciendo ingeniería en electrotecnia, y me pide ayuda........
Lo de las validaciones lo tendré q ver bien cómo arreglármelas..

Espero sus respuestas y saludos.

Las referencias técnicamente nunca son nulas. Por lo tanto ese if ya no es necesario, incluso el compilador da error, el cual deberías leer.
Por otro lado, cuando usas referencias, no usas el operador -> que sólo aplica a punteros, usas el operador .(punto).

El push_back te da error porque creaste una lista de punteros, no de objetos. Quitale el * a la lista. Aquí sería interesante que tu o tu primo investigaran un poco sobre las semánticas de copiado y de movido.
También dentro de los for, no obtengas un puntero, obten una referencia al objeto. C++11 hace algo de magia y automáticamente usa semántica de movimiento, lo cual lo hace algo mas eficiente.
Hay un patrón de diseño llamada Null Object (objeto nulo). Revísenlo para ver el caso donde no se encuentre la persona. De hecho eferion ya te dijo masomenos como funciona.
Te recomiendo que uses const en los parametros que no modificas, que me parece son todos. Igual, eso ayuda al compilador a crear mejor código máquina.

Finalmente, ten en cuenta lo que menciona eferion. Es bueno ayudarte (aunque me ignoren ) y que termines tu aprendiendo. Que mal por tu primo, porque el mundo que viene exige soltura con la programación al igual que en el habla. En un futuro no muy lejano, no saber programar lo básica será lo mismo que ser analfabeta.

[detective_jd]

Hola eferion y Instru, gracias por responder, con respecto a mi primo ni empezaron a programar en la utu porque recién empezaron las clases, y van a demorar bastante como para empezar a programar.
Por otro lado, estoy en que me rindo Instru hice lo que me dijiste, y hasta me puse a investigar lo de Null Object, y lo que he encontrado es con java, de c++ encontré más o menos esto:

http://stackoverflow.com/questions/2...sult-not-found

Lo malo es que casi todos te dicen el uso de punteros, además con tengo serios problemas con el update, con lo de null tuve que poner excepciones pero cuando ingreso un nombre que no es, termina la app:

ManPersonas.h

Código C++:

#include <list>
#include <algorithm>
#include "Persona.h"
#include "Util.h"
#ifndef MANPERSONAS_H
#define MANPERSONAS_H
using namespace std;    
    std::list<Persona>lista;
    static int maxIdP = 0;
    int getMaxIdP(){
        return ++maxIdP;
    }
    bool check(const Persona& p){ 
        for(Persona per : lista){
            if(per.nom == p.nom){
                return true;
            }
        }        
        return false;         
    }
    Persona getRow(std::string xnom){
        for(Persona p : lista){
            if(p.nom == xnom){
                return p;
            }
        }
        throw "no existe";
    }
    Persona getRow(int xid){
        for(Persona p : lista){
            if(p.id == xid){
                return p;
            }
        }
        throw "no existe";
    }
    bool create(const Persona& p){
        if(check(p)){            
            return false;
        }
        else{
            lista.push_back(p);
            return true;
        }
    }
    bool update(const Persona& p){ 
        Persona per = getRow(p.id);
        std::list<Persona>::iterator findIter = find(lista.begin(), lista.end(), p);
        if(per.nom != p.nom){
            if(check(p)){
                return false;
            }
        }
        lista.remove(per);
        lista.insert(findIter,p); //asi con el remove y insert me da este error: Segmentation fault; real time: 21s; user: 0ms; system: 4s
        return true;
    }
    void deleted(const Persona& p){
        lista.remove(p);        
    }
    std::list<Persona> read(){
        return lista;        
    }   
#endif  /* MANPERSONAS_H */
 

Pero y: ¿ahora que?

espero sus respuestas y saludos

[Instru]

Se ve mucho mejor el código.

En el update hay varias cosas.

En C++11 en adelante, puedes usar auto como tipo de variable, siempre y cuando el valor que le asignes identifique claramente el tipo. En este caso te puedes beneficiar:

Código:

auto findIter = find(lista.begin(), lista.end(), p);

Ahora ten en cuenta que si p no esta en la lista, entonces finIter == lista.end().
No te parece que este código no tiene mucho sentido?

Código:

if(per.nom != p.nom){
            if(check(p)){
                return false;
            }
        }

Si nom no es igual, checamos que sea igual??????
Sospecho que te da error, por eso mismo. Al parecer estas intentando asignar algo en lista.end() la cual es una localidad invalida. Revisa esa parte.

Creo que lo de null object encontraste información que no es. Yo me refería al patron de diseño. En resumen es que tengas un campo o "algo" dentro de la clase o estructura Persona que identifique si el objeto es válido. Entonces en vez de regresar Null, o lanzar una excepción, regresas un objeto tipo Persona pero con el campo "valido" como falso. Aclaro, esto es un ejemplo.

Código:

return Persona(false);

Es sólo un ejemplo, debes mejorarlo.