Skip to main content
adicionou 93 caracteres ao conteúdo
Fonte Link

É impressão minha ou você se esqueceu de liberar a memória alocada pela lista? Embora não seja praticamente necessário, é muito oportuno.

Eu também, por praticidade, costumo utilizar uma única lista encadeada, mas do tipo cell*, em escopo global para trabalhar, por isso o procedimento abaixo é vazio de parâmetro. No seu casoPara quem opta por fornecer o endereço de uma lista local a uma função, porém, basta fazerrealizar alguns ajustes para que ele receba o endereço da listaneste algoritmo.

void free_llist(void)
{
    if (linked_list)
    {
        cell* aux = linked_list;

        while (linked_list)
        {
            aux = linked_list;
            linked_list = linked_list->next;
            free(aux);
            aux = (cell*) 0;
        }

        free(linked_list);
        linked_list = (cell*) 0;
    }

    return;
}

É impressão minha ou você se esqueceu de liberar a memória alocada pela lista? Embora não seja praticamente necessário, é muito oportuno.

Eu, por praticidade, costumo utilizar uma única lista encadeada do tipo cell* em escopo global para trabalhar, por isso o procedimento abaixo é vazio de parâmetro. No seu caso, porém, basta fazer alguns ajustes para que ele receba o endereço da lista.

void free_llist(void)
{
    if (linked_list)
    {
        cell* aux = linked_list;

        while (linked_list)
        {
            aux = linked_list;
            linked_list = linked_list->next;
            free(aux);
            aux = (cell*) 0;
        }

        free(linked_list);
        linked_list = (cell*) 0;
    }

    return;
}

É impressão minha ou você se esqueceu de liberar a memória alocada pela lista? Embora não seja praticamente necessário, é muito oportuno.

Eu também, por praticidade, costumo utilizar uma única lista encadeada, mas do tipo cell*, em escopo global para trabalhar, por isso o procedimento abaixo é vazio de parâmetro. Para quem opta por fornecer o endereço de uma lista local a uma função, porém, basta realizar alguns ajustes neste algoritmo.

void free_llist(void)
{
    if (linked_list)
    {
        cell* aux = linked_list;

        while (linked_list)
        {
            aux = linked_list;
            linked_list = linked_list->next;
            free(aux);
            aux = (cell*) 0;
        }

        free(linked_list);
        linked_list = (cell*) 0;
    }

    return;
}
Post restaurado por Marcelo Siqueira Lima
adicionou 93 caracteres ao conteúdo
Fonte Link

NuncaÉ impressão minha ou você se esqueçaesqueceu de liberar de maneira dinâmica o bloco de memória que foi alocado do mesmo jeito. Após isso, "anule" o ponteiro indexado a ele para impedir que "fique solto". Do contrário o programa vai apresentar erros inesperadosmemória alocada pela lista? Embora não seja praticamente necessário, graves ou até difíceis de serem localizados. Isso é incontestávelmuito oportuno.

EntãoEu, por praticidade, costumo utilizar uma única lista encadeada do tipo cell* em escopo global para cadatrabalhar, por isso o procedimento abaixo é vazio de parâmetro. No seu caso, porém, basta fazer alguns ajustes para que ele receba o endereço da lista.

char*void strfree_llist(void)
{
 =   if (char*linked_list) 
 malloc(n * sizeof {
        cell* aux = linked_list;

        while (charlinked_list)
        {
            aux = linked_list;
            linked_list = linked_list->next;
            free(aux);

Devemos garantir

            aux = (cell*) 0;
        }

        free(strlinked_list);
str = NULL; // str    linked_list = (char*cell*) 0;
    }

    return;
}

Isso é programação segura.

Outra dica é evitar o uso de malloc(), porque esse procedimento não inicializa o ponteiro gerado e, portanto, seu conteúdo pode ter qualquer tipo de "lixo". Devemos utilizar calloc(): designa uma porção de memória limpa, ativada para zero (0).

Nunca se esqueça de liberar de maneira dinâmica o bloco de memória que foi alocado do mesmo jeito. Após isso, "anule" o ponteiro indexado a ele para impedir que "fique solto". Do contrário o programa vai apresentar erros inesperados, graves ou até difíceis de serem localizados. Isso é incontestável.

Então, para cada

char* str = (char*) malloc(n * sizeof (char));

Devemos garantir

free(str);
str = NULL; // str = (char*) 0;

Isso é programação segura.

Outra dica é evitar o uso de malloc(), porque esse procedimento não inicializa o ponteiro gerado e, portanto, seu conteúdo pode ter qualquer tipo de "lixo". Devemos utilizar calloc(): designa uma porção de memória limpa, ativada para zero (0).

É impressão minha ou você se esqueceu de liberar a memória alocada pela lista? Embora não seja praticamente necessário, é muito oportuno.

Eu, por praticidade, costumo utilizar uma única lista encadeada do tipo cell* em escopo global para trabalhar, por isso o procedimento abaixo é vazio de parâmetro. No seu caso, porém, basta fazer alguns ajustes para que ele receba o endereço da lista.

void free_llist(void)
{
    if (linked_list) 
    {
        cell* aux = linked_list;

        while (linked_list)
        {
            aux = linked_list;
            linked_list = linked_list->next;
            free(aux);
            aux = (cell*) 0;
        }

        free(linked_list);
        linked_list = (cell*) 0;
    }

    return;
}
Post removido por Marcelo Siqueira Lima
Fonte Link

Nunca se esqueça de liberar de maneira dinâmica o bloco de memória que foi alocado do mesmo jeito. Após isso, "anule" o ponteiro indexado a ele para impedir que "fique solto". Do contrário o programa vai apresentar erros inesperados, graves ou até difíceis de serem localizados. Isso é incontestável.

Então, para cada

char* str = (char*) malloc(n * sizeof (char));

Devemos garantir

free(str);
str = NULL; // str = (char*) 0;

Isso é programação segura.

Outra dica é evitar o uso de malloc(), porque esse procedimento não inicializa o ponteiro gerado e, portanto, seu conteúdo pode ter qualquer tipo de "lixo". Devemos utilizar calloc(): designa uma porção de memória limpa, ativada para zero (0).