为什么此代码不适用于所有输入

Question

我正在尝试编写将两个字符串作为输入的 python 代码，并检查第二个字符串的所有字符是否都存在于第一个字符串中。如果是这样，那么输出是第二个字符串。如果不是，则输出为字符串“This doesn't work”。我已经针对各种输入测试了此过程，它通常有效，但并非总是如此。例如，如果我的两个输入分别是“helo”和“olhe”，则输出是“This doesn't work”，而它应该是“olhe”，因为“olhe”中的所有字符都出现在“helo”中。

这是我的代码

def fix_machine(debris,product):
    n = 1
    while True:
          first_element = product[0:n]
          find_first_element = debris.find(first_element)
          if first_element == product:
               return product
               break
          n = n + 1
          if find_first_element == -1:
               return "This does not work"
               break

那么为什么这不起作用呢？

Answer 1

假设您严格希望所有第二个字符串字符首先出现，您可以将其作为一个更优雅的解决方案。

def fix_machine(first, second):
    for character in second:
        if character not in first:
            return False
    return second

这将为您的代码返回正确的输入。不完全确定您的代码有什么问题，因为我没有逐步完成。

编辑：Albert 有一个更优雅的解决方案，参考他的

Answer 2

据我了解，您只想比较两个包含不同字符的字符串。

为此，我建议将两个字符串都转换为 set，然后提供（与 list 相比）对其元素的无序比较。

def check_string(first_str, second_str):
    # convert strings to sets in order to compare characters
    if set(first_string) == set(second_string):
        return second_str
    else:
        return 'This does not work.'


first_string = 'helo'
second_string = 'olhe'    
print(check_string(first_string, second_string))
# prints True

first_string = 'helo'
second_string = 'hello'    
print(check_string(first_string, second_string))
# prints True

first_string = 'helo'
second_string = 'helofoo'    
print(check_string(first_string, second_string))
# prints 'This does not work.'

Answer 3

我同意您应该使用调试器单步执行该代码（如果您还没有设置 IDE，可以尝试 PyCharm）以查看问题所在。很难解释出了什么问题，但我认为这与first_element = product[0:n] 有关。这将返回越来越大的字符串段。即第二次运行时的“ol”。

这是另一种写法

def fix_machine(debris, product):
    all_present = all(letter in debris for letter in product)
    return product if all_present else 'This does not work'