El problema ya lo ha expuesto Saul en su respuesta. Además el if
no es necesario, y de serlo lo que ha de ejecutarse cuando se cumpla debe ir identado un nivel más para que quede dentro del condicional.
Recuerda que en Python el indexado empieza en 0, por lo que el último indice de una lista es su longitud menos 1. Imaginemos una matriz de 3 X 3, los índices que necesitas generar son:
[(0, 0), ____ , (0, 2)]
[ ____ , (1, 1), ____ ]
[(2, 0), ____ , (2, 2)]
El índice de la fila ya lo generas en el for
, el índice de la columna en el caso de la diagonal principal es siempre el mismo que el de la fila. El indice de la columna en el caso de la diagonal secundaria es siempre el índice del último elemento de la fila (len(matriz) - 1
) menos el indice de la fila en la que estamos. En este punto conviene observar que Python permite el indexado negativo, con lista[-1]
obtenemos el último elemento de la lista, por lo que a[i][n-i-1]
es equivalente a a[i][-i-1]
.
Bien, dicho esto y dado que la pregunta es algo más general, podemos implementar esto de otras formas sin salir de Python ni usar ninguna biblioteca. Si solo necesitas la diferencia puedes evitar el cálculo por separado de la suma de ambas matrices, lo que simplifica el código:
def diagonal_difference(matriz):
dif = 0
for i in range(len(matriz)):
dif += matriz[i][i] - matriz[i][-i-1]
return dif
Podemos ir un paso más allá para que el código sea más eficiente usando enumerate
para obtener la fila y su índice en vez de usar range
:
def diagonal_difference(matriz):
dif = 0
for i, row in enumerate(matriz):
dif += row[i] - row[-i-1]
return dif
Otro paso más es usar la función preconstruida sum
en un generador por compresión, lo que evita también la llamada a len
:
def diagonal_difference(matriz):
dif = sum((row[i] - row[-i - 1] for i, row in enumerate(matriz)))
return dif
Si quieres obtener el valor absoluto de la diferencia basta con que al retornar hagas return abs(dif)
. La matriz a de ser cuadrada por definición, por lo que de no validar esto antes, sería oportuno realizar una validación dentro de la propia función:
def diagonal_difference(matriz):
if not all(len(row)==len(matriz) for row in matriz):
raise ValueError("The matrix must be square")
# ...
Nota: dif += ...
equivale en este caso a dif = dif + ...