6

试图合并 8 个预先排序的数组。我对C相当陌生,但这是我迄今为止想出的。不用说它不起作用。我想不通的是为什么。我在这里基于 C++ 合并排序实现,并尝试将其扩展到 8 维并稍微简化它,但由于某种原因,它倾向于给我第一个数组的元素,然后是 34、30,然后它重复 23直到最后。它甚至没有发现 21 是第一次迭代中的最小值。

int test1[5] = {23, 24, 25, 33, 51};
int test2[5] = {21, 34, 44, 50, 62};
int test3[5] = {34, 36, 41, 44, 46};
int test4[5] = {30, 31, 32, 35, 40};
int test5[5] = {54, 56, 57, 58, 60};
int test6[5] = {31, 33, 36, 51, 52};
int test7[5] = {44, 46, 76, 78, 79};
int test8[5] = {23, 33, 43, 54, 63};

int output[40];
int t1, t2, t3, t4, t5, t6, t7, t8;
t1 = 0;
t2 = 0;
t3 = 0;
t4 = 0;
t5 = 0;
t6 = 0;
t7 = 0;
t8 = 0;
int p = 0;
int temp1;
int temp2;

while(p < 40) {
    if (t1 < 5) {
        temp1 = 1;
        temp2 = test1[t1];
    }else if (test2[t2] <= test1[t1] && t2 < 5) {
        temp1 = 2;
        temp2 = test2[t2];
    }else if (test3[t3] <= temp2 && t3 < 5) {
        temp1 = 3;
        temp2 = test3[t3];
    }else if (test4[t4] <= temp2 && t4 < 5) {
        temp1 = 4;
        temp2 = test4[t4];
    }else if (test5[t5] <= temp2 && t5 < 5) {
        temp1 = 5;
        temp2 = test5[t5];
    }else if (test6[t6] <= temp2 && t6 < 5) {
        temp1 = 6;
        temp2 = test6[t6];
    }else if (test7[t7] <= temp2 && t7 < 5) {
        temp1 = 7;
        temp2 = test7[t7];
    }else if (test8[t8] <= temp2 && t8 < 5) {
        temp1 = 8;
        temp2 = test8[t8];
    }
    switch(temp1) {
        case 1:
            output[p] = temp2;
            t1++;
            break;
        case 2:
            output[p] = temp2;
            t2++;
            break;
        case 3:
            output[p] = temp2;
            t3++;
            break;
        case 4:
            output[p] = temp2;
            t4++;
            break;
        case 5:
            output[p] = temp2;
            t5++;
            break;
        case 6:
            output[p] = temp2;
            t6++;
            break;
        case 7:
            output[p] = temp2;
            t7++;
            break;
        case 8:
            output[p] = temp2;
            t8++;
            break;
    }
    printf("%d\n",  output[p]);
    p++;
}

谢谢你尽你所能的帮助。

4

3 回答 3

4

这就是您获得第一个数组的前 5 个元素的原因:

if (t1 < 5) {
        temp1 = 1;
        temp2 = test1[t1];

您的代码专门确保首先选择第一个数组的前 5 个元素。您应该将 test1 中下一个元素的值与其他数组中的下一个元素进行比较,而不是盲目地选择它。此外,您对 if..then..else if.. else if... 的使用不正确。如果你发现数组 2 的下一个元素小于数组 1 的下一个元素,你仍然需要检查数组 3、4 和 5 是否更小。

尝试像这样构建您的代码

int temp1 = -1;
if (t1 < 5) {
    temp1=1;
    temp2=test1[t1];
}
if ((t2 < 5) && ((temp1 < 0) || (test2[t2] < temp2))) {
    temp1=2;
    temp2=test2[t2];
}
if (t3...

后跟您现有的 switch 语句。

于 2012-05-27T14:24:37.500 回答
3

斧头已经告诉你出了什么问题,当且仅当第二个条件有效第一个条件无效时,if ... else if ...才会执行第二个块。

但是,我相信您的switch()if-else基于的程序有点难以扩展,因为您需要更改所有固定字段和值来对另一组数组进行排序。以下程序/功能提供与您的相同,但更容易扩展。它使用一个额外的数组cursor来保存当前位置(类似于你的t1, t2,...)(Ideone Demo)。

#include <stdio.h>
#include <stdlib.h>
#include <limits.h>

int multimerge(
    int * const * const arrays,      // arrays holding the data
    int const * const arraysizes,    // sizes of the arrays in `arrays`
    int number_of_arrays,            // number of arrays
    int * const output               // pointer to output buffer
){
    int i = 0;       // output cursor
    int j = 0;       // index for minimum search
    int min;         // minimum in this iteration
    int minposition; // position of the minimum

    // cursor for the arrays
    int * cursor = calloc(number_of_arrays,sizeof(int));

    if(cursor == NULL)
        return -1;

    while(1){
        min = INT_MAX;
        minposition = -1; // invalid position

        // Go through the current positions and get the minimum
        for(j = 0; j < number_of_arrays; ++j){

            if(cursor[j] < arraysizes[j] &&  // ensure that the cursor is still valid
               arrays[j][cursor[j]] < min){  // the element is smaller
                min = arrays[j][cursor[j]];  // save the minimum ...
                minposition = j;             // ... and its position
            }
        }

        // if there is no minimum, then the position will be invalid

        if(minposition == -1)
            break;

        // update the output and the specific cursor            
        output[i++] = min;
        cursor[minposition]++;
    }
    free(cursor);

    return 0;
}

int main()
{
    int i = 0;
    int test1[5] = {23, 24, 25, 33, 51};
    int test2[5] = {21, 34, 44, 50, 62};
    int test3[5] = {34, 36, 41, 44, 46};
    int test4[5] = {30, 31, 32, 35, 40};
    int test5[5] = {54, 56, 57, 58, 60};
    int test6[5] = {31, 33, 36, 51, 52};
    int test7[5] = {44, 46, 76, 78, 79};
    int test8[5] = {23, 33, 43, 54, 63};

    int *test[] = {test1, test2, test3, test4, test5, test6, test7, test8};
    int testsizes[] = {5,5,5,5,5,5,5,5};

    int output[40];

    multimerge(test,testsizes,8,output);

    while(i < 30){
        printf("%i\n",output[i++]);
    }    

    return 0;
}
于 2012-05-27T14:55:01.360 回答
2

我在(遥远的)过去所做的是用标签排序对每个列表的第一个元素进行排序,选择排序列表中的第一个元素添加到合并列表中,然后将移动的项目替换为下一个它的列表(这就是为什么您需要某种风格的标签,以识别替换应该来自哪里)并重新排序。头部元素的排序可以使用冒泡排序,因为在初始排序之后,您只需要一次通过即可在移除/替换后正确重新排序事物。

于 2012-05-27T14:12:41.473 回答