请问我的代码还有哪些错误吗？谢谢啊

NYJYA

题目描述
subStr是C++里面的计算一个字符串子串的函数，请实现subStr函数。
给定字符串s，子串的起始位置i和子串长度l, subStr返回从i开始的长度为l的字符串作为子串。
代码

#include<iostream>
using namespace std;
#include<cstring>
#include<iomanip>
char p[1000];
char *substr(char *s,int i,int l)
{
        int k = strlen(s),j = 0,M;
        
        if(i >= k || i < 0)
        strcpy(p,"Not Valid!" );
        else if(l >= (k-i))
        {
                M = i;
                while(s[M] != '\0')
                {
                        p[j] = s[M];
                        j++;
                        M++;
                }
                p[j] = '\0';
        }
        else 
        {
                M = i;
                while(j != l)
                {
                        p[j] = s[M];
                        j++;
                        M++;
                }
                p[j] = '\0';
        }
        return p;
}
int main()
{
        char s[1000],a[1000];
        int i,l;
        cin >> s;
        cin >> i >> l;
        strcpy(a,substr(s,i,l));
        cout << a << endl;
        return 0;
} 

Mondayisgood

p[1000] 需要初始化
s[1000],a[1000] 需要初始化
比如
char p[1000] = {0};

NYJYA

Mondayisgood 发表于 2019-7-27 10:46
p[1000] 需要初始化
s[1000],a[1000] 需要初始化
比如

初始化后还是出现错误