2014-12-21 150 views
2

在其中一个练习中,我必须计算班级平均值。但我在这里面临一些问题。 计算班级平均数,我必须迭代每个正在通过的学生。 我的get_class_average()函数没有按预期工作。尽管我用两个学生名称(爱丽丝,劳埃德)调用此函数,但只有爱丽丝得到处理,我的答案是来到91.15,而不是85.85 ...Python/Codeacademy:班级平均计算

lloyd = { 
    "name": "Lloyd", 
    "homework": [90.0, 97.0, 75.0, 92.0], 
    "quizzes": [88.0, 40.0, 94.0], 
    "tests": [75.0, 90.0] 
} 
alice = { 
    "name": "Alice", 
    "homework": [100.0, 92.0, 98.0, 100.0], 
    "quizzes": [82.0, 83.0, 91.0], 
    "tests": [89.0, 97.0] 
} 
tyler = { 
    "name": "Tyler", 
    "homework": [0.0, 87.0, 75.0, 22.0], 
    "quizzes": [0.0, 75.0, 78.0], 
    "tests": [100.0, 100.0] 
} 

# Add your function below! 
def average(numbers): 
    total=sum(numbers) 
    total= float(total) 
    result = total/len(numbers) 
    return result 

def get_average(student): 
    homework = 0.10 * average(student['homework']) 
    quizzes = 0.30 * average(student['quizzes']) 
    tests = 0.60 * average(student['tests']) 

    return (homework + quizzes +tests) 


# This section is not working properly  
def get_class_average(students): 
    results =[] 
    for item in students: <-- This loop may not be working 
     print item 
     result =get_average(item) 
     results.append(result) 
     print results 
     return average(results) 


print get_class_average([alice,lloyd]) 
+0

函数方法只能有一个'return'。你的'get_class_average'方法在for循环结尾处有一个return语句。您应该将每个人的平均值存储在临时列表中,然后将其返回。 –

+0

iluengo:这不是真的。函数可以有多个'return'状态。这只是导致错误的特定代码。 – Nebril

+0

嗯,真的,我在这个特殊情况下谈论,而不是一般的编程。顺便说一句,thx纠正。 –

回答

2

你缩进行

return average(results)

是循环的一部分。由于return中断循环(包括for循环),因此只有循环的第一次迭代(即与Alice一起)正在运行。试着把这一行放在与for循环相同的级别上。这样,循环就会完成,并且在你的函数返回结果之后。

-1

您的参数没有定义,但您为学生中的每个项目设置了一个循环,但学生未定义,然后您面临代码问题。

尝试做这样的事情:

students = [lloyd, alice, tyler] 

def get_class_average(students): 
results = [] 
for student in students: 
    results.append(get_average(student)) 
return average(results) 

print get_class_average(students) 

这里我定义我想,学生具有可变的学生计算出平均值,然后用已经是你的一个功能我刚加入的结果,我们的名单在我们的变量“结果”。

+0

如果您向下滚动到问题的最后一行,您可以看到学生来自哪里。 – Seb