Skip to main content
adicionou 279 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

Este código tem alguns problemas.

  • Não tem porque usar malloc() se a área que a string deve ficar já está reservada dentro da estrutura. Até poderia alocar se desejar, e talvez faça sentido para os nomes, mas aí precisa declarar como const char * e não [tamanho]. Tem que ser bem mais cuidadoso quando faz istoisso. O que inclusive está vazando memória. Em grande volume istoisso seria trágico. E alocar apenas 2 caracteres onde precisa de vários também não dá muito certo, é que neste caso coincidentemente funciona.
  • Não que seja um problema, mas não vejo motivo para criar uma estrutura local, inicializar seus membros e depois copiar para o array. Escreva diretamente no array, sem criar nada intermediário, inicializar ou copiar.
  • As strings possuem um terminador, então precisa reservar espaço para ele. O problema específico que está encontrando é que tem 4 bytes reservados para o ano, então os 4 caracteres do ano são colocados ali, e um 5o. é colocado em seguida. Quando coloca o nome do diretor, o seu primeiro caractere vai em cima do terminador da string do ano. Aí quando vai ler o ano ele não tem fim, só finalizaria no fim do nome do diretor, por isso fica tudo junto. Se tivesse 5 bytes, o terminador seria preservado e tudo funcionaria ok.

Assim funciona:

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

typedef struct {
    char nome[81];
    char ano[5];
    char diretor[81];
} Filme;

void analise(Filme *filme, const char *arg1, const char *arg2, const char *arg3) {
    strcpy(filme->nome, arg1);
    strcpy(filme->ano, arg2);
    strcpy(filme->diretor, arg3);
}

void carregar(Filme filmes[]) {
    analise(&filmes[0], "E o Vento Levou", "1939", "Victor");
    analise(&filmes[1], "teste", "998", "bar");
    analise(&filmes[2], "Os Passaros", "1963", "Alfred Hitchcock");
}

int main() {
    Filme filmes[1000];
    carregar(filmes);
    printf("\nmain:\n");
    for (int i = 0; i < 3; i++) {
        printf("- Nome:    %s\n", filmes[i].nome);
        printf("- Ano:     %s\n", filmes[i].ano);
        printf("- Diretor: %s\n", filmes[i].diretor);
        printf("----------------\n");
    }
}

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

Este código tem alguns problemas.

  • Não tem porque usar malloc() se a área que a string deve ficar já está reservada dentro da estrutura. Até poderia alocar se desejar, e talvez faça sentido para os nomes, mas aí precisa declarar como const char * e não [tamanho]. Tem que ser bem mais cuidadoso quando faz isto. O que inclusive está vazando memória. Em grande volume isto seria trágico. E alocar apenas 2 caracteres onde precisa de vários também não dá muito certo, é que neste caso coincidentemente funciona.
  • Não que seja um problema, mas não vejo motivo para criar uma estrutura local, inicializar seus membros e depois copiar para o array. Escreva diretamente no array, sem criar nada intermediário, inicializar ou copiar.
  • As strings possuem um terminador, então precisa reservar espaço para ele. O problema específico que está encontrando é que tem 4 bytes reservados para o ano, então os 4 caracteres do ano são colocados ali, e um 5o. é colocado em seguida. Quando coloca o nome do diretor, o seu primeiro caractere vai em cima do terminador da string do ano. Aí quando vai ler o ano ele não tem fim, só finalizaria no fim do nome do diretor, por isso fica tudo junto. Se tivesse 5 bytes, o terminador seria preservado e tudo funcionaria ok.

Assim funciona:

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

typedef struct {
    char nome[81];
    char ano[5];
    char diretor[81];
} Filme;

void analise(Filme *filme, const char *arg1, const char *arg2, const char *arg3) {
    strcpy(filme->nome, arg1);
    strcpy(filme->ano, arg2);
    strcpy(filme->diretor, arg3);
}

void carregar(Filme filmes[]) {
    analise(&filmes[0], "E o Vento Levou", "1939", "Victor");
    analise(&filmes[1], "teste", "998", "bar");
    analise(&filmes[2], "Os Passaros", "1963", "Alfred Hitchcock");
}

int main() {
    Filme filmes[1000];
    carregar(filmes);
    printf("\nmain:\n");
    for (int i = 0; i < 3; i++) {
        printf("- Nome:    %s\n", filmes[i].nome);
        printf("- Ano:     %s\n", filmes[i].ano);
        printf("- Diretor: %s\n", filmes[i].diretor);
        printf("----------------\n");
    }
}

Este código tem alguns problemas.

  • Não tem porque usar malloc() se a área que a string deve ficar já está reservada dentro da estrutura. Até poderia alocar se desejar, e talvez faça sentido para os nomes, mas aí precisa declarar como const char * e não [tamanho]. Tem que ser bem mais cuidadoso quando faz isso. O que inclusive está vazando memória. Em grande volume isso seria trágico. E alocar apenas 2 caracteres onde precisa de vários também não dá muito certo, é que neste caso coincidentemente funciona.
  • Não que seja um problema, mas não vejo motivo para criar uma estrutura local, inicializar seus membros e depois copiar para o array. Escreva diretamente no array, sem criar nada intermediário, inicializar ou copiar.
  • As strings possuem um terminador, então precisa reservar espaço para ele. O problema específico que está encontrando é que tem 4 bytes reservados para o ano, então os 4 caracteres do ano são colocados ali, e um 5o. é colocado em seguida. Quando coloca o nome do diretor, o seu primeiro caractere vai em cima do terminador da string do ano. Aí quando vai ler o ano ele não tem fim, só finalizaria no fim do nome do diretor, por isso fica tudo junto. Se tivesse 5 bytes, o terminador seria preservado e tudo funcionaria ok.

Assim funciona:

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

typedef struct {
    char nome[81];
    char ano[5];
    char diretor[81];
} Filme;

void analise(Filme *filme, const char *arg1, const char *arg2, const char *arg3) {
    strcpy(filme->nome, arg1);
    strcpy(filme->ano, arg2);
    strcpy(filme->diretor, arg3);
}

void carregar(Filme filmes[]) {
    analise(&filmes[0], "E o Vento Levou", "1939", "Victor");
    analise(&filmes[1], "teste", "998", "bar");
    analise(&filmes[2], "Os Passaros", "1963", "Alfred Hitchcock");
}

int main() {
    Filme filmes[1000];
    carregar(filmes);
    printf("\nmain:\n");
    for (int i = 0; i < 3; i++) {
        printf("- Nome:    %s\n", filmes[i].nome);
        printf("- Ano:     %s\n", filmes[i].ano);
        printf("- Diretor: %s\n", filmes[i].diretor);
        printf("----------------\n");
    }
}

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

adicionou 450 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

Este código tem alguns problemas.

  • Não tem porque usar malloc() se a área que a string deve ficar já está reservada dentro da estrutura. Até poderia alocar se desejar, e talvez faça sentido para os nomes, mas aí precisa declarar como const char * e não [tamanho]. TemqueTem que ser bem mais cuidadoso quando faz isto. O que inclusive está vazando memória. Em grande volume isto seria trágico. E alocar apenas 2 caracteres onde precisa de vários também não dá muito certo, é que neste caso coincidentemente funciona.
  • Não que seja um problema, mas não vejo motivo para criar uma estrutura local, inicializar seus membros e depois copiar para o array. Escreva diretamente no array, sem criar nada intermediário, inicializar ou copiar.
  • As strings possuem um terminador, então precisa reservar espaço para ele. O problema específico que está encontrando é que tem 4 bytes reservados para o ano, então os 4 caracteres do ano são colocados ali, e um 5o. é colocado em seguida. Quando coloca o nome do diretor, o seu primeiro caractere vai em cima do terminador da string do ano. Aí quando vai ler o ano ele não tem fim, só finalizaria no fim do nome do diretor, por isso fica tudo junto. Se tivesse 5 bytes, o terminador seria preservado e tudo funcionaria ok.

Assim funciona:

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

typedef struct {
    char nome[81];
    char ano[5];
    char diretor[81];
} Filme;

void analise(Filme *filme, const char *arg1, const char *arg2, const char *arg3) {
    strcpy(filme->nome, arg1);
    strcpy(filme->ano, arg2);
    strcpy(filme->diretor, arg3);
}

void carregar(Filme filmes[]) {
    analise(&filmes[0], "E o Vento Levou", "1939", "Victor");
    analise(&filmes[1], "teste", "998", "bar");
    analise(&filmes[2], "Os Passaros", "1963", "Alfred Hitchcock");
}

int main() {
    Filme filmes[1000];
    carregar(filmes);
    printf("\nmain:\n");
    for (int i = 0; i < 3; i++) {
        printf("- Nome:    %s\n", filmes[i].nome);
        printf("- Ano:     %s\n", filmes[i].ano);
        printf("- Diretor: %s\n", filmes[i].diretor);
        printf("----------------\n");
    }
}

Este código tem alguns problemas.

  • Não tem porque usar malloc() se a área que a string deve ficar já está reservada dentro da estrutura. Até poderia alocar se desejar, e talvez faça sentido para os nomes, mas aí precisa declarar como const char * e não [tamanho]. Temque ser bem mais cuidadoso quando faz isto.
  • Não que seja um problema, mas não vejo motivo para criar uma estrutura local, inicializar seus membros e depois copiar para o array. Escreva diretamente no array, sem criar nada intermediário, inicializar ou copiar.
  • As strings possuem um terminador, então precisa reservar espaço para ele.

Assim funciona:

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

typedef struct {
    char nome[81];
    char ano[5];
    char diretor[81];
} Filme;

void analise(Filme *filme, const char *arg1, const char *arg2, const char *arg3) {
    strcpy(filme->nome, arg1);
    strcpy(filme->ano, arg2);
    strcpy(filme->diretor, arg3);
}

void carregar(Filme filmes[]) {
    analise(&filmes[0], "E o Vento Levou", "1939", "Victor");
    analise(&filmes[1], "teste", "998", "bar");
    analise(&filmes[2], "Os Passaros", "1963", "Alfred Hitchcock");
}

int main() {
    Filme filmes[1000];
    carregar(filmes);
    printf("\nmain:\n");
    for (int i = 0; i < 3; i++) {
        printf("- Nome:    %s\n", filmes[i].nome);
        printf("- Ano:     %s\n", filmes[i].ano);
        printf("- Diretor: %s\n", filmes[i].diretor);
        printf("----------------\n");
    }
}

Este código tem alguns problemas.

  • Não tem porque usar malloc() se a área que a string deve ficar já está reservada dentro da estrutura. Até poderia alocar se desejar, e talvez faça sentido para os nomes, mas aí precisa declarar como const char * e não [tamanho]. Tem que ser bem mais cuidadoso quando faz isto. O que inclusive está vazando memória. Em grande volume isto seria trágico. E alocar apenas 2 caracteres onde precisa de vários também não dá muito certo, é que neste caso coincidentemente funciona.
  • Não que seja um problema, mas não vejo motivo para criar uma estrutura local, inicializar seus membros e depois copiar para o array. Escreva diretamente no array, sem criar nada intermediário, inicializar ou copiar.
  • As strings possuem um terminador, então precisa reservar espaço para ele. O problema específico que está encontrando é que tem 4 bytes reservados para o ano, então os 4 caracteres do ano são colocados ali, e um 5o. é colocado em seguida. Quando coloca o nome do diretor, o seu primeiro caractere vai em cima do terminador da string do ano. Aí quando vai ler o ano ele não tem fim, só finalizaria no fim do nome do diretor, por isso fica tudo junto. Se tivesse 5 bytes, o terminador seria preservado e tudo funcionaria ok.

Assim funciona:

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

typedef struct {
    char nome[81];
    char ano[5];
    char diretor[81];
} Filme;

void analise(Filme *filme, const char *arg1, const char *arg2, const char *arg3) {
    strcpy(filme->nome, arg1);
    strcpy(filme->ano, arg2);
    strcpy(filme->diretor, arg3);
}

void carregar(Filme filmes[]) {
    analise(&filmes[0], "E o Vento Levou", "1939", "Victor");
    analise(&filmes[1], "teste", "998", "bar");
    analise(&filmes[2], "Os Passaros", "1963", "Alfred Hitchcock");
}

int main() {
    Filme filmes[1000];
    carregar(filmes);
    printf("\nmain:\n");
    for (int i = 0; i < 3; i++) {
        printf("- Nome:    %s\n", filmes[i].nome);
        printf("- Ano:     %s\n", filmes[i].ano);
        printf("- Diretor: %s\n", filmes[i].diretor);
        printf("----------------\n");
    }
}
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

Este código tem alguns problemas.

  • Não tem porque usar malloc() se a área que a string deve ficar já está reservada dentro da estrutura. Até poderia alocar se desejar, e talvez faça sentido para os nomes, mas aí precisa declarar como const char * e não [tamanho]. Temque ser bem mais cuidadoso quando faz isto.
  • Não que seja um problema, mas não vejo motivo para criar uma estrutura local, inicializar seus membros e depois copiar para o array. Escreva diretamente no array, sem criar nada intermediário, inicializar ou copiar.
  • As strings possuem um terminador, então precisa reservar espaço para ele.

Assim funciona:

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

typedef struct {
    char nome[81];
    char ano[5];
    char diretor[81];
} Filme;

void analise(Filme *filme, const char *arg1, const char *arg2, const char *arg3) {
    strcpy(filme->nome, arg1);
    strcpy(filme->ano, arg2);
    strcpy(filme->diretor, arg3);
}

void carregar(Filme filmes[]) {
    analise(&filmes[0], "E o Vento Levou", "1939", "Victor");
    analise(&filmes[1], "teste", "998", "bar");
    analise(&filmes[2], "Os Passaros", "1963", "Alfred Hitchcock");
}

int main() {
    Filme filmes[1000];
    carregar(filmes);
    printf("\nmain:\n");
    for (int i = 0; i < 3; i++) {
        printf("- Nome:    %s\n", filmes[i].nome);
        printf("- Ano:     %s\n", filmes[i].ano);
        printf("- Diretor: %s\n", filmes[i].diretor);
        printf("----------------\n");
    }
}