maxx 函数有什么问题

Question

我是新来的。我正在尝试回答以下问题。

1. 编写一个程序，生成一个包含 20 个介于 1 和 100 之间的随机数的列表。 (a) 打印清单。 (b) 打印列表中元素的平均值。 (c) 打印列表中的最大值和最小值。 (d) 打印列表中第二大和第二小的条目 (e) 打印列表中有多少个偶数。

我的问题是问题（c）

v=[...]

maxx=v[0]

for i in range(0,len(v)-1):
    if maxx<v[i]:
        maxx=v[i]
print(maxx)

我知道这个算法可以解决我的问题。但是我想了解我的 max 函数有什么问题

import random 

v=[]
sum=0
avg=0
maxx=0
minn=0

def generate(v):
    for i in range(0,20): 
        x=random.randint(1,100) 
        v.append(x)
    return v 
print(generate(v))

def soma(sum):
    for k in range(0,len(v)): 
        sum=sum+v[k] 
    return sum

def average(avg):
    avg=soma(sum)/20 
    return avg
print(average(avg))

def maxx(maxx):
    maxx=v[0] 
    for i in range(0,len(v)-1):
        if v[i+1]>v[i]:
            maxx=v[i+1]
        else:
            maxx=v[i]
    return maxx
print(maxx(maxx))


I expect the function to generate the maximum number in the list and not the last number as it is happening.


import random
v=[]
sum=0
avg=0
maxx=0
minn=0

def generate(v):
    for i in range(0,20): 
        x=random.randint(1,100) 
        v.append(x) 
    return v 
print(generate(v))

def soma(sum):
    for k in range(0,len(v)): 
        sum=sum+v[k]
    return sum

def average(avg):
    avg=soma(sum)/20 
    return avg
print(average(avg))

def maxx(maxx):
    maxx=v[0] 
    for i in range(0,len(v)-1):
        if v[i+1]>v[i]:
            maxx=v[i+1]
        else:
            maxx=v[i]
    return maxx
print(maxx(maxx))

Answer 1

您设置maxx 函数的方式只会让列表的最后一个或倒数第二个元素成为最大值。而不是与v[i] 比较，您需要与您当前的最大值进行比较：

def maxx(maxx):
    maxx=v[0] 
    for i in range(0,len(v)-1):
        if v[i+1]>maxx:
            maxx=v[i+1]
    return maxx

Answer 2

有几种方法可以改进您的实现，包括使用 enumerate 循环列表索引和项目以及使用可挖掘的名称变量。

def maxx(_list):
    _max = -float("inf")
    for n in _list:
    # use for i,n in enumerate(_list): if you need i
        if n > _max:
            _max = n
    return _max