我正在做一个家庭工作问题,不使用框架方法拆分字符串.
以下是我提出的工作代码.
我想知道如何改善O(n)的运行时间？
此外,欢迎任何改进建议.
- public static string[] split(string txt,char[] delim)
- {
- char[] text = txt.tocharArray();
- string[] result = new string[0];
- int count = 0;
- int i = 0;
- StringBuilder buff = new StringBuilder();
- while(i < text.Length)
- {
- bool found = false;
- foreach(char del in delim)
- {
- if(del == txt[i])
- {
- found = true;
- break;
- }
- }
- if(found)
- {
- count++;
- Array.Resize(ref result,count);
- result[count - 1] = buff.ToString();
- buff = new StringBuilder();
- }
- else
- {
- buff.Append(txt[i]);
- }
- i++;
- }
- if(buff.Length != 0)
- {
- count++;
- Array.Resize(ref result,count);
- result[count - 1] = buff.ToString();
- }
- return(result);
- }
解决方法
我有一些更改会同时使这个函数更像C,并将运行时间减少到O(n)：
1)你应该多次动态调整结果数组的大小,而不是动态地调整结果数组的大小,而应该创建足够的点来保存最大数量的字符串(你知道这个数字小于txt.Length),然后在最后一次调整它的大小.归还它.
2)不要使用StringBuilder组合你的结果,而是制作长度为txt.Length且索引变量为j的char [] buff并执行buff [j] = txt [i].
我认为你的功能应该是O(N).从技术上讲,它将是O(N * M),其中M是分隔符的数量.
编辑1：
这是一个改变,它将使O(N)O(M)而不是O(N * M)：
您应该循环遍历ADVANCE中的分隔符并设置如下所示的数组,而不是循环遍历字符串中每个字符的分隔符：
- bool[] isDelimiter = new bool[128]; // increase size if you are allowing non-ascii
- foreach(char delim in isDelimiter)
- {
- isDelimiter[(int)char] = true;
- }
然后你可以使用这个数组在恒定时间内测试字符串的每个字符.
