一道面试题引发的思考

1.昨天（10月10日）腾讯一面，一道简单的面试题，本以为写得还不错，结果今天回来仔细思考并coding，发现此程序中漏洞多多。

2.题意：删除一个字符串中重复出现的字符，只留下第一次出现的字符。如：输入abaacdba，输出abcd。

3.思路：定义一个数组来存放每个字符出现的次数，如果发现字符已经出现，则删除该字符。

4.代码：

（1）当时写的版本：

char* DeleteFun(char* str)
{
	int times[WordNumber];
	for (int i = 0; i < WordNumber; i++)
	{
		times[i] = 0;
	}

	char* p = str;
	int index;
	while (*p != '\0')
	{
		index = *p - 'a';
		if (times[index] == 0)
		{
			times[index]++;
		}
		p++;
	}

	p = str;
	char* ret = str;
	
	while (*p != '\0')
	{
		index = *p - 'a';
		if (times[index] != 0)
		{
			times[index] = 0;
			*ret = *p;
			ret++;
		}
		p++;
	}
	ret++;
	*ret = '\0';

	return str;
}

（2）当时检查了一遍，发现没有加 *ret = '\0'这一句，加上之后自以为还比较满意。

5.回来发现的问题

（1）最后ret++是多余的，有错的，加了这一句会多一个字符；

（2）时间花费了2*N，开销有点大；

（3）这里只考虑了26个小写字母，而没有考虑大写字母和特殊字符。

6.刚刚仔细想想，并且在与kk同学的讨论中，把程序改进了一下，如下：

char* DeleteFun1(char* str)
{
	int times[256];
	//memset(times, 0, 256 * sizeof(int));
	memset(times, 0, sizeof(times));

	char *p = str;
	char *q = str;
	while (*p != '\0')
	{
		if (times[*p] != 0)
		{
			p++;
		}
		else
		{
			times[*p] = 1;
			*q = *p;
			p++;
			q++;
		}
	}
	*q = '\0';
	return str;
}

几点思考：

（1）出现次数数组大小为256，可以容纳所有的大小写字母和特殊字符；

（2）初始化使用memset，比用for循环效率高，而且写得更简单；

（3）只要用一个循环即可，即扫描的时候即判断这个字符是否是出现了，如果没有出现则留下这个字符，如果已经出现了则直接pass掉；

（4）不用加index这个变量，也不用做操作*p - 'a'，可以直接用*p来作下标（因为开了256大小的数组）；

（5）sizeof(times)的大小和256 * sizeof(int)的大小是一样的。