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

NYJYA

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

1. #include<iostream>
   
2. using namespace std;
   
3. #include<cstring>
   
4. #include<iomanip>
   
5. char p[1000];
   
6. char *substr(char *s,int i,int l)
   
7. {
   
8.         int k = strlen(s),j = 0,M;
   
9.        
   
10.         if(i >= k || i < 0)
    
11.         strcpy(p,"Not Valid!" );
    
12.         else if(l >= (k-i))
    
13.         {
    
14.                 M = i;
    
15.                 while(s[M] != '\0')
    
16.                 {
    
17.                         p[j] = s[M];
    
18.                         j++;
    
19.                         M++;
    
20.                 }
    
21.                 p[j] = '\0';
    
22.         }
    
23.         else
    
24.         {
    
25.                 M = i;
    
26.                 while(j != l)
    
27.                 {
    
28.                         p[j] = s[M];
    
29.                         j++;
    
30.                         M++;
    
31.                 }
    
32.                 p[j] = '\0';
    
33.         }
    
34.         return p;
    
35. }
    
36. int main()
    
37. {
    
38.         char s[1000],a[1000];
    
39.         int i,l;
    
40.         cin >> s;
    
41.         cin >> i >> l;
    
42.         strcpy(a,substr(s,i,l));
    
43.         cout << a << endl;
    
44.         return 0;
    
45. }

复制代码

Mondayisgood

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

NYJYA

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

初始化后还是出现错误