我的加密程序需要帮助吗

Question

说明：

编辑：我只需要方法菜单返回“coming soon”，因为它是当前的状态，如果用户输入c、p或s，它将什么也不返回。我不明白原因。在

def PrintDescription():
    print 'This program encrypts and descrypts messages using multiple \
encryption methods.\nInput files must be in the same directory as this program.\
\nOutput files will be created in this same directory.'

def StartMenu():
    print 'Do you wish to encrypt or decrypt?'
    print '<e>ncrypt'
    print '<d>ecrypt'
    print '<q>uit'

def MethodMenu():
  print 'Which method would you like to use?'
  print '<c>aesarian fixed offset'
  print '<p>seudo-random offset'
  print '<s>ubstitution cipher'
  a = raw_input("")
  while a not in ('c', 'p', 's'):
    if a:
      print "Error: You must type c, p, or s"
      a = raw_input("")
    if a == 'c' or a=='p' or a=='s':
      print 'Coming Soon'         

def main():
    alphabet = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789,.?! \t\n\r"
    PrintDescription()
    a = None
    while a not in ('e', 'd', 'q'):
        if a:
            print "Error: You must type e, d, or q"
        else:
            StartMenu()
        a = raw_input("")
        if a == 'e' or a=='d':
          MethodMenu()
        if a == 'q':
          break  

main()

Answer 1

按照您的逻辑，您应该将函数MethodMenu()更改为：

def MethodMenu():
  print 'Which method would you like to use?'
  print '<c>aesarian fixed offset'
  print '<p>seudo-random offset'
  print '<s>ubstitution cipher'
  a = None
  while a not in ('c', 'p', 's'):
    if a:
      print "Error: You must type c, p, or s"
    a = raw_input("")
    if a == 'c' or a=='p' or a=='s':
      print 'Coming Soon'    

但是为什么要用a而不是user_input或其他什么东西？！你应该使用有表现力的变量名！；）

Answer 2

在我提出我的解决方案之前，这里有一些意见。在

1. MethodMenu（）函数当前不返回任何内容。我想你是想返回用户的选择。在
2. 我看到StartMenu（）和MethodMenu（）之间有一个模式：每个都显示一个选项列表，并反复获取用户的输入，直到用户输入正确的输入。但是，StartMenu（）函数不管理用户的输入，而MethodMenu（）函数在设计上不一致。在
3. 由于获取用户输入并验证它的操作发生了两次，所以最好将代码块移到一个可以调用的单独函数中，而不是多次编写同一代码块。在
4. 我注意到单字母变量a的用户。一般来说，我建议使用更具描述性的名称，如用户选择，用户答案，或用户输入。在

我的解决方案是：

def PrintDescription():
    print 'This program encrypts and descrypts messages using multiple \
encryption methods.\nInput files must be in the same directory as this program.\
\nOutput files will be created in this same directory.'

def GetChoice(acceptable_answers):
    while True:
        user_choice = raw_input('')
        if user_choice in acceptable_answers:
            return user_choice
        else:
            print 'Please try:', ', '.join(acceptable_answers)

def StartMenu():
    print 'Do you wish to encrypt or decrypt?'
    print '<e>ncrypt'
    print '<d>ecrypt'
    print '<q>uit'
    user_choice = GetChoice('edq')
    return user_choice

def MethodMenu():
    print 'Which method would you like to use?'
    print '<c>aesarian fixed offset'
    print '<p>seudo-random offset'
    print '<s>ubstitution cipher'
    user_choice = GetChoice('cps')
    return user_choice

def main():
    alphabet = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789,.?! \t\n\r"
    PrintDescription()

    while True:
        user_choice = StartMenu()
        if user_choice in ('e', 'd'):
            user_choice = MethodMenu()
            # Do something based on the user_choice
        if user_choice == 'q':
            break

main()

更新

如果您必须知道MethodMenu()有什么问题，这里有一个解释：用户第一次键入正确的选择（c、p或s）：整个while循环被跳过，这意味着不会打印“Coming Soon”。您可以修改您的解决方案，或者使用hek2mgl的。在