将char *数组中的字符串分配给另一个char *数组

Question

我正在尝试将数组a中的十六进制值转换为二进制值，并将转换后的值分配给数组b，然后打印数组b。但是数组b中的所有值都相同。输出为：

111100001011000100010111101010001101
111100001011000100010111101010001101
111100001011000100010111101010001101

如果我使用b[i] = strdup(hexToBin(a[i]));而不是b[i] = hexToBin(a[i]);，则输出将是：

111100001011
111100001011000100010111
111100001011000100010111101010001101

这与指针有关吗？字符*是一个指针，它指向字符串的第一个字符，并且该指针后的所有字符都被打印吗？正确的做法是什么？

#include <stdio.h>
#include <string.h>

char bin[100] = "";

char * hexToBin(char  hex[50]);

int main(void) {

    char * a[] = {
        "f0b",
        "117",
        "a8d",
    };

    char * b[3];

    for(int i = 0; i < 3; i++) {
        b[i] = hexToBin(a[i]);
    }

    for(int i = 0; i < 3; i++) {
        printf("%s\n", b[i]);
    }

}

char * hexToBin(char  hex[50]) {    

    for(int i=0; hex[i]!='\0'; i++)
    {
        switch(hex[i])
        {
            case '0':
                strcat(bin, "0000");
                break;
            case '1':
                strcat(bin, "0001");
                break;
            case '2':
                strcat(bin, "0010");
                break;
            case '3':
                strcat(bin, "0011");
                break;
            case '4':
                strcat(bin, "0100");
                break;
            case '5':
                strcat(bin, "0101");
                break;
            case '6':
                strcat(bin, "0110");
                break;
            case '7':
                strcat(bin, "0111");
                break;
            case '8':
                strcat(bin, "1000");
                break;
            case '9':
                strcat(bin, "1001");
                break;
            case 'a':
            case 'A':
                strcat(bin, "1010");
                break;
            case 'b':
            case 'B':
                strcat(bin, "1011");
                break;
            case 'c':
            case 'C':
                strcat(bin, "1100");
                break;
            case 'd':
            case 'D':
                strcat(bin, "1101");
                break;
            case 'e':
            case 'E':
                strcat(bin, "1110");
                break;
            case 'f':
            case 'F':
                strcat(bin, "1111");
                break;
            default:
                printf("Invalid hexadecimal input.");
        }
    }
    return bin;
}

Answer 1

hexToBin函数返回一个指向全局bin数组的第一个元素的指针。 每次！。

这意味着b中的所有指针将与指向bin数组中相同的第一个元素的指针完全相同。

如果您知道字符串的最大长度，建议您将b做成char的 arrays 的数组。例如

char b[3][500];  // 3 arrays of 499-character strings (+1 for the null-terminator)

然后，而不是hexToBin返回指向单个全局数组的指针，而是将指向要填充的字符串的指针传递为hexToBin的参数：

void hexToBin(char *hex, char *bin);

并命名为

hexToBin(a[i], b[i]);

Answer 2

您只有一个bin。您的hexToBin所做的是将其附加到该bin上，然后返回该bin。换句话说，当您多次调用它时，结果始终是相同的指针，因为您总是return bin;。

因此，如果您这样做：

b[i] = hexToBin(a[i]);

然后最后，b的所有元素都指向bin，这就是为什么在打印它们时会得到相同的输出。如果您改为这样做：

b[i] = strdup(hexToBin(a[i]));

然后结果不一样，因为它们都没有分配bin，而是当时bin的副本。这就是为什么结果不同的原因。因此，b[0]指向一个副本，然后再次附加bin，但这并不会更改b[0]副本。

如果您使用strdup，请不要忘记释放它分配的内存：

for(int i = 0; i < 3; i++) {
    free(b[i]);
}

Answer 3

[我的回答是错误的。我将其保留在此处以供参考，但其他答案更可取。]

您的代码对于初学者的代码来说看起来不错，我喜欢您的风格。我特别喜欢这条线：

    char * b[3];

不幸的是，对于这个特定的应用程序，您必须用不太优雅的行代替它，例如

    char b[3][5];

前一行不为您的输出保留任何存储空间。后一行每十六进制保留五个字节。您需要某种方式的存储空间。