4 回答

TA貢獻1765條經驗 獲得超5個贊
恭喜你學習了Python!我認為你的代碼有點混亂。我將發布您的版本以及我的一些評論,然后我將發布一個清理版本,我認為它可以更好地實現您想要做的事情。希望它具有指導意義。有更優雅的方法可以做到這一點,但我試圖保留你的結構盡可能有意義。
您的代碼:
user_input = (input("Enter three numbers separated by a space: ")).split()
x = int(user_input[0])
y = int(user_input[1])
z = int(user_input[2])
def calculate(x, y, z):
c = ((x/y) + z) if y != 0 else print("Second input value cannot be 0") #Why do you need a ZeroDivisonError check below when you check for Zero here?
return c
try:
input_values_str = str(user_input) # why are you taking an input here? Did you take the input above? And why convert it back into a string?
c = ((x/y) + z)
for val in input_values_str: # Why are you iterating through string? You're basically checking 3 times if the length of your string is 3 characters
if len(user_input) == 3:
print("Correct number of values.")
else:
print("Incorrect number of values entered.")
except ValueError:
print(user_input," is not valid input.")
except ZeroDivisionError:
y = 0 # why set y back to zero in the event oa ZeroDivisonError? Won't that just create a ZeroDivison Error? Might make more sense to get a new input from the user
print("Second value cannot be 0")
print("Formula: ({}/{}) + {} = {}".format(x, y, z, calculate(x, y, z))) # if the objective is to only print something, why not do it in the calculate function? Then we can contain your ZeroDivisionError in their and request a new y
我的代碼:
def calculate(x, y, z):
try:
solution = (x/y) + z
print(f"Formula: ({x}/{y}) + {z} = {solution}")
except ZeroDivisionError:
y = int(input("2nd value cannot be a Zero. Please input a new y value: "))
calculate(x, y, z)
try:
user_input = (input("Enter three numbers separated by a space: ")).split()
while len(user_input) != 3:
print("Incorrect number of values entered.")
user_input = (input("Enter three numbers separated by a space: ")).split()
x = int(user_input[0])
y = int(user_input[1])
z = int(user_input[2])
calculate(x, y, z)
except ValueError:
print(user_input," is not valid input.")

TA貢獻1836條經驗 獲得超4個贊
我只是調整了您的代碼以滿足您的期望,并且易于理解。
user_input = (input("Enter three numbers separated by a space: ")).split()
x = int(user_input[0])
y = int(user_input[1])
z = int(user_input[2])
try:
c = ((x/y) + z) if y != 0 else print("Second input value cannot be 0")
except ValueError:
print(user_input," is not valid input.")
print("Formula: ({}/{}) + {} = {}".format(x, y, z, c))

TA貢獻1863條經驗 獲得超2個贊
我根本不明白為什么你需要 try/ except/ except 塊。y=0您已經在您的方法中進行了檢查calculate,并且您已經知道您有有效的輸入,否則 和split強制轉換int()將會失敗。其中的循環for一遍又一遍地打印相同的輸出字符串。如果你把所有這些都去掉,你的代碼似乎工作正常
try:
file_name = open('/tmp/data.txt', 'r')
except FileNotFoundError:
print("File could not be found. Please check spelling of file name!")
sys.exit()
#Read lines in file
Lines = file_name.read().splitlines()
user_input = (input("Enter three numbers separated by a space: ")).split()
x = int(user_input[0])
y = int(user_input[1])
z = int(user_input[2])
def calculate(x, y, z):
'''calculate (x/y)+z'''
c = ((x / y) + z) if y != 0 else print("Second input value cannot be 0")
return c
print("Formula: ({}/{}) + {} = {}".format(x, y, z, calculate(x, y, z)))

TA貢獻1843條經驗 獲得超7個贊
您引發了一個異常來查看 y == 0 是否,但第一個值 (x) 也不能為 0。并且您需要將 2 個 if 語句放在 c = ... 之前
嘗試這個:
user_input = (input("Enter three numbers separated by a space: ")).split()
x = int(user_input[0])
y = int(user_input[1])
z = int(user_input[2])
def calculate(x, y, z):
'''calculate (x/y)+z'''
if y == 0 or x == y:
print("Invalid Input")
else:
c = ((x/y) + z)
return c
try:
input_values_str = str(user_input)
c = ((x/y) + z)
for val in input_values_str:
if len(user_input) == 3:
print("Correct number of values.")
else:
print("Incorrect number of values entered.")
except ValueError:
print(user_input," is not valid input.")
except ZeroDivisionError:
y = 0
x = 0
print("Second value cannot be 0. First Value cannot be 0")
print("Formula: ({}/{}) + {} = {}".format(x, y, z, calculate(x, y, z)))
添加回答
舉報