我的算法中的缺陷在哪里找到包含精确字符集的最小子字符串的大小?

时间:2016-08-12 14:46:14

标签: c# string algorithm time-complexity

设置是我有一个像

这样的字符串
s = "GAAATAAA" 

之类的字典
surpdic = { 'A' -> 4 }

字典的含义是s4个剩余A个字符。

我的算法试图找到的是包含s 4 s的A的最小子字符串的大小。我无法弄清楚为什么它不适用于所有的测试用例。

它应该像

一样工作
GAAATAAA
 ||| |||
 i|| j||  j - i = 4, mindiff = 4
  ||  ||
  i|  j|  j - i = 4, mindiff = 4
   |   | 
   i   j  j - i = 4, mindiff = 4

我提供的例子。换句话说,在字符串中从左到右,找到包含所有字符的第一个跨度,然后有效地取出左指针并将其移动到下一个可能跨度的开头;一直跟踪最小跨度。

int mindiff = Int32.MaxValue; 
int left = 0; 
while(!surpdic.ContainsKey(s[left++]));
for(int right = left; right < s.Length; ++right) 
{                   
    if(surpdic.ContainsKey(s[right])) 
        surpdic[s[right]] -= 1; 
    if(surpdic.Values.All(count => count == 0)) 
    {
        int diff = right - left; 
        if(diff < mindiff) 
            mindiff = diff;

        surpdic[s[left]] += 1; 
        while(!surpdic.ContainsKey(s[left++])); 
    }
}

编辑:所以这是一个给我一个运行时错误的案例。

using System.IO;
using System.Linq;
using System.Text;
using System;
using System.Collections.Generic;
public class Solution
{   

    static int SmallestSubstringContainingChars(string source, Dictionary<char,int> surpdic)
    {
        int mindiff = Int32.MaxValue; 
        int left = 0; 
        while(!surpdic.ContainsKey(source[left++]));
        for(int right = left; right < source.Length; ++right) 
        {                   
            if(surpdic.ContainsKey(source[right])) 
                surpdic[source[right]] -= 1; 
            if(surpdic.Values.All(count => count == 0)) 
            {
                int diff = right - left; 
                if(diff < mindiff) 
                    mindiff = diff;

                surpdic[source[left]] += 1; 
                while(!surpdic.ContainsKey(source[left++])); 
            }
        }
        return mindiff + 1;
    }

    public void Main(string[] args)
    {
        string s = "ACTGATTT";
        Dictionary<char,int> d = new Dictionary<char,int>() { { 'A' , 1 }, { 'T' , 3 } };
        Console.WriteLine( SmallestSubstringContainingChars(s,d));
    }
}

1 个答案:

答案 0 :(得分:0)

我在你的代码中发现了很多问题。

首先是你向左递增的方式。

此外,您正在修改字典,因此下一次执行将以错误的字典开头,这就是我创建副本的原因。

如果字典条目的值已经为0,则不要递减字典条目,否则您的All代码将无效。

检查以下代码,如果您不了解任何内容,请在下方发表评论。如果找不到所需的模式,它将返回-1:

static int SmallestSubstringContainingChars(string source, Dictionary<char, int> surpdic)
{
    int mindiff = -2;

    int left = 0;
    while (left<source.Length)
    {
        if (surpdic.ContainsKey(source[left]))
        {
            Dictionary<char, int> md = new Dictionary<char, int>(surpdic);
            md[source[left]] -= 1;
            for (int right = left; right < source.Length; ++right)
            {
                if (md.ContainsKey(source[right]) && md[source[right]]>0)
                    md[source[right]] -= 1;
                if (md.Values.All(count => count == 0))
                {
                    int diff = right - left;
                    if (mindiff==-2 || diff < mindiff)
                        mindiff = diff;
                }
            }
        }
        left++;
    }
    return mindiff + 1;
}