[C] Rubrica

[-hide-]

Ciao ragazzi.
Sto, come di consueto, iniziando ad uscire pazzo.
A tratti la funzione visualizza funziona ed altre volte no. Mi sapete dare una mano?
Vi è anche un altro bug. Alla visualizzazione del menu in uscita da una funzione non prende l'input. Che succede?!

Altre cose sono ancora in lavorazione

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define ELEM 100
#define NCHAR 20

// Definizione della struttura
typedef struct {
        char nome[NCHAR];
        char cognome[NCHAR];
        char telefono[NCHAR];
        char citta[NCHAR];
        } account;
        
// Variabili globali
account rubrica[ELEM];
int num_account = 0;

// Dichiarazione delle funzioni
void InsContatto (char nom[], char cogn[], char tel[], char cit[]);
void ModContatto (char nom[], char cogn[], char tel[], char cit[]);
void Cancella (char nom[], char cogn[]);
void Visualizza (char nom[], char cogn[]);

main() 
{ //OPEN main
     char scelta; /* per inserire la scelta del menù */
     char n[NCHAR]; /* per inserire il nome */ 
     char m[NCHAR]; /* per inserire il cognome */
     char c[NCHAR]; /* per inserire la città */
     char t[15]; /* per inserire il numero di telefono */
     int i;
     for (;scelta!= '0';)
     { //OPEN for
          printf ("\t  INDEX\n");
          printf ("***************************\n");
          printf ("* 1 - Inserisci Contatto  *\n");
          printf ("* 2 - Modifica Contatto   *\n");
          printf ("* 3 - Cancella Contatto   *\n");
          printf ("* 4 - Visualizza Contatto *\n");
          printf ("*                         *\n");
          printf ("* 0 - Esci dal programma  *\n");
          printf ("***************************\n\n");
          printf ("Digita: ");
          scanf ("%c", &scelta);
          while (getchar() != '\n'); // Evita che scanf ripete due volte il menù
          switch (scelta) 
          { //OPEN switch
                 case '0': // Exit
                           printf ("\n\t\tIl programmatore ti saluta\n");
                           printf ("\t\tGrazie per la scelta\n\n");
                           break;
                 case '1': // Inserisci contatto
                           printf ("\nNome: ");
                           scanf ("%s", n);
                           printf ("Cognome: ");
                           scanf ("%s", m);
                           printf ("Telefono: ");
                           scanf ("%s", t);
                           printf ("Citta': ");
                           scanf ("%s", c);
                           printf ("\n");
                           InsContatto(n,m,t,c);
                           break;
                 case '2': // Modifica Contatto -->WORK IN PROGRESS
                           printf ("\nChi vuoi modificare?\n");
                           printf ("Nome: ");
                           scanf ("%s",n);
                           printf ("Cognome: ");
                           break;
                 case '3': // Cancella
                           printf ("\nNome: ");
                           scanf ("%s",n);
                           printf ("Cognome: ");
                           scanf ("%s", m);
                           Cancella (n,m);
                           break;
                 case '4': // Visualizza
                           printf ("\nNome: ");
                           scanf ("%s",n);
                           printf ("Cognome: ");
                           scanf ("%s", m);
                           Visualizza (n,m);
                           break;           
          } //CLOSED switch
    } //CLOSED for
    system ("pause");
} //CLOSED main

// Definizione delle funzioni
void InsContatto (char nom[], char cogn[], char tel[], char cit[])
{ //OPEN InsContatto
     int i;
     if (num_account == ELEM)
         printf ("Hai raggiunto il limite di contatti offerto\n\n");
         return;
     for (i=0; i<=num_account; i++)
         if (strcmp (nom,rubrica[i].nome)==0)
         if (strcmp (cogn,rubrica[i].cognome)==0)
            printf ("Nome e Cognome utente esistono in memoria\n\n");
            return;
     strcpy (rubrica[num_account].nome,nom);
     strcpy (rubrica[num_account].cognome,cogn);
     strcpy (rubrica[num_account].telefono,tel);  
     strcpy (rubrica[num_account].citta,cit);
     num_account++;
} //CLOSED InsContatto

void ModContatto (char nom[], char cogn[], char tel[], char cit[])
//WORK IN PROGRESS
{ //OPEN ModContatto 
     Cancella (nom,cogn);
     InsContatto (nom,cogn,tel,cit);
     return;
} //CLOSED ModContatto

