1

Por que essa sintaxe está errada:

#include <stdio.h>

typedef struct {

  char* nome;
  char* numero;

}Agenda;

void adiciona(Agenda* reg, int i) {

    scanf("%s", reg[i]->nome);
    scanf("%s", reg[i]->numero);

}


void imprime(Agenda* reg, int i) {

  for(int j = 0; j < i; j++) {

    printf("Nome: %s | ", reg[j]->nome);
    printf("Numero: %s\n", reg[j]->numero);
  }

}



int main() {

  int sair = 0;
  int i = 0;

  Agenda registro[10];

  while(!sair) {

    int escolha;
    printf("O que voce deseja ? (1)Inserir (2)Imprimir (3)Sair\n");
    scanf("%d", &escolha);

    switch(escolha) {       
        case 1:
            adiciona(registro, i);
            i++;
            break;
        case 2:
            imprime(registro, i);
            break;
        case 3:
            sair = 1;
            break;
      } 


   }


}

E por que quando peço para imprimir o nome e o contato sai uns caracteres estranhos e caso seja mais de um sai repetido? Isto é, como se a primeira posição e a segunda do vetor tivessem o mesmo valor.

1 Resposta 1

2

Tem dois problemas. Um deles é que não está alocando memória para as strings que serão apontadas na estrutura. Note que ali só terá um ponteiro indicando onde está o texto. E onde está? onde ele foi colocado? Em lugar algum qualquer. E depois o acesso será feito em algum lugar praticamente aleatório da memória já que não se sabe onde foi alocado.

Alocando a memória com malloc() garantirá que o texto tenha um lugar para ser armazenado e retorne um ponteiro para ele, esse ponteiro é que deve ser colocado na estrutura.

E não pode usar o ponteiro ->, o . é o correto já que a alocação das estruturas vão direto no array e não alocadas indiretamente, como ocorre como o texto.

Ainda teria o problema de não estar liberando a memória com free(), mas em um exercício simples assim não é um problema.

Idealmente deveria ter um limite de quantos caracteres podem ser digitados.

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

typedef struct {
    char *nome;
    char *numero;
} Agenda;

void adiciona(Agenda* reg, int i) {
    reg[i].nome = malloc(30);
    reg[i].numero = malloc(6);
    scanf("%s", reg[i].nome);
    scanf("%s", reg[i].numero);
}

void imprime(Agenda* reg, int i) {
    for (int j = 0; j < i; j++) {
        printf("Nome: %s | ", reg[j].nome);
        printf("Numero: %s\n", reg[j].numero);
    }
}

int main() {
    int sair = 0;
    int i = 0;
    Agenda registro[10];
    while (!sair) {
        int escolha;
        printf("O que voce deseja ? (1)Inserir (2)Imprimir (3)Sair\n");
        scanf("%d", &escolha);
        switch(escolha) {       
            case 1:
                adiciona(registro, i);
                i++;
                break;
            case 2:
                imprime(registro, i);
                break;
            case 3:
                sair = 1;
                break;
        } 
    }
}

Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.

1

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .