3

to tendo problema na hora de executar esse código, cujo objetivo é adicionar um elemento no final da lista, o programa fica aguardando algo que não sei o que é, então o programa da erro. Alguma luz?

To começando agora listas, então a possibilidade de ser um erro besta é grande hehe.

typedef struct evento_t {
double tempo;
char descricao[50];
int id_origem, id_destino;
struct evento_t *prox;
} evento_t;


bool eventos_adicionar_fim (evento_t **lista, double tempo, char descricao[], int   id_origem, int id_destino) {

evento_t *novoelemento = (evento_t*)malloc(sizeof(evento_t));
evento_t *auxiliar = *lista, *anterior = NULL;

if (novoelemento!=NULL) {
    novoelemento->tempo = tempo;
    strcpy (novoelemento->descricao, descricao);
    novoelemento->id_origem = id_origem;
    novoelemento->id_destino = id_destino;
    novoelemento->prox = NULL;

    while (auxiliar!=NULL) {
        anterior = auxiliar;
        auxiliar = auxiliar->prox;
    }

    anterior->prox = novoelemento;

    return true;
}

else {
    exit(1);
    return false;
}

}

9
  • 1
    Note que você não previu o caso de a lista ter zero elementos (o while não executará e anterior será NULL). Nessa situação faça *lista = novoelemento;. Qual é exatamente o erro que você está observando? Crash? 27/02/2014 às 23:29
  • Cade a definição da função main? Seu código parece estar incompleto. poste aqui pra gente dar uma olhada: pastebin.com
    – gvsrepins
    27/02/2014 às 23:31
  • Poste na própria pergunta, não tem porque por em um local externo. 27/02/2014 às 23:34
  • Como estou executando pelo cmd, o cmd para de responder. Já verifiquei o erro do NULL e ainda dá no mesmo. Acho que o main ta correto, porque tenho uma função pra adicionar no inicio e ela ta funcionando normal.
    – Kay
    27/02/2014 às 23:35
  • modifiquei o código
    – Kay
    27/02/2014 às 23:37

1 Resposta 1

2

O problema está em tentar inserir algo no fim de uma lista vazia. Nesse caso tanto o auxiliar quanto o anterior sejam NULL. O crash acontecerá ao fazer anterior->prox = novoelemento;. Em vez disso adicione uma checagem:

if (*lista == NULL) {
    *lista = novoelemento;
} else {
    // código anterior
}

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 .