我在使用两个函数编码时遇到了一些问题

这里有一些建议/和修复。还有很多东西可以改进，但我想展示一些基本的东西，希望它们不会太复杂而难以理解，但可以使您的代码更好。这个想法是编写易于阅读和更改的代码。usd = 0.000905eur = 0.000807350908yen = 0.0919061643rmb = 0.00603703605def print_choices():    """    This is a doc string. Here you should describe what your function is doing.    This function prints the choices, that a user can make    """    print("Please choose which currency you want to convert:")    # DRY: (=Don't Repeat Yourself). you have the exchange rate already in    # variables. so use them, so that if the exchange rate changes you     # need to change only one line in your code.    print("A - Korean Won to US Dollar (Exchange Rate: %f)" % usd)    print("B - Korean Won to Euro (Exchange Rate: %f)" % eur)    print("C - Korean Won to Japanese Yen (Exchange Rate: %f)" % yen)    print("D - Korean Won to Chinese RMB (Exchange Rate: %f)" % rmb)    print("E - Quit")    # describe what happens if you enter another value    # lateron you might want to rewrite your code such, that it rejects     # any option, that is not A, B, C, D, E and asks for input until it is    # correct.    print("any other input will Quit ")# variables A,B,C,D are not used in your code, so remove them# function names should only use lowercase characters and '_' # This is a way of telling others, that this is a variable or a function# It is got practice, that function names start with a verb and then an object.def get_user_input():    """     This function prompts the user for an option and an amount in    Korean Won and returns it to the caller.    returns: options, amount    """    option = input("Enter your option: ")    # convert option to uppercase, so even if the user enters 'a',     # option 'A' will be chosen    option = option.upper()    # eval should not be used it is a dangerous function. Use float instead    amount = float(input("Enter the amount in Korean Won: "))    return option, amountdef calculate_and_print_converted_amount(option, amount):    """     depending on option, amount is converted to another currency    This function calculates and displays the converted amount    and the currency.     If option "E" is selected Nothing is displayed.    if an unknown option is selected a small warning will be displayed    and no currency conversion is performed.    """    if option == "A":        conversionrate = usd        currency = "USD"    elif option == "B":        conversionrate = eur        currency = "Euro"    elif option== "C":        conversionrate = yen        currency = "Yen"    elif option== "D":        conversionrate = rmb        currency = "Chinese RMB"    elif option== "E":        return    else:        # quit is no python command use return instead to return from        # a function.        # alternatively you can exit the entire python script         # with sys.exit()        print("unknown option %s. Will quit" % option)        return    # Use only one print statement, so you can change formatting by changin    # only one line    totalamount = amount * conversionrate    print ("%.2f Won equals to %.2f %s" % (amount, totalamount, currency))def main():    print_choices()    (option, amount) = get_user_input()    calculate_and_print_converted_amount(option, amount)    # if you wanted to you could have one function (calculate_amount),     # that just calculates the amount and returns the amount and the     # currency and another function printing the answer.    # you see that many programs separate 'calculation' from 'presentation'    # one function does the work, the other one decides how to 'present'     # the information. (print to the terminal, write to a file, display     # with a GUI framework.    # If you'd separate code like that you could re use the calculation if    # you change your user interface and you just had to reimplement the    # function presenting the resultmain()

我在使用两个函数编码时遇到了一些问题

3回答