【问题标题】:Boyer moore algorithm - count all matching substringsBoyer moore 算法 - 计算所有匹配的子串
【发布时间】:2019-08-04 20:53:28
【问题描述】:

我正在python中实现boyer moore算法,我需要计算一个子字符串在一个字符串中出现的次数。

我的字符串存储在一个向量中:

string = ['A', 'B', 'B', 'C', 'F', 'D', 'B', 'B']

针也是一个向量:

needle = ['B', 'B']

我的问题是:

我实现的算法只返回第一次出现针头的索引。 在上面的例子中,它返回1,但是,正如我之前所说,我需要每次针是否出现在数组中,并期望它返回2;

示例和预期回报

1

haystack = ['A', 'B', 'B', 'C', 'F', 'D', 'B', 'B']
needle = ['B', 'B'] 
expected return = 2

2

haystack = ['A', 'F', 'T', 'C', 'B', 'F', 'T', 'a', 'F', 'T']
needles = ['F', 'T']
expected return = 3

我的代码我 在search 函数中,我尝试用counter += 1 更改return i(索引)来计算匹配的针数,但是,如果我这样做,它会给我以下错误:

Traceback (most recent call last):
  File "booyer-moore.py", line 103, in <module>
    main(files)
  File "booyer-moore.py", line 98, in main
    index = search(haystack, needle)
  File "booyer-moore.py", line 18, in search
    i += max(offset_table[len(needle) - 1 - j], character_table.get(haystack[i]))
IndexError: list index out of range

def search(haystack, needle):
    counter = 0
    if len(needle) == 0:
        return 0
    character_table = createCharacterTable(needle)
    offset_table = createOffsetTable(needle)

    i = len(needle) - 1

    while i < len(haystack):
        j = len(needle) - 1
        while needle[j] == haystack[i]:
            if j == 0:
                return i
                # counter += 1
            i -= 1
            j -= 1
        i += max(offset_table[len(needle) - 1 - j], character_table.get(haystack[i]))
    return counter


def createCharacterTable(needle):
    table = {}
    for i in range(len(needle) - 1):
        table[needle[i]] = len(needle) - 1 - i
    return table


def createOffsetTable(needle):
    table = []
    lastPrefixPosition = len(needle)
    for i in reversed(range(len(needle))):
        if isPrefix(needle, i + 1):
            lastPrefixPosition = i + 1
        table.append(lastPrefixPosition - i + len(needle) + 1)
    for i in range(len(needle) - 1):
        slen = suffixLength(needle, i)
        table[slen] = len(needle) - 1 - i + slen
    return table


def isPrefix(needle, position):
    j = 0
    for i in range(position, len(needle)):
        if needle[i] != needle[j]:
            return False
        j += 1
    return True


def suffixLength(needle, position):
    length = 0
    j = len(needle) - 1
    for i in reversed(range(position + 1)):
        if needle[i] == needle[j]:
            length += 1
        else:
            break
        j -= 1
    return length

【问题讨论】:

    标签: python boyer-moore


    【解决方案1】:

    发现了几个错误,我已在您的解决方案中纠正了这些错误。

    def search(haystack, needle):
        if len(needle) == 0:
            return 0
        character_table = createCharacterTable(needle)
        offset_table = createOffsetTable(needle)
        i = len(needle) - 1
        while i < len(haystack):
            j = len(needle) - 1
            while needle[j] == haystack[i]:
                if j == 0:
                    yield i
                    break
                i -= 1
                j -= 1
            if(character_table.get(haystack[i])):
                i += max(character_table.get(haystack[i]),offset_table[len(needle) - 1 - j])
            else:
                i += offset_table[len(needle) - 1 - j]
    
    haystack = ['A', 'F', 'T', 'C', 'B', 'F', 'T', 'a', 'F', 'T']
    needle = ['F', 'T']
    generator=search(haystack,needle)
    for i in generator:
        print("Result: "+str(i))
    

    返回多个解决方案的问题可以通过使用生成器来解决,即产生值而不是返回它们。

    在您的 character_table.get(haystack[i]) 中添加了一个检查,因为我不断收到此错误,因为您的 character_table 字典中的值在不匹配的情况下不存在。

    i += max(character_table.get(haystack[i]),offset_table[len(needle) - 1 - j])
    
    TypeError: '>' not supported between instances of 'int' and 'NoneType'
    

    结果

    Result: 1
    Result: 5
    

    现在我打印了您的 offsetTable,它给了我 [1, 5] 这对我来说看起来不正确。在我得到的执行过程中打印 i 和 j 的值:

    {'F': 1} [1, 5]
    j is 1
    i is 2
    Result: 1
    j is 0
    i is 6
    Result: 5
    j is 0
    i is 10
    

    所以在索引 6 和 10 之间有一个跳跃。你能解释一下你的偏移表的逻辑吗?如果我能找出逻辑中的任何错误,我会更新这个答案。

    【讨论】:

      【解决方案2】:

      不确定它是否适合之后的max,但我想你应该将i重置为内部启动之前的值(如果你的针大小为2,那么i -= 1出现一次),更新计数器然后休息:

              if j == 0:
                  i += len(needle) - 1
                  counter += 1
                  break
              i -= 1
              j -= 1
      

      【讨论】:

      • 我用这个干草堆试了试:haystack = ['A', 'C', 'B', 'B', 'F', 'D', 'B', 'B', 'B', 'B', 'B', 'B', 'B', 'A', 'B', 'B'] 和这个针:needle = ['B', 'B'],它给了我2 作为答案
      • 不知道,真的,我想你应该对@Axxelerated 的回答做出反应并找到解决问题的方法。
      猜你喜欢
      • 1970-01-01
      • 1970-01-01
      • 1970-01-01
      • 2018-12-16
      • 1970-01-01
      • 1970-01-01
      • 1970-01-01
      • 1970-01-01
      • 1970-01-01
      相关资源
      最近更新 更多