C语言：命令行传入字符串转数组及打印无输出问题排查

咱们直接说问题核心：你的代码现在没输出，主要是赋值逻辑错了+完全没写输出代码，咱们一步步拆解修复：

你的代码里的关键问题 #

• 循环索引完全错误：在for循环里每次都把i初始化为0，这意味着所有的sprintf操作都在覆盖m[0]这一个元素，后面的参数会把前面的内容冲掉，数组其他位置根本没用到。
• 没有输出逻辑：代码只做了结构体数组的赋值，但完全没写printf这类输出语句，哪怕赋值成功，你也看不到任何内容。
• 语法错误：sprintf(m[i].m, "%s.csv", argv[z])...里的...是无效语法，必须删掉才能正常编译。

修正后的完整代码 #

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

typedef struct { 
    char m[9]; 
} moeda;

int main(int argc, char *argv[]){
    int n = argc - 1;
    // 处理无参数传入的边界情况
    if (n == 0) {
        printf("请传入至少一个货币代码参数\n");
        return 1;
    }

    moeda *m = (moeda*)malloc(n * sizeof(moeda));
    // 检查内存分配是否成功
    if (!m) {
        perror("malloc分配内存失败");
        return 1;
    }

    for(int z = 1; z <= n; z++) {
        // 用z-1作为数组索引，对应每个传入的参数
        int i = z - 1;
        // 用snprintf替代sprintf，避免字符串超长溢出结构体数组
        snprintf(m[i].m, sizeof(m[i].m), "%s.csv", argv[z]);
        // 打印当前元素，实现你要的输出效果
        printf("%s ", m[i].m);
    }
    // 换行让输出更整洁
    printf("\n");

    // 释放动态分配的内存，避免内存泄漏
    free(m);
    return 0;
}

额外优化说明 #

• 把sprintf换成snprintf：你的结构体里m是9个字符的数组，snprintf会自动截断超长内容并添加字符串结束符，避免内存溢出问题。
• 加了边界检查：处理无参数、内存分配失败的情况，让程序更健壮。
• 释放内存：避免长期运行的内存泄漏问题。

测试结果 #

编译后执行./a.out BTC IOT NEO ETH XRP，会输出：

BTC.csv IOT.csv NEO.csv ETH.csv XRP.csv 

如果只需要前两个结果，把循环条件改成z <= 2即可。

内容的提问来源于stack exchange，提问作者D Lorens