我正在做一个家庭作业程序,需要创建一个矩阵,我执行了将带有一些记录(以前创建)的数组作为参数的函数,但是当我创建大量记录时,例如:100条记录,我得到列表索引超出范围
创建数组:
if(null === $result){}
数组创建函数调用:
def crearVector(n):
vec = [None] * n
for i in range(len(vec)):
codigo = random.randint(1, 1000)
precio = round(random.uniform(1, 100000), 2)
ubicacion = random.randint(1, 23)
estado = random.randint(0, 1)
cantidad = random.randint(0, 1000)
puntuacion = random.randint(1, 5)
publicacion = Publicacion(codigo, precio, ubicacion, estado, cantidad, puntuacion)
vec[i] = publicacion
return vec
矩阵创建和显示功能:
def test():
n = validateHigherThan(0)
vec = crearVector(n)
矩阵创建:
mat = crearMatriz(vec)
forma = int(input("How would you like to show the matrix?(0: matrix, 1: list): "))
if forma == 1:
mostrarMatrizLista(mat)
elif forma == 0:
mostrarMatriz(mat)
答案 0 :(得分:1)
您创建一个23 * 5的矩阵
fil = vec[i].ubicacion
应该是
fil = vec[i].ubicacion-1
因为ubicacion可以具有1到23之间的随机数。
更好的是,您可以使用全局变量来定义这些参数,而不必在两个函数中对其进行硬编码。
col_limit = 5
fil_limit = 23
def crearVector(n):
vec = [None] * n
for i in range(len(vec)):
codigo = random.randint(1, 1000)
precio = round(random.uniform(1, 100000), 2)
ubicacion = random.randint(1, fil_limit)
estado = random.randint(0, 1)
cantidad = random.randint(0, 1000)
puntuacion = random.randint(1, col_limit)
vec[i] = Publicacion(codigo, precio, ubicacion, estado, cantidad, puntuacion)
return vec
def crearMatriz(vec):
mat = [[0] * col_limit for i in range(fil_limit)]
for i in range(len(vec)):
fil = vec[i].ubicacion - 1
col = vec[i].puntuacion- 1
mat[fil][col] += 1
return mat
答案 1 :(得分:0)
我相信您在矩阵上的fil列的ubicacion上缺少-1。请参见下面的代码。定义以0s开头的列时要小心。
# ubicacion = random.randint(1, 23) (range is 1 to 23)
def crearMatriz(vec):
mat = [[0] * 5 for i in range(23)]
for i in range(len(vec)):
fil = vec[i].ubicacion-1 # <- this -1 is what you were missing, when ubicacion = 23 list goes out of range.
col = vec[i].puntuacion-1
mat[fil][col] += 1
return mat
此外,我认为您的程序实际上不需要那么多的可读性,因为它是一项家庭作业,但是,例如,使用一些变量来帮助您定义最大范围对调试很有帮助。
import random
CODIGO_MAX = 1000
PRECIO_MAX = 100000
UBICACION_MAX = 23
ESTADO_MAX = 1
CANTIDAD_MAX = 1000
PUNTUACION_MAX = 5
def crearVector(n):
vec = [None] * n
for i in range(len(vec)):
codigo = random.randint(1, CODIGO_MAX)
precio = round(random.uniform(1, PRECIO_MAX), 2)
ubicacion = random.randint(1, UBICACION_MAX)
estado = random.randint(0, ESTADO_MAX)
cantidad = random.randint(0, CANTIDAD_MAX)
puntuacion = random.randint(1, PUNTUACION_MAX)
publicacion = Publicacion(codigo, precio, ubicacion, estado, cantidad, puntuacion)
vec[i] = publicacion
return vec
def crearMatriz(vec):
mat = [[0] * PUNTUACION_MAX for i in range(UBICACION_MAX)]
for i in range(len(vec)):
fil = vec[i].ubicacion-1
col = vec[i].puntuacion-1
mat[fil][col] += 1
return mat
crearMatriz(crearVector(1000))