c - 难以理解 void 数组并在函数调用中使用它们

Question

我正在尝试一个问题，该问题要求我创建一个名为 randSwap 的函数，该函数接受 2 个空指针数组，并且有 50% 的机会在 2 个数组之间交换相同索引的 2 个值。下面是我的代码，但是每当我去编译时，它都会给我一些错误讨论:

警告:初始化从未经强制转换的整数生成指针。

void *array1[4] = {3,4,2,5};

以及其他一些与数组 1 和 2 相关的内容。

当我运行程序时，array1 的值被损坏，产生一个大的整数，而 array2 两次打印其原始值。

我对使用 void 类型非常不熟悉，因为我认为这就是问题所在。

#include <stdio.h>

int randSwap(void *array1[], void *array2[], int length)
{
    static int numofswaps = 0;
    int value;
    int toswap;
    void *temp;


    value = rand() % 2;
    if(value == 2)
    {
        toswap = rand() % length;
        temp = array1[toswap];
        array1[toswap] = array2[toswap];
        array2[toswap] = temp;

        numofswaps++; 
    }

    return numofswaps;
}

int main(void)
{
    int i;
    void *array1[4] = {3,4,2,5};
    void *array2[4] = {6,3,7,4};
    int length = 4;
    int numofswaps;
    srand(time(NULL));

    printf("Array1\n");
    for(i=0; i<length; i++);
    {

        printf("%d\t", (int*)array1[i]);
    }

    printf("\nArray2\n");
    for(i=0; i<length; i++)
    {
        printf("%d\t", (int*)array2[i]);
    }

    numofswaps = randSwap(array1, array2, length);
    numofswaps = randSwap(array1, array2, length);

    printf("\nArray1\n");
    for(i=0; i<length; i++);
    {

        printf("%d\t", (int*)array1[i]);
    }

    printf("\nArray2\n");
    for(i=0; i<length; i++)
    {
        printf("%d\t", (int*)array2[i]);
    }
    printf("\n");
    return 0;
 }

Answer 1

您的代码中存在大量小问题。继续我的评论，您正在尝试用 4- int 初始化一个指针数组[4]值。而不是试图声明 array1和 array2作为无效，只需将它们声明为 int[]并将它们作为 void* 传递至 randSwap ，例如

    int i,
        array1[] = {3, 4, 2, 5},
        array2[] = {6, 3, 7, 4},
        length = sizeof array1 / sizeof *array1,
        numofswaps = 0;

下一个 numSwaps , 您使用 modulo ( % ) 差一:

    value = rand() % 2;     /* produces value 0 - 1 */
    if (value == 2)         /* will never equal 2 */

您只需要测试 0或 1 ，例如

    value = rand() % 2;     /* produces value 0 - 1 */
    if (value)              /* either 0 or 1 */
    {
        ...

不声明，numofswaps作为static在randSwap ，而是将其声明为简单的 int在main()并将其作为参数传递给 randSwap .

处理 numSwap 中的数组传递为 void * ，您需要将它们转换为 int*在尝试访问它们的值之前避免尝试取消引用 void指针(不允许)。你可以做类似的事情:

int randSwap (void *array1, void *array2, int length, int ns)
{
    int value,
        toswap,
        temp,
        *a1 = (int*)array1, /* note: the explicit casts are optional, the */
        *a2 = (int*)array2; /* declaration of a1 & a2 as int* is sufficient */

    value = rand() % 2;     /* produces value 0 - 1 */
    // if (value == 2)         /* will never equal 2 */
    if (value)              /* either 0 or 1 */
    {
        toswap = rand() % length;
        temp = a1[toswap];
        a1[toswap] = a2[toswap];
        a2[toswap] = temp;

        ns++; 
    }

    return ns;
}

( ns 代表 numofswaps -- 我不喜欢打字...)

将所有部分放在一起，您可以将代码重新编写为类似于以下内容:

#include <stdio.h>
#include <stdlib.h> /* for rand() and srand() */
#include <time.h>   /* for time() */

int randSwap (void *array1, void *array2, int length, int ns)
{
    int value,
        toswap,
        temp,
        *a1 = array1,
        *a2 = array2;

    value = rand() % 2;     /* produces value 0 - 1 */
    // if (value == 2)         /* will never equal 2 */
    if (value)              /* either 0 or 1 */
    {
        toswap = rand() % length;
        temp = a1[toswap];
        a1[toswap] = a2[toswap];
        a2[toswap] = temp;

        ns++; 
    }

    return ns;
}

int main (void)
{
    int i,
        array1[] = {3, 4, 2, 5},
        array2[] = {6, 3, 7, 4},
        length = sizeof array1 / sizeof *array1,
        numofswaps = 0;

    srand(time(NULL));

    printf ("Array1\n");
    for (i = 0; i < length; i++)
        printf (" %d", array1[i]);

    printf("\nArray2\n");
    for (i = 0; i < length; i++)
        printf (" %d", array2[i]);

    numofswaps = randSwap (array1, array2, length, numofswaps);
    numofswaps = randSwap (array1, array2, length, numofswaps);

    printf ("\nArray1\n");
    for (i = 0; i < length; i++)
        printf (" %d", array1[i]);

    printf ("\nArray2\n");
    for (i = 0; i < length; i++)
        printf(" %d", array2[i]);

    // putchar ('\n');     /* don't printf a single-character */
    printf ("\n\nnumber of swaps: %d\n", numofswaps);

    return 0;
}

(注意:你没有在numofswaps中输出main()，我在最后给你加上了)

示例使用/输出

 $ ./bin/ptrswap
Array1
 3 4 2 5
Array2
 6 3 7 4
Array1
 6 4 2 5
Array2
 3 3 7 4

number of swaps: 1

作为对 randSwap 的进一步细化, 无需申报 numofswaps作为static无需将其作为参数传递。您需要做的就是拥有 randSwap返回 0或 1指示是否发生了交换并将交换加总在 main() 中，例如

int randSwap (void *array1, void *array2, int length)
{
    int toswap, temp,
        *a1 = array1,
        *a2 = array2;

    if (rand() % 2) {   /* either 0 or 1 */
        toswap = rand() % length;
        temp = a1[toswap];
        a1[toswap] = a2[toswap];
        a2[toswap] = temp;

        return 1;
    }

    return 0;
}

然后在main() ,

    if (randSwap (array1, array2, length))
        numofswaps++;
    if (randSwap (array1, array2, length))
        numofswaps++;

(注意:这是处理它的更简洁的方式——相同的输出)

检查一下，如果您还有其他问题，请告诉我。