char* f(char * str, char * s1){char * result; int i = 0; char * p; result = strtok(str, s1); while (p != NULL) { cout << result; p = strtok(NULL, s1); strcat(result, p); } return result;}

这段代码存在几个问题。首先，函数没有指定返回值类型，应该在函数名前加上返回类型，比如说char*。其次，变量p在循环体内没有被初始化，应该在循环开始前将其赋值为result。最后，如果输入的字符串中不包含分隔符s1，那么result将会是NULL指针，无法进行strcat操作，需要先对result进行动态内存分配。

以下是修正后的代码：

char* f(char* str, const char* s1) {
    char* result = NULL;
    char* p = NULL;
    int i = 0;
    if (str != NULL) {
        result = strtok(str, s1);
        p = result;
    }
    while (p != NULL) {
        cout << p;
        p = strtok(NULL, s1);
        if (p != NULL) {
            result = (char*)realloc(result, strlen(result) + strlen(p) + 1);
            strcat(result, p);
        }
    }
    return result;
}

需要注意的是，修改后的代码使用了动态内存分配，需要在调用结束后手动释放内存，以免造成内存泄漏。