void Cancella (char nom[], char cogn[])
{ //OPEN cancella
     int i;
     Visualizza (nom,cogn);
     rubrica[i].nome[0] ='\0';
     rubrica[i].cognome[0] ='\0';
     rubrica[i].telefono[0] ='\0';
     rubrica[i].citta[0] ='\0';
     return;
} //CLOSED Cancella
 
void Visualizza (char nom[], char cogn[])
{ //OPEN Visualizza
     int i;
     for (i=0; i<=num_account; i++)
         if (strcmp (nom,rubrica[i].nome)==0)
         if (strcmp (cogn,rubrica[i].cognome)==0)
         {
            printf ("Il contatto scelto è: \n");
            printf ("\n%s %s\n", rubrica[i].nome, rubrica[i].cognome);
            printf ("%s\n%s", rubrica[i].telefono, rubrica[i].citta);
            printf ("\n\n");
            }
         else
         {
            printf ("Il contatto non esiste");
            printf ("\n\n");  
         }
         return;
} //CLOSED Visualizza

[DanieleC88]

Ci sono diversi errori anche abbastanza gravi direi.
Ad esempio:

void InsContatto (char nom[], char cogn[], char tel[], char cit[])
{ //OPEN InsContatto
     int i;
     if (num_account == ELEM)
         printf ("Hai raggiunto il limite di contatti offerto\n\n");
         return;
     for (i=0; i<=num_account; i++)
         if (strcmp (nom,rubrica[i].nome)==0)
         if (strcmp (cogn,rubrica[i].cognome)==0)
            printf ("Nome e Cognome utente esistono in memoria\n\n");
            return;
     strcpy (rubrica[num_account].nome,nom);
     strcpy (rubrica[num_account].cognome,cogn);
     strcpy (rubrica[num_account].telefono,tel);  
     strcpy (rubrica[num_account].citta,cit);
     num_account++;
} //CLOSED InsContatto

Questa funzione terminerà sempre alla quarta riga, senza inserire niente; abituati ad usare parentesi graffe di apertura e chiusura per ogni blocco di codice che scrivi.

In più gli if annidati li potevi sostituire con un singolo if, testando più condizioni alla volta con l'operatore AND (&&).

