不用这么麻烦吧！不知道你这个gets(s1)是干什么的，最好不要用gets();

求数组s1的长度用sizeof()就行了,前面把s2初始化了，后面就不用在最后加“\0”了

初始化很重要，边界操作容易出错。

 char s1[10] = "abcdefg";
 char s2[30] = {};
 int i, j=0, k=0;
 scanf("%d", &k);
 j = sizeof(s1);
 if (k > j)
 {
      k = j;
 }

for (i = 0;i < k;i++)
{
     s2[i] = s1[i];
}

 printf("%s\n", s2);