Hàm chuỗi chạy sai

newbieIT · November 8, 2018, 2:14pm

Nhờ anh chị coi giúp, e viết hàm thêm dấu “,” vào vị trí k như sau. Chương trình in ra đúng nhưng phía sau còn có thêm một dãy kí tự ascII không mong muốn. Anh chị coi giúp e sai chỗ nào

#include <stdio.h>
#include <string.h>
#define max 30
char* addptu(char *s, int k)
{
	int i;
	for (i = strlen(s) - 1; i >= k; i--)
	{
		s[i + 1] = s[i];
	}
	int x = strlen(s) + 1;
	s[x] = '/0';
	s[k] = ',';
	return s;
}
void main()
{
	char so[max];
	int k;
	gets(so);
	scanf("%d", &k);
	printf("%s", addptu(so,k));
}

rogp10 · November 8, 2018, 2:15pm

Bạn đã ghi đè lên byte cuối cùng thì strlen sau đó sẽ chạy tuốt luôn

Vả lại còn phải xét trường hợp chuỗi ngắn.

newbieIT · November 8, 2018, 2:23pm

là sao anh? e gán ký tự cuối cùng là NULL để kết thúc chuỗi mà?

SITUVN.gcd · November 8, 2018, 2:38pm

Thế mà cũng chạy được á???
Kí tự NULL là '\0' chứ không phải '/0'.
Dấu gạch (chia) ngược.

newbieIT · November 8, 2018, 2:50pm

:)) e đánh nhầm ấy, còn cái ghi đè lên ý a là mình ghi đè lên ký tự null nên nó chạy hết max(30) ký tự phải ko?
Vậy giải pháp là gán cho chuỗi khác hả?

SITUVN.gcd · November 8, 2018, 3:04pm

Trong vòng lặp, bạn đã ghi đè mất kí tự NULL nên khi khi gọi strlen() ở sau thì giá trị này sai.

Chuyển:

lên trước vòng lặp.

newbieIT · November 8, 2018, 3:14pm

oh e hiểu rồi, cám ơn anh nhiều