不用這么麻煩吧！不知道你這個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);