Question

我使用python进行了一些尝试，虽然 encrypt（）函数可以正常工作，但 decrypt（）函数却没有任何输出，甚至没有错误：（< / p>

我的代码：

import os
abc=['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z', ' ', '.', ',', '1', '2', '3', '4',   '5', '6', '7', '8', '9', '0', '+', '-', ':', "'"]
mixed=abc[::-1]
os.system("clear")
def menu():
    print "-----------"
    print "[1] Encrypt"
    print "[2] Decrypt"
    print "-----------"
    if input(">>> ")==1:
        encrypt()
    elif input(">>> ")==2:
        decrypt()

def encrypt():
    os.system('clear')
    text=raw_input(">>> ").lower()
    text=list(text)
    textnew=text
    for i in range(len(text)):
        textnew[i]=mixed[abc.index(text[i])]
    print ''.join(textnew)
    menu()

def decrypt():
    os.system('clear')
    text=raw_input(">>> ").lower()
    text=list(text)
    textnew=text
    for i in range(len(text)):
        textnew[i]=abc[mixed.index(text[i])]
    print ''.join(textnew)
    menu()

menu()

Answer 1

以下是一些建议：

使用raw_input
不要重复自己-不需要4个打印语句
您的函数会显示相同的提示。这使用户感到困惑。至少添加一些关于正在发生的事情的指示
与其递归地调用main()（最终达到堆栈可以调用的函数数量的极限），不如使main()拥有一个循环，每次都调用您的函数。
养成编写if __name__ = '__main__':的习惯，因为当您开始编写Python模块时，您不希望脚本在导入时运行所有内容，而只是向调用脚本提供功能。
以80个字符换行

修改后的代码：

import os
abc=['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 
     'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z', ' ', '.', 
     ',', '1', '2', '3', '4',   '5', '6', '7', '8', '9', '0', '+', '-', 
     ':', "'"]
mixed=abc[::-1]
os.system("clear")

def menu():
    header = '\n'.join(["-----------","[1] Encrypt",
                        "[2] Decrypt","-----------"])
    while True:
        print header
        user_input = raw_input(">>> ")
        # print "DEBUG:user input:",user_input
        if user_input == '1':
            encrypt()
        elif user_input == '2':
            decrypt()
        elif user_input == 'q':
            exit()
        else:
            print("Bad input")

def get_input():
    os.system('clear')
    print "Encrypt"
    text=raw_input(">>> ").lower()
    return list(text)


def encrypt():
    text = get_text()
    textnew=text
    for i in range(len(text)):
        textnew[i]=mixed[abc.index(text[i])]
    print ''.join(textnew)

def decrypt():
    text = get_input()
    textnew=text
    for i in range(len(text)):
        textnew[i]=abc[mixed.index(text[i])]
    print ''.join(textnew)

if __name__ = '__main__':    
    menu()

几乎相同的功能，但是不起作用？

1 个答案: