Question

说明：

编辑：我只需要方法菜单就可以返回“即将推出” - 因为它目前处于当前状态，如果用户输入，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（）函数当前不返回任何内容。我认为你的意思是返回用户的选择。
我在StartMenu（）和MethodMenu（）之间看到一个模式：每个模式都会显示一个选择列表并反复获取用户的输入，直到用户输入正确的输入。但是，StartMenu（）函数不管理用户的输入，而MethodMenu（）执行==＆gt;设计不一致。
由于获取用户输入并验证它的行为发生了两次，因此最好将该代码块移动到一个单独的函数中，您可以调用该函数而不是多次编写相同的代码块。
我注意到单字母变量 a 的用户。我一般，我建议使用更具描述性的名称，例如 user_choice ， user_answer 或 user_input 。

不用多说，我的解决方案：

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循环，这意味着'即将推出'将不会被打印。您可以修改解决方案，也可以使用 hek2mgl 。

Answer 2

遵循逻辑，您应该将函数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或其他什么？！你应该使用富有表现力的变量名！ ;）

需要我的加密程序帮助

2 个答案:

更新