Apéndice: PEP 8, docstrings y comentarios

import csv

# ---------------------------------------------------------------
# ANTES: corre bien, y cuesta leerlo.
# ---------------------------------------------------------------
def f(a):
    l=[]
    for x in a:
        if x['cantidad'] not in ('','-','N/A','NaN','null') and x['precio_unitario'] not in ('','-','N/A','NaN','null') and int(x['cantidad'])>0:
            l.append({'c':x['codigo'],'t':int(x['cantidad'])*float(x['precio_unitario'])})
    return l


# ---------------------------------------------------------------
# DESPUES: el mismo resultado, con los nombres de PEP 8.
# ---------------------------------------------------------------
NULOS = {"", "-", "N/A", "NaN", "null"}


def valorizar_stock(filas):
    """Valor inmovilizado por producto: cantidad por precio unitario.

    Devuelve una lista de dicts con 'codigo' y 'valor'. Saltea las filas
    con cantidad o precio ausentes, y las de cantidad no positiva: la
    politica la fija el enunciado, no esta funcion.
    """
    valorizadas = []
    for fila in filas:
        crudo = fila["cantidad"].strip()
        if crudo in NULOS:
            continue
        precio = fila["precio_unitario"].strip()
        if precio in NULOS:
            continue
        cantidad = int(crudo)
        if cantidad <= 0:
            continue
        valorizadas.append({
            "codigo": fila["codigo"],
            "valor": cantidad * float(precio),
        })
    return valorizadas


with open("inventario_productos.csv", encoding="utf-8", newline="") as f_entrada:
    filas = list(csv.DictReader(f_entrada))

print("las dos versiones dan lo mismo:")
antes = f(filas)
despues = valorizar_stock(filas)
print("  antes  :", len(antes), "productos")
print("  despues:", len(despues), "productos")
print("  iguales:", [round(x["t"], 2) for x in antes]
      == [round(x["valor"], 2) for x in despues])

print()
print("las tres primeras:")
for item in despues[:3]:
    print(f"   {item['codigo']}  {item['valor']:>12,.2f}")

print()
print("Que cambio, y por que importa:")
print("  f            -> valorizar_stock  : el nombre dice que hace")
print("  a            -> filas            : el nombre dice que recibe")
print("  x            -> fila             : idem, en el bucle")
print("  l / c / t    -> codigo / valor   : no hay que adivinar")
print("  la linea de 100 caracteres se parte y entra en la pantalla")
print("  la lista de nulos deja de estar repetida en un if")
print("  el docstring dice la politica, que el codigo solo ejecuta")

NULOS = {"", "-", "N/A", "NaN", "null"}


def promedio(notas, politica="descartar"):
    """Promedio de las notas, segun que hacer con las ausentes.

    Args:
        notas: lista de cadenas tal como vienen del CSV.
        politica: "descartar" saltea las ausentes; "cero" las cuenta
            como 0. Cambia el denominador, no solo el numerador.

    Returns:
        El promedio redondeado a 2 decimales, o None si no queda
        ninguna nota usable.

    Raises:
        ValueError: si la politica no es una de las dos previstas.
    """
    if politica not in ("descartar", "cero"):
        raise ValueError(f"politica desconocida: {politica!r}")

    usables = []
    for nota in notas:
        crudo = nota.strip()
        if crudo in NULOS:
            if politica == "cero":
                usables.append(0.0)
            continue
        usables.append(float(crudo))

    if not usables:
        return None
    return round(sum(usables) / len(usables), 2)


CASO = ["9.7", "4.7", "-", "8.6", ""]
print("notas:", CASO)
print("  descartar ->", promedio(CASO), "(sobre 3 notas)")
print("  cero      ->", promedio(CASO, "cero"), "(sobre 5)")

print()
print("Y esto es lo que ve quien la use, sin abrir el archivo:")
print("-" * 58)
print(promedio.__doc__)
print("-" * 58)

print()
print("Las reglas de PEP 257, que son cuatro:")
print("  1. Va entre triple comilla, aunque sea de una linea.")
print("  2. La primera linea resume todo, en una sola linea y con punto")
print("     final. PEP 257 la pide imperativa en ingles ('Return the...');")
print("     en espanol este curso usa una frase descriptiva, y lo que no")
print("     cambia es que sea UNA linea y que cierre con punto.")
print("  3. Si hay mas, va una linea en blanco despues de la primera:")
print("     help() y los editores muestran solo esa primera linea.")
print("  4. Dice QUE hace y QUE devuelve, no COMO. El como esta abajo.")

print()
print("La prueba de si un docstring sirve: ¿alcanza para usar la funcion")
print("sin leerle el cuerpo? Aca si, incluido el detalle que importa:")
print("que la politica cambia el DENOMINADOR, no solo el numerador.")

import csv

with open("estudiantes_notas.csv", encoding="utf-8", newline="") as f:
    filas = list(csv.DictReader(f))

# ---- COMENTARIOS QUE SOBRAN --------------------------------------
# Recorremos las filas
for fila in filas[:1]:
    # Obtenemos el legajo
    legajo = fila["legajo"]
    # Incrementamos el contador en 1
    contador = 1

print("Los tres de arriba repiten lo que el codigo ya dice.")
print("Cuando el codigo cambie, el comentario va a quedar mintiendo:")
print("nadie se acuerda de actualizar un comentario que no aporta.")

print()
# ---- COMENTARIOS QUE SALVAN --------------------------------------
NULOS = {"", "-", "N/A", "NaN", "null"}

usables = []
for fila in filas:
    crudo = fila["nota_3"].strip()
    # 'NaN' entra a esta lista a proposito: float('NaN') NO falla,
    # devuelve nan, y un solo nan convierte el promedio entero en nan.
    if crudo in NULOS:
        continue
    valor = float(crudo)
    # El rango lo fija el reglamento de la materia, no los datos: hay
    # una nota de -1 sembrada y es un valor imposible, no un atipico.
    if not 0 <= valor <= 10:
        continue
    usables.append(valor)

print("notas usables:", len(usables), "de", len(filas))
print("promedio:", round(sum(usables) / len(usables), 2))

print()
print("Esos dos comentarios dicen POR QUE, no QUE. Ninguno de los dos")
print("se deduce leyendo el codigo, y los dos evitan que el que venga")
print("despues 'simplifique' la condicion y rompa el calculo.")

print()
print("La regla: si el comentario se puede borrar sin perder nada,")
print("sobraba. Si explica una decision, una unidad, un caso raro o")
print("un porque, se queda.")

print()
print("Y el mejor comentario suele ser un nombre:")
por_ciento_asistencia = 0.75
print("  umbral = 0.75             -> ¿de que?")
print("  por_ciento_asistencia =", por_ciento_asistencia,
      "-> no hace falta comentario")