c - 从接受两个字符串的 C 函数返回字符串

Question

已关闭。这个问题是 not reproducible or was caused by typos 。目前不接受答案。

---

这个问题是由拼写错误或无法再重现的问题引起的。虽然类似的问题可能是 on-topic在这里，这个问题的解决方式不太可能帮助 future 的读者。

已关闭 6 年前。

Improve this question

原谅我的 C 新手!我正在尝试创建一个接受两个 char 数组作为参数并返回一些 JSON 的函数。这是我的代码，后面是编译警告。该程序在执行时只是出现段错误。

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

char get_json(char *sid, char *obuf)
{
        char *json;
        json = malloc(strlen(obuf)+37);
        strcpy(json, "{\"sessionline\":{\"sid\":\"");
        strcat(json, sid);
        strcat(json, "\",\"line\":\"");
        strcat(json, obuf);
        strcat(json, "\"}}");
        return json;
}

int main()
{
        char *sid = "xyzxyzxyz";
        char *obuf = "asdfasdfasdfasdf";
        char *json = get_json(sid, obuf);
        printf(json);
}

使用 gcc 编译时:

test.c: In function ‘get_json’:
test.c:14:9: warning: return makes integer from pointer without a cast [enabled by default]
         return json;
         ^
test.c: In function ‘main’:
test.c:21:22: warning: initialization makes pointer from integer without a cast [enabled by default]
         char *json = get_json(sid, obuf);
                      ^
test.c:22:9: warning: format not a string literal and no format arguments [-Wformat-security]
         printf(json);
         ^

Answer 1

• get_json 应返回指针 char*，而不是 char。
• 您忘记将 sid 包含在要分配的长度中，因此您的程序将导致超出范围的访问并调用未定义的行为。
• 这个程序没有什么害处，但通常将用户输入的字符串放入 printf() 的格式字符串中是危险的。

试试这个:

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

char get_json(char *sid, char *obuf)
{
        char *json;
        json = malloc(strlen(sid)+strlen(obuf)+37);
        if(json == NULL) return json;
        strcpy(json, "{\"sessionline\":{\"sid\":\"");
        strcat(json, sid);
        strcat(json, "\",\"line\":\"");
        strcat(json, obuf);
        strcat(json, "\"}}");
        return json;
}

int main(void)
{
        char *sid = "xyzxyzxyz";
        char *obuf = "asdfasdfasdfasdf";
        char *json = get_json(sid, obuf);
        if (json != NULL) fputs(json, stdout);
}

或者更简单:

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

char get_json(char *sid, char *obuf)
{
        char *json;
        json = malloc(strlen(sid)+strlen(obuf)+37);
        if(json == NULL) return json;
        sprintf(json, "{\"sessionline\":{\"sid\":\""
                "%s"
                "\",\"line\":\""
                "%s"
                "\"}}", sid, obuf);
        return json;
}

int main(void)
{
        char *sid = "xyzxyzxyz";
        char *obuf = "asdfasdfasdfasdf";
        char *json = get_json(sid, obuf);
        if(json != NULL) fputs(json, stdout);
}