如何使这段 Python 代码更可用和可读？

Question

python 初学者，但已经编程了大约 5 年。我怀疑关于以面向对象的方式做事我还有很多东西需要学习，但我知道基础知识。我计划编写一个计算器，以显示它可以应对挑战并从中获得知识。我刚刚开始，这就是我所拥有的，它对我来说看起来真的很难看。你会如何做？

PS 这只是一个简单的脚本，用于从括号内取出问题，将其相加，展示工作，然后评估整个问题。

import re

def EvalParenths(problem):
    contents = ""
    if re.match( "\(", problem):
        contents = re.match("(\(.*\))", problem)
        parenthsAnswer = contents.group(0)
        problem = problem.replace(parenthsAnswer, '')
        print "   \ \n   "  + str(eval(parenthsAnswer)) + problem
        problem = problem.replace(parenthsAnswer, '')
        answer = eval(parenthsAnswer+problem)
        print "    \ \n    " + str(answer)
    else:
        print "Didn't Find Parenthesis"

def ProblemHasParenths(problem):
    return re.match( "\(", problem)

"""""
Example Problem: (12/4)*2

"""""

problem = raw_input()

if ProblemHasParenths:
    EvalParenths(problem)

Answer 1

一些问题：

contents = re.match("(\(.*\))", problem)

当给定输入(1+2)/(3+4)时，它将尝试评估1+2)/(3+4。

它也不会一直进入嵌套括号，为此您需要使用递归，

我认为您应该在“查看答案”之前再次尝试。

Answer 2

如果您想制作一个简单的计算器，您可以尝试实现Shunting-yard 算法。

但是，如果您想使用正则表达式方法，我仍然会做一些不同的事情：

import re

#In python functions/methods usually are lowercase
#and words are seperated by _ while classes use CamelCasing
def eval_step_by_step(expression):
    """Evaluates math expression. Doesn't do any error checking.
        expression (string) - math expression"""

    print expression
    #For pretty formating.
    expr_len = len(expression)
    #While there's parentheses in the expression.
    while True:
        #re.match checks for a match only at the beginning of the string,
        #while re.search checks for a match anywhere in the string.

        #Matches all numbers, +, -, *, / and whitespace within parentheses
        #lazily (innermost first).
        contents = re.search("\(([0-9|\*|/|\+|\-|\s]*?)\)", expression) 
        #If we didn't find anything, print result and break out of loop.
        if not contents:
            #string.format() is the Python 3 way of formating strings
            #(Also works in Python 2.6).

            #Print eval(expression) aligned right in a "field" with width
            #of expr_len characters.
            print "{0:{1}}".format(eval(expression), expr_len)
            break

        #group(0) [match] is everything matching our search,
        #group(1) [parentheses_text] is just epression withing parentheses.
        match, parentheses_text = contents.group(0), contents.group(1)
        expression = expression.replace(match, str(eval(parentheses_text)))
        #Aligns text to the right. Have to use ">" here
        #because expression is not a number.
        print "{0:>{1}}".format(expression, expr_len)

#For example try: (4+3+(32-1)*3)*3
problem = raw_input("Input math problem: ")

eval_step_by_step(problem)

它的工作原理与您的函数并不完全相同，但您可以轻松地对函数进行修改以匹配我的函数。正如您所看到的，我还添加了很多注释来解释一些内容。

Answer 3

我可能会将出现的 替换

re.match( "\(", problem)

为

problem.startswith("(")

在

contents = re.match("(\(.*\))", problem)
parenthsAnswer = contents.group(0)

您不检查内容是否匹配的情况下，因此如果您将输入“(1”传递给它，那么当您尝试评估内容时会出现异常。group(0)

Don在真实的程序中并不是每个都使用 eval ！

您可以使用 pyparsing 来制作一个完整的解析器，但我认为每个人都应该自己尝试至少一次作为练习！

Answer 4

为什么不只匹配双括号和匹配括号？单个 ( 的第一个匹配并不是真正必要的，因为如果双打的匹配失败，这意味着没有表达式可供您评估。

import re

def eval_parentheses(problem):
    contents = re.match("(\(.*\))", problem)
    if contents:
    ...
    else:
        print "Couldn't find parentheses!"

此外，括号选择算法可以稍微改进嵌套括号等