Comincia a correggerti queste (sono ripetute in un po' tutto il programma, cerca e distruggi ), poi vediamo il resto.

[-hide-]

Ho risolto tutti i miei problemi.

Questo è il codice ora da migliorare.

                        /***************************\
                         ***************************
                         *****Lorenzo Carnevale*****
                         *****     RUBRICA     *****
                         ***************************
                         ***************************

  Programma per al gestione di una rubrica contatti.                         
  Memorizzare NOME (20caratteri) , COGNOME (20caratteri), NR. di TEL.,       
  CITTA'(10caratteri).                                                        
  Utenti massimi [100]                                                       
  Implementare le seguenti funzioni:  - Inserisci contatto                    
                                      - Modifica Contatto                    
                                      - Cancella contatto                     
                                      - Visualizza contatti                   
*******************************************************************************/

                                                             /* RUBRICA V 1.0 */

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define ELEM 100
#define NCHAR 20

// Definizione della struttura
typedef struct {
        char nome[NCHAR];
        char cognome[NCHAR];
        char telefono[NCHAR];
        char citta[NCHAR];
        } account;
        
// Variabili globali
account rubrica[ELEM];
int num_account = 0;

// Dichiarazione delle funzioni
void InsContatto (char nom[], char cogn[], char tel[], char cit[]);
void ModContatto (char nom[], char cogn[]);
void Cancella (char nom[], char cogn[]);
void Visualizza (char nom[], char cogn[]);

main() 
{ //OPEN main
     int scelta; /* per inserire la scelta del menù */
     char n[NCHAR]; /* per inserire il nome */ 
     char m[NCHAR]; /* per inserire il cognome */
     char c[NCHAR]; /* per inserire la città */
     char t[15]; /* per inserire il numero di telefono */
     int i;
     for (;scelta!= 0;)
     { //OPEN for
          printf ("\t  INDEX\n");
          printf ("***************************\n");
          printf ("* 1 - Inserisci Contatto  *\n");
          printf ("* 2 - Modifica Contatto   *\n");
          printf ("* 3 - Cancella Contatto   *\n");
          printf ("* 4 - Visualizza Contatto *\n");
          printf ("*                         *\n");
          printf ("* 0 - Esci dal programma  *\n");
          printf ("***************************\n\n");
          printf ("Digita: ");
          scanf ("%d", &scelta);
          switch (scelta) 
          { //OPEN switch
                 case 0: // Exit
                           printf ("\n\t\tGrazie per la scelta\n\n");
                           break;
                 case 1: // Inserisci contatto
                           printf ("\nNome: ");
                           scanf ("%s", n);
                           printf ("Cognome: ");
                           scanf ("%s", m);
                           printf ("Telefono: ");
                           scanf ("%s", t);
                           printf ("Citta': ");
                           scanf ("%s", c);
                           printf ("\n");
                           InsContatto(n,m,t,c);
                           break;
                 case 2: // Modifica Contatto
                           printf ("\nNome: ");
                           scanf ("%s",n);
                           printf ("Cognome: ");
                           scanf ("%s", m);
                           ModContatto (n,m);
                           break;
                 case 3: // Cancella
                           printf ("\nNome: ");
                           scanf ("%s",n);
                           printf ("Cognome: ");
                           scanf ("%s", m);
                           Cancella (n,m);
                           break;
                 case 4: // Visualizza
                           printf ("\nNome: ");
                           scanf ("%s",n);
                           printf ("Cognome: ");
                           scanf ("%s", m);
                           Visualizza (n,m);
                           break;           
          } //CLOSED switch
    } //CLOSED for
} //CLOSED main

// Definizione delle funzioni
void InsContatto (char nom[], char cogn[], char tel[], char cit[])
{ //OPEN InsContatto
     int i;
     if (num_account == ELEM)
     { //OPEN if
         printf ("Hai raggiunto il limite di contatti offerto\n\n");
         return;
     } //CLOSED if
     for (i=0; i<=num_account; i++)
     { //OPEN for
         if ((strcmp (nom,rubrica[i].nome)==0) && (strcmp (cogn,rubrica[i].cognome)==0))
         { //OPEN if
            printf ("Nome e Cognome utente esistono in memoria\n\n");
            return;
         } //CLOSED if
     } //CLOSED for
     strcpy (rubrica[num_account].nome,nom);
     strcpy (rubrica[num_account].cognome,cogn);
     strcpy (rubrica[num_account].telefono,tel);  
     strcpy (rubrica[num_account].citta,cit);
     num_account++;
     return;
} //CLOSED InsContatto

void ModContatto (char nom[], char cogn[])
{ //OPEN ModContatto 
     int i;
     for (i=0; i<=num_account; i++)
     { //OPEN for
         if ((strcmp (nom,rubrica[i].nome)!=0) && (strcmp (cogn,rubrica[i].cognome)!=0))
         { //OPEN if 
            printf ("Il contatto non esiste");
            printf ("\n\n");  
            return;
         } //CLOSED if
         else
         { //OPEN else
             printf("Nuovo Nome: ");
             scanf("%s", &rubrica[i].nome);
             printf("Nuovo Cognome: ");
             scanf("%s", &rubrica[i].cognome);
             printf("Nuovo Telefono: ");
             scanf("%s", &rubrica[i].telefono);
             printf("Nuova Citta': ");
             scanf("%s", &rubrica[i].citta);
             return;
         } //CLOSED else
     } //CLOSED for
} //CLOSED ModContatto

void Cancella (char nom[], char cogn[])
{ //OPEN cancella
     int i;
     for (i=0; i<=num_account; i++)
     { //OPEN for
         if ((strcmp (nom,rubrica[i].nome)!=0) && (strcmp (cogn,rubrica[i].cognome)!=0))
         { //OPEN if 
            printf ("Il contatto non esiste");
            printf ("\n\n");  
            return;
         } //CLOSED if
         else
         { //OPEN else
            rubrica[i].nome[0] ='\0';
            rubrica[i].cognome[0] ='\0';
            rubrica[i].telefono[0] ='\0';
            rubrica[i].citta[0] ='\0';
            return;
         } //CLOSED else
     } //CLOSED for
} //CLOSED Cancella
 
void Visualizza (char nom[], char cogn[])
{ //OPEN Visualizza
     int i;
     for (i=0; i<=num_account; i++)
     { //OPEN for
         if ((strcmp (nom,rubrica[i].nome)==0) && (strcmp (cogn,rubrica[i].cognome)==0))
         { //OPEN if
            printf ("Il contatto scelto e': \n");
            printf ("\n%s %s\n", rubrica[i].nome, rubrica[i].cognome);
            printf ("%s\n%s", rubrica[i].telefono, rubrica[i].citta);
            printf ("\n\n");
            return;
         } //CLOSED if
         else
         { //OPEN else
            printf ("Il contatto non esiste");
            printf ("\n\n");  
            return;
         } //CLOSED else
     } //CLOSED for
} //CLOSED Visualizza