Question

我正在尝试将int数组转换为char指针字符串（将数组值转换为十六进制）。我正在使用代码块编辑器。
所以，

                           int arr[4] = {30, 40, 15, 205};

应转换为

                            char *str =  "1e280fcd";

我已经编写了以下程序：

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

int main()
{
    char *cur_str, *prev_str;
    char *comp_str, *comp_str_copy;
    int arr[] = {30,40,15,205}, i, j, arr_length;
    arr_length = (sizeof(arr)/sizeof(arr[0]));
    prev_str = malloc(sizeof(0));
    cur_str = malloc(sizeof(0));
    comp_str = malloc(sizeof(0));
    for(i=0;i<arr_length;i++)
    {
        cur_str = malloc(2);
        sprintf(cur_str,"%02x",arr[i]);

        comp_str = malloc(sizeof(cur_str)+sizeof(prev_str));
        sprintf(comp_str,"%s%s",prev_str,cur_str);
        printf("%s\n",comp_str);

        free(prev_str);
        prev_str = malloc(sizeof(comp_str));
        sprintf(prev_str,"%s",comp_str);

        if(i==(arr_length-1)){
            comp_str_copy = malloc(sizeof(comp_str));
            sprintf(comp_str_copy,"%s",comp_str);
        }

        free(comp_str);
        free(cur_str);
    }
    printf("%s %d\n",comp_str_copy,strlen(comp_str_copy));
    return 0;
}

该程序的输出为

分段错误或
在初始位置具有垃圾值的字符串

我已经在不同的在线编译器上运行了相同的程序。它们都给出正确的字符串作为输出。编辑器是我正在使用问题还是内存管理方法？

Answer 1

在

    cur_str = malloc(2);
    sprintf(cur_str,"%02x",arr[i]);

sprintf 写入3个字符，包括最后一个空字符，而您只分配了2个

在

comp_str = malloc(sizeof(cur_str)+sizeof(prev_str));

分配长度不正确，因为size_of不能返回您期望的值，必须是

comp_str = malloc(strlen(cur_str)+strlen(prev_str)+1);

但是当然，假设prev_str也是开头的正确字符串，事实并非如此

这两个 malloc 会产生内存泄漏，因为没有空闲空间（未使用）

cur_str = malloc(sizeof(0));
comp_str = malloc(sizeof(0));

为什么不使用 realloc 来增加prev_str的大小？

请注意，最终所需的大小很容易知道：sizeof(arr)/sizeof(arr[0]) * 2 + 1如果数字限制为255（六位数为2位）

提案（不假定所有数字<256）：

#include <string.h>
#include <stdlib.h>

int main()
{
    int arr[] = {30,40,15,205};
    size_t arr_length = sizeof(arr)/sizeof(arr[0]);
    size_t i;
    size_t sz = 0;
    char * r = malloc(0);

    for (i=0; i!= arr_length; ++i)
    {
      char s[20];
      int l = sprintf(s, "%02x",arr[i]);

      r = realloc(r, sz + l + 1);
      strcpy(r + sz, s);
      sz += l;
    }
    printf("%s %d\n", r, sz);
    free(r);
    return 0;
}

编译和执行：

pi@raspberrypi:/tmp $ gcc -g -pedantic -Wextra a.c
pi@raspberrypi:/tmp $ ./a.out
1e280fcd 8

在 valgrind

下执行

pi@raspberrypi:/tmp $ valgrind ./a.out
==2985== Memcheck, a memory error detector
==2985== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==2985== Using Valgrind-3.13.0 and LibVEX; rerun with -h for copyright info
==2985== Command: ./a.out
==2985== 
1e280fcd 8
==2985== 
==2985== HEAP SUMMARY:
==2985==     in use at exit: 0 bytes in 0 blocks
==2985==   total heap usage: 6 allocs, 6 frees, 1,048 bytes allocated
==2985== 
==2985== All heap blocks were freed -- no leaks are possible
==2985== 
==2985== For counts of detected and suppressed errors, rerun with: -v
==2985== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 6 from 3)

Answer 2

这是一个简单任务的大量动态内存分配。如果数组和字符串将保持“小”状态，请使用堆栈：

#include <assert.h>
#include <stdio.h>

int main() {
    int const arr[] = {30, 40, 15, 205};
    int const arr_length = sizeof(arr) / sizeof(arr[0]);
    char str[2 * arr_length + 1];
    int len = 0;
    for (int i = 0; i < arr_length; i++) {
        len += sprintf(&str[len], "%02x", arr[i]);
    }
    assert(len == 2 * arr_length);
    printf("%s %d\n", str, len);
    return 0;
}

但是，如果您确实需要动态字符串，即char *str只需将char str[2 * arr_length + 1];修改为

char *str = malloc(2 * arr_length + 1);

并添加free(str);

NB：所有这些都假定您的整数数组值小于256。

Answer 3

以下建议的代码：

消除了不需要的代码逻辑
消除不需要的变量
消除了对动态内存的不必要使用
干净地编译
没有段错误
执行所需的功能
请注意，strlen()返回size_t，而不是int
请注意，sizeof()返回的是size_t，而不是int
将数组arr[]的长度乘以2以计算显示转换后的数组所需的字符数
在计算出的长度上加1以允许尾随NUL字节
按必要的顺序列出变量，以便在需要时始终可以使用特定的参数

现在，建议的代码：

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

int main( void )
{
    int arr[] = {30,40,15,205};
    char comp_str[ sizeof( arr )*2 +1 ] = {'\0'};  

    size_t arr_length = (sizeof(arr)/sizeof(arr[0]));

    for( size_t i=0; i<arr_length; i++ )
    {          
        sprintf( comp_str,"%s%02x", comp_str, arr[i] );
        printf("%s\n",comp_str);
    }
}

运行建议的代码将导致：

1e
1e28
1e280f
1e280fcd

指向char指针字符串的整数数组

3 个答案: