截取两个字符串部分内容拼接返回char*无输出，求代码修复

你尝试编写一个函数截取两个字符串的部分内容拼接成新字符串，以char*类型返回，但运行后没有任何输出，而且代码还未完成（*(s1 + ...部分逻辑缺失）。我帮你梳理核心问题并给出完整的修复方案：

核心问题分析 #

你的代码存在几个关键问题，直接导致了无输出或未定义行为：

• 局部数组不能返回：temp是函数内的局部栈数组，函数执行结束后这块内存会被系统释放，返回它的指针会触发未定义行为，后续访问该指针可能导致程序崩溃或无输出。
• 代码逻辑不完整：*(s1 + ...处的循环逻辑未写完，无法正确完成字符串截取。
• 缺少字符串结束符：拼接后的C风格字符串必须以'\0'结尾，否则输出时会读取到垃圾数据或无法识别有效字符串。
• 内存管理缺失：如果要返回自定义字符串，需要使用动态内存分配，确保返回后内存依然有效。

修复后的完整代码 #

我假设你想实现的逻辑是：当s1长度为3的倍数时，截取s1的前1/3部分，再拼接s2的前半部分（你可以根据需求调整截取规则），最后返回拼接后的字符串。以下是修复后的代码：

#include <iostream>
#include <cstring> // 建议用cstring替代string.h，更符合C++标准规范
using namespace std;

char* findpass(char* s1, char* s2) {
    int sl1 = strlen(s1);
    int sl2 = strlen(s2);
    
    // 计算需要分配的总内存：s1截取长度 + s2截取长度 + 1（存储字符串结束符）
    int s1_part_len = sl1 % 3 == 0 ? sl1 / 3 : 0;
    int s2_part_len = sl2 / 2; // 示例：截取s2的前半部分，可按需修改
    int total_len = s1_part_len + s2_part_len + 1;
    
    // 动态分配内存，避免局部数组的内存释放问题
    char* result = new char[total_len];
    int index = 0;
    
    // 截取s1的前1/3部分
    if (sl1 % 3 == 0) {
        for (int i = 0; i < s1_part_len; i++) {
            result[index++] = s1[i];
        }
    }
    
    // 截取s2的目标部分
    for (int i = 0; i < s2_part_len; i++) {
        result[index++] = s2[i];
    }
    
    // 必须添加字符串结束符，确保是合法的C风格字符串
    result[index] = '\0';
    
    return result;
}

int main() {
    char str1[] = "abcdef"; // 长度6，是3的倍数，前1/3为"ab"
    char str2[] = "123456"; // 长度6，前半部分为"123"
    
    char* pass = findpass(str1, str2);
    if (pass != nullptr) {
        cout << "拼接后的字符串：" << pass << endl;
        // 记得释放动态分配的内存，避免内存泄漏
        delete[] pass;
    }
    
    return 0;
}

关键修改点说明 #

1. 动态内存分配：用new char[total_len]分配内存，确保返回的指针在函数结束后依然有效，不会被系统回收。
2. 添加结束符：拼接完成后手动添加'\0'，让cout能正确识别并输出字符串。
3. 内存释放：在主函数中使用完返回的字符串后，用delete[]释放内存，避免内存泄漏。
4. 完善逻辑：补全了字符串截取的循环逻辑，你可以根据实际需求调整s1和s2的截取规则。

为什么之前无输出？ #

即使你补全了循环逻辑，返回局部数组temp的指针后，这块内存已经被系统回收，cout访问该指针时，要么读取到垃圾数据，要么因非法内存访问导致程序无输出甚至崩溃。

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