[SOLUCIONADO] Programa C++ repite codigo

[Aleix_1379]

Hola, estoy haciendo este programa con C++

1. Escribir un programa que almacene en un array los nombres y números de teléfono
de 2 personas. El programa debe leer los datos introducidos por el usuario y
guardarlos en memoria. Después debe ser capaz de buscar el nombre correspondiente
a un número de teléfono y el teléfono correspondiente a una persona. Ambas
opciones deben se accesibles a través de un menú, así como la opción de salir del
programa. El menú debe tener esta forma, más o menos:
a) Buscar por nombre
b) Buscar por número de teléfono
c) Salir
Pulsa una opción:

Código C++:

#include <iostream>
#include <string.h>
 
#define MAX 31
#define MP 2
 
using namespace std;
/*-----------------------------------------------------------------------------------------------------------------*/
struct persona {
    char nom[MAX],tel[MAX];
    persona ();
} t[MP];
/*-----------------------------------------------------------------------------------------------------------------*/
char menu ();
void cercar_per_nom ();
void cercar_per_telefon ();
void mostrar_persona (persona p);
void no_trobat (const char p1[],char p2[]);
/*-----------------------------------------------------------------------------------------------------------------*/
int main()
{
    char x;
    do
    {
        x = menu ();
        switch (x)
        {
        case 'a':
            cercar_per_nom ();
            break;
 
        case 'b':
            cercar_per_telefon ();
            break;
        }
    }
    while (x != 'c');
    return 0;
}
/*-----------------------------------------------------------------------------------------------------------------*/
persona::persona()
{
    int i;
    for (i = 0; i < MP; i++)
    {
        cout << "Introdueix el nom i telefon " << " i == " <<i << " < " << MP << " : ";
        cin >> t[i].nom >> t[i].tel;
    }
}
/*-----------------------------------------------------------------------------------------------------------------*/
char menu ()
{
    char x;
    do
    {
        cout << "a) Cercar per nom\nb) Cercar per numero de telefon\nc) Sortir" << endl;
        cin.get(x);
    }
    while (x < 'a' || x > 'c');
    return x;
}
/*-----------------------------------------------------------------------------------------------------------------*/
void cercar_per_nom ()
{
    int i;
    char nom[MAX];
    cout << "Introdueix el nom que vols cercar: ";
    cin >> nom;
    i = 0;
    while (i < MP && strcmp (nom,t[i].nom) != 0) i++;
    if (i == MP) mostrar_persona (t[i]);
    else no_trobat ("nom",nom);
    cin.get();
}
/*-----------------------------------------------------------------------------------------------------------------*/
void cercar_per_telefon()
{
    int i;
    char tel[MAX];
    cout << "Introdueix el telefon pel que vols cercar: ";
    cin >> tel;
    i = 0;
    while (i < MP && strcmp (tel,t[i].tel) != 0) i++;
    if (i == MP) mostrar_persona (t[i]);
    else no_trobat ("telefon",tel);
    cin.get();
}
/*-----------------------------------------------------------------------------------------------------------------*/
void mostrar_persona (persona p)
{
    cout << "Nom: " << p.nom << " Telefon: " << p.tel;
}
/*-----------------------------------------------------------------------------------------------------------------*/
void no_trobat (const char p1[],char p2[])
{
    cout << "No s'ha trobat cap persona amb el " << p1 << " : " << p2;
}
/*-----------------------------------------------------------------------------------------------------------------*/
 

Primero intento llenar el array con un bucle pero cuando acaba vuelve a empezar.

[vosk]

Esto está mal, es decir está bien pero no funcionará tal como esperas. Para cada declaracion de la estructura 'persona' se ejecuta la llamada a la funcion 'persona()' (el segundo campo de tu struct es una llamada a la funcion); como se da el caso que tienes declarado un array de 2 estructuras pues se ejecuta dos veces la funcion 'persona()' ok?

Código:

#define MP 2

struct persona {
    char nom[MAX],tel[MAX];
    persona ();
} t[MP];

Al hacer t[MP] la aplicacion ejecuta lo siguiente: inicia la carga de t[0], reserva MAX bytes para 'nom' en t[0], reserva MAX bytes para 'tel' en t[0], ejecuta 'persona()', inicia la carga de t[1], reserva MAX bytes para 'nom' en t[1], reserva MAX bytes para 'tel' en t[1] y ejecuta 'persona()'. Si declaras el array con 5 structs veras que la funcion se ejecuta 5 veces.

Lo ideal para eso es llamar a la funcion de inicio desde el main; de momento entras los nombres disponibles manualmente y mas adelante seguramente haras un archivo a modo de agenda. Esta funcion que te carga los datos iniciales no puede estar dentro del struct a menos que el struct sea una clase. Si aun no has trabajado con clases puedes quitar la funcion del struct y llamarla desde el main (o si pones una opcion de recargar):

Código:

typedef struct {
    char nom[MAX];
    char telefon[MAX];
} AGENDA;

AGENDA agenda[MP];


void carrega_agenda() {
    for (int i = 0; i < MP; i++)
    {
        cout << "Introdueix el nom i telefon " << " i == " <<i << " < " << MP << " : ";
        cin >> t[i].nom >> t[i].tel;
    }
}


int main() {
    carrega_agenda();

    do {
        ...
    } while...
    
    return 0;
}

Y ya que estas trabajando en c++ tal vez te interese echar un vistazo (ademas de a las clases) a los vectores, te permitira usar una lista de personas no limitada.

Saludos
vosk

[Aleix_1379]

Gracias, me ha funcionado, el otro problema era que el menú aparecía 2 veces, cuando hacia C esto me pasaba cunado no limpiaba el buffer, en C++ lo he hecho así:

Código C++:

cout << "Nombre y teléfono: ";
cin >> t[i].nom >> t[i].tel;
cin.sync();
 

Si es así, ¿deberia usar cin.sync(); después de cada cin >> ?

Gracias

[vosk]

Es una opcion, pero sync no solo se aplica al buffer de entrada. En c++ puedes limpiar el buffer de entrada igual que en C, o puedes usar la version de c++ (a fin de cuentas es lo mismo: leer hasta que no quede nada):

Código:

void fflush_stdin_c() {
    char c;
    while ((c = getchar()) != EOF && c != '\n');
}

void fflush_stdin_cpp_versio1() {
    char c;
    while((c = cin.get() ) != EOF && c != '\n');
}

void fflush_stdin_cpp_versio2() {
    cin.ignore(numeric_limits<streamsize>::max(), '\n');
}

Si buscas un poco por ahi seguro que encuentras mas variantes para el flush de stdin.

Saludos
vosk

[Aleix_1379]

Gracias ;)

[amchacon]

C++ no necesita limpiar el buffer.

[vosk]

Buena observacion :))