28讲测试题动手一 不理解自己程序哪里错了

泼墨染笛香

28讲测试题动手一。我的代码在需要替换的字符为'looe'时，未输出'由于替换字符出现冲突，该密文无法解密！'，不理解问题在哪。
题目链接：https://fishc.com.cn/thread-185426-1-1.html
我的代码：

#-- 动手1 最后情况少最后一句输出 不懂
oldSentence = input('请输入需要加密的明文：')
oldWord = input('请输入需要替换的字符：')
newWord = input('请输入将要替换的字符：')
if len(oldWord) != len(newWord):
    print('需要替换的字符数量必须跟将要替换的字符数量一致')
else:
    table = str.maketrans(oldWord,newWord)
    newSentence = oldSentence.translate(table)
    print('加密后的密文是：',newSentence)

def compare(Word):
    for i in range(4):
        for j in range(i+1,4):
            if Word[i] == Word[j]:
                return(1)
            else:
                return(0)

if compare(oldWord)==1 or compare(newWord)==1:
    print('由于替换字符出现冲突，该密文无法解密！')

我的结果：
'

FishC

你的代码逻辑有一点小问题，导致最后一句话没有输出。在你的`compare`函数中，你使用了一个循环嵌套，但是在循环内部却没有对不同的字符进行比较。这样的话，无论输入什么字符，函数都会返回1，所以最后一句话始终不会输出。

修正这个问题很简单，你只需要在循环内部添加对不同字符的比较即可。参考下面修改后的代码：

oldSentence = input('请输入需要加密的明文：')
oldWord = input('请输入需要替换的字符：')
newWord = input('请输入将要替换的字符：')

if len(oldWord) != len(newWord):
    print('需要替换的字符数量必须跟将要替换的字符数量一致')
else:
    table = str.maketrans(oldWord, newWord)
    newSentence = oldSentence.translate(table)
    print('加密后的密文是：', newSentence)

def compare(Word):
    for i in range(len(Word)):
        for j in range(i+1, len(Word)):
            if Word[i] == Word[j]:
                return 1
    return 0

if compare(oldWord)==1 or compare(newWord)==1:
    print('由于替换字符出现冲突，该密文无法解密！')

希望能帮到你！如果还有其他问题，请随时提问。

以上回复来自 FishC AI 小助理，如未能正确解答您的问题，请继续追问。

1q23w31

def compare(Word):
    for i in range(4):
        for j in range(i+1,4):
            if Word[i] == Word[j]:
                return 1
            else:
                return 0

这段代码有问题，compare函数只比对了第一个字符与第二个字符，两者不同直接返回0了，所以后续判断失败，没有输出
具体过程如下：
compare(looe)
比较第一个字符l与第二个字符o
二者不同，返回0
条件判断失败，无输出

小雨初霁天初晴

你compare里的else:
                return(0)不知道有什么用，直接去掉加上continue应该就好了