Question

作为Python的新手，我编写了一个工作函数，它将比较两个字符串并搜索两个字符串共享的最长子字符串。例如，当函数比较“goggle”和“google”时，它会将“go”和“gle”标识为两个常见的子串（不包括单个字母），但只会返回“gle”，因为它是最长的一个。 / p>

我想知道我的代码的任何部分是否可以改进/重写，因为它可能被认为是冗长和复杂的。我也很高兴看到解决方案的其他方法。提前谢谢！

def longsub(string1, string2):
    sublist = []
    i=j=a=b=count=length=0

    while i < len(string1):
        while j < len(string2):
            if string1[i:a+1] == string2[j:b+1] and (a+1) <= len(string1) and (b+1) <= len(string2):
                a+=1
                b+=1
                count+=1
            else:
                if count > 0:
                    sublist.append(string1[i:a])
                count = 0
                j+=1
                b=j
                a=i
        j=b=0
        i+=1
        a=i

    while len(sublist) > 1:
        for each in sublist:
            if len(each) >= length:
                length = len(each)
            else:
                sublist.remove(each)

    return sublist[0]

编辑：比较“goggle”和“google”可能是一个不好的例子，因为它们的长度相等，同一位置的公共段最长。实际输入将更接近于此：“xabcdkejp”和“zkdieaboabcd”。正确的输出应该是“abcd”。

Answer 1

在标准库中实际上恰好有一个函数：difflib.SequencMatcher.find_longest_match

Answer 2

编辑：此算法仅在单词在相同索引中具有最长段时才有效

你只能用一个循环逃脱。使用辅助变量。像这样的东西（需要重构）http://codepad.org/qErRBPav：

word1 = "google"
word2 = "goggle"

longestSegment = ""
tempSegment = ""

for i in range(len(word1)):
    if word1[i] == word2[i]:
        tempSegment += word1[i]
    else: tempSegment = ""

    if len(tempSegment) > len(longestSegment):
        longestSegment = tempSegment

print longestSegment # "gle"

编辑：mgilson建议使用find_longest_match（适用于不同的细分位置）：

from difflib import SequenceMatcher

word1 = "google"
word2 = "goggle"

s = SequenceMatcher(None, word1, word2)
match = s.find_longest_match(0, len(word1), 0, len(word2))

print word1[match.a:(match.b+match.size)] # "gle"

Python：比较两个字符串并返回它们共有的最长段

2 个答案: