0

Estou tendo um problema em um algoritmo de ordenação mergesort em que a saída do vetor ordenado aparece um numero repetido e também um endereço de memória. Segue os codigos:

MergeSort

/*Funcao que implementa o mergeSort*/
void merge(int arr[], int l, int m, int r) {
    int i, j, k;
    int n1 = m - l + 1;
    int n2 =  r - m;

    /* cria um array temporario*/
    int L[n1], R[n2];

    /* Copia os dados para os arrays temporarios L[] e R[] */
    for (i = 0; i < n1; i++) {
        L[i] = arr[l + i];
    }
    for (j = 0; j < n2; j++) {
        R[j] = arr[m + 1 + j];
    }

    /* Merge os arrays temporarios de volta para o array[l..r]*/
    i = 0; /*index inicial do primeiro subarray*/
    j = 0; /*index inicial do segundo subarray*/
    k = l; /*index inicial do array merged*/
    while (i < n1 && j < n2) {
        if (L[i] <= R[j]) {
            arr[k] = L[i];
            i++;
        } else {
            arr[k] = R[j];
            j++;
        }
        k++;
    }

    /*Copia os elementos restantes para L[], se houver algum*/
    while (i < n1) {
        arr[k] = L[i];
        i++;
        k++;
    }

    /*Copia os elementos restantes para R[], se houver algum*/
    while (j < n2) {
        arr[k] = R[j];
        j++;
        k++;
    }
}


/*L e para o index da esquerda e r para o index da direita do
subarray de A que vai ser sorted*/
void mergeSort(int arr[], int l, int r)
{
    if (l < r) {
        /* O mesmo que (l+r)/2, mas evita overflow de l grande e h*/
        int m = l+(r-l)/2;

        /*Sort primeira e segunda metade */
        mergeSort(arr, l, m);
        mergeSort(arr, m+1, r);

        merge(arr, l, m, r);
    }
}

Print (Funcao que printa o vetor)

void print(int *A, int n) {
    printf("[");
    for(int i = 0; i < n; i++) {
        printf(" %i ", A[i]);
    }
    printf("]");
    printf("\n\n");
}

Main

int main(int argc, char const *argv[]) {
    int A[] = {1,7,3,8,5,2,9};
    int n = sizeof(A)/4;

    print(A,n);

    stoogeSort(A,0,n);

    print(A,n);

    return 0;
}

Saída

vetor desordenado
[ 1  7  3  8  5  2  9 ]

vetor ordenado
[ 1  2  3  5  7  7  8  9  6422300 ]

Qual seria o possível problema? Percebi que depois de passar o "n" como parâmetro da função mergeSort, seu valor é alterado, porém não sei como isso acontece.

2 Respostas 2

1

Sempre que você vê valores aleatórios deve desconfiar:

vetor ordenado
[ 1 2 3 5 7 7 8 9 6422300 ]
                             ^^^^^^

Na maior parte das vezes significa que está a aceder a partes da memoria que não devia e por isso representa comportamento indefinido. No seu exemplo é precisamente o que está a acontecer!

A sua chamada ao sort:

stoogeSort(A,0,n);

Foi feita passando n quando devia ser passada com n - 1, e por isso deveria estar assim:

stoogeSort(A,0,n - 1);

Como passou n, a função acedeu a mais valores na memoria do que deveria e acabou por sobrepor valores de variáveis, no caso acertou na sua variável n que ficou modificada no fim da ordenação.

Veja o código a funcionar com essa alteração

Nota: Não assuma que o tamanho de um int é sempre 4 bytes, como fez em int n = sizeof(A)/4;. Faça o correto e consulte o tamanho do int com sizeof(int).

0

Por algum motivo, o valor que você está passando de N está mudando. Portanto, sugiro que você use um #define N 6 ao invés de int n = sizeof(A)/4.

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 .