1

我想知道是否有办法让这段代码更有效率？只是说我对 python 和整个编程有点陌生。任何提示都会很棒。谢谢是提前。

这是我从这里得到任务的地方：http ://www.101computing.net/how-old-is-your-cat/

该程序只是将猫的年龄转换为人类年龄的形式。

convertedAge = 0
stage = 0

question = input("Is you cat under 1 year old?.. Y/N")

if ((question.lower() == "y") or (question.lower() == "yes")):
  ageOfCat = int(input("How old is your cat (in months)?")) #cat < 1 year old
  if 1 <= ageOfCat <= 2:
    convertedAge = "9 to 10 months"
  elif ageOfCat == 3:
    convertedAge = "2 to 3 years"
  elif ageOfCat == 4:
    convertedAge = "5 to 6 years"
  elif ageOfCat == 5:
    convertedAge = "8 to 9 years"
  elif ageOfCat == 6:
    convertedAge = "10 years"
  elif 7 <= ageOfCat <= 8:
    convertedAge = "13 years"
  elif 8 <= ageOfCat <= 11:
    convertedAge = "14 years"
  print("In human years your cat is the equivalent of " + str(convertedAge) + " old.")
else:
  ageOfCat = int(input("How old is your cat (in years)?")) #cat > 1 year old
  if ageOfCat == 1:
    convertedAge = 15
  elif ageOfCat == 2:
    convertedAge = 15 + 9
  else:
    convertedAge = 15 + 9 + ((ageOfCat-2) * 4)
  print("In human years your cat is the equivalent of " + str(convertedAge) + " years old.")
4

2 回答 2

4

首先，您可以尝试使用字典来消除所有这些 if/else 块：

convertedAges = {
    1: "9 to 10 months",
    2: "9 to 10 months",
    3: "2 to 3 years", # and so on
}

然后使用字典：

convertedAge = convertedAges[ageOfCat]

老实说，您应该专注于可读性，尤其是在您刚刚开始的时候。就像你的第一个if可能只是

if question.lower() in "yes": # "y" is in "yes", so is "yes" (a string is a substring of itself)

如果您开始看到自己一遍又一遍地重复相同（或非常相似）的台词，请停下来想想您要完成的工作。

于 2016-03-25T01:48:47.307 回答
3

您可以使用 alist而不是该if结构：

if ((question.lower() == "y") or (question.lower() == "yes")):
    ageOfCat = int(input("How old is your cat (in months)?")) #cat < 1 year old
    ages = [None,
            '9 to 10 months',
            '9 to 10 months',
            '2 to 3 years',
            '5 to 6 years',
            '8 to 9 years',
            '10 years',
            '13 years',
            '13 years',
            '14 years',
            '14 years',
            '14 years']
    convertedAge = ages[ageOfCat]

然后您可以发送多个参数print()而不是连接（并且您不需要将字符串转换为字符串）：

print("In human years your cat is the equivalent of", convertedAge, "old.")

你可以添加到convertedAge老猫：

else:
    ageOfCat = int(input("How old is your cat (in years)?")) #cat > 1 year old
    convertedAge = 15
    if ageOfCat > 1:
        convertedAge += 9
    if ageOfCat > 2:
        convertedAge += (ageOfCat-2) * 4
print("In human years your cat is the equivalent of", convertedAge, "years old.")
于 2016-03-25T01:54:34.883 回答