4

Estou tendo dificuldades ao atribuir um valor a uma variável do tipo char de um struct eu estou fazendo o seguinte

#include <stdio.h>

typedef struct Animal{
    char nome[5]; // indiquei que a variavel nome tem 5 caractes
    int idade;
}Cachorro;

int main(){
    Cachorro Dog;
    Dog.idade = 9;
    Dog.nome = "Salfr"; // tento atribuir "Salfr" a minha variavel
    printf("'%s' '%d'", Dog.nome, Dog.idade);
    return 0;
}

so que esta ocorrendo um erro de Falha de segmentação quando eu compilo aparece test.c|11|warning: assignment makes integer from pointer without a cast|

e quando eu executo aparece Segmentation Fault

agora quando eu indico que minha variavel char nome e um ponteiro funciona normalmente segue abaixo

#include <stdio.h>

typedef struct Animal{
    char *nome;
    int idade;
}Cachorro;

int main(){
    Cachorro Dog;
    Dog.idade = 9;
    Dog.nome = "Salfr";
    printf("'%s' '%d'", Dog.nome, Dog.idade);
    return 0;
}

ao compilar e executar recebo o seguinte retorno

'Safari' '9'

o meu problema esta relacionado com o tipo char referente a esta minha pergunta Existe algum problema em atribuir um valor a um ponteiro? eu sei que esse metodo nao e aconselhavel mas foi o unico modo que funcionou ate o momento alguem sabe por que esta acontecendo Segmentation Fault no primeiro exemplo?

2

1 Resposta 1

8

O principal erro é achar que o operador de atribuição funciona para strings. Não dá, precisa chamar a função que copia os caracteres para onde deseja (strcpy()).

Ainda há o problema de faltar um byte para o caractere terminador \0. Toda string em C precisa de um byte a mais. C é uma linguagem bruta, o programador tem que se preocupar com tudo.

#include <stdio.h>
#include <string.h>
 
typedef struct Animal {
    char nome[6]; //o tamanho tem que ser 6 para acomodar o terminador
    int idade;
} Cachorro;
 
int main() {
    Cachorro dog;
    dog.idade = 9;
    strcpy(dog.nome, "Salfr"); //é a forma correta pra copiar uma string para uma variável
    printf("'%s' '%d'", dog.nome, dog.idade);
}

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

A versão com ponteiro funciona, desde que não deseje mexer no texto nunca. Ela está fazendo a estrutura apontar para um região da memória "pertencente ao código" e é protegida. O normal não é fazer isto. Isso pode ter funcionando por coincidência. Aí ela é uma solução pior porque não aloca memória necessária para sustentar o objeto texto. O texto só poderia ser manipulado após essa alocação. Mesmo que o fizesse ainda teria o problema da cópia que não está sendo feita.

Fiz alterações cosméticas, mas importantes. Fique atento.

Funcionar é uma coisa, estar certo é outra. Aconselho aprender o certo e não confiar no que funcionou. Pode ter sido só coincidência.

5
  • fogo e que eu encontrei varios exemplos na internet usando ponteiros voce poderia me indicar algum livro bom de c? @bigown
    – user45474
    Commented 23/06/2016 às 3:31
  • A maioria do material na internet sobre programação é muito ruim. Veja pt.stackoverflow.com/tags/c/info. Mas vai perguntando, ou pesquisando aqui, tem muita coisa legal e avaliada por outras pessoas.
    – Maniero
    Commented 23/06/2016 às 3:45
  • 1
    Correção: não há nenhum problema apontar para uma string literal, espaço já é reservado pelo compilador e a string irá para a região de memória junto com constantes, o único problema é que você não poderá alterar a string, mas apontar para lá e ler o conteúdo, sem problemas. Commented 23/06/2016 às 12:29
  • @MurilloHenrique é verdade, é que considerei que em geral não é o que a pessoa deseja em código normal.
    – Maniero
    Commented 23/06/2016 às 12:41
  • 1
    @bigown Com certeza, ainda mais se mais para frente, em um código extenso, houver erro por escrita no conteúdo de um ponteiro, rastrear isso seria muito complicado. Commented 23/06/2016 às 12:47

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