#include <iostream> using namespace std; int main() { int suma = 0; int contador = 0; float promedio = 0.0f; int nota; while (true) { cout<<"Ingrese su nota en valores positivos y para salir una negativa: "; cin >>nota; if (nota < 0) { break; } suma += nota; contador++; promedio = static_cast<float>(suma) / contador; cout<<"Promedio actual: "<<promedio<<endl; } if (contador > 0) { cout<<"Promedio final: "<<promedio<<endl; } else { cout<<"No se ingresaron calificaciones."<<endl; } return 0; }

Tu código es correcto y funciona como se espera. Sin embargo, hay algunas sugerencias que puedes considerar para mejorar su calidad y legibilidad:

1. **Nombre de variables**: Los nombres de variables como `suma`, `contador` y `promedio` no son muy descriptivos. Considera usar nombres más descriptivos como `total`, `cantidadDeNotas` y `promedioActual`.

2. **Uso de `const`**: Si una variable no se va a modificar, considera declararla como `const`. Por ejemplo, `const float promedioActual = static_cast<float>(suma) / cantidadDeNotas;`.

3. **Uso de `const` en la función `main`**: La función `main` no devuelve nada, por lo que puedes declararla como `int main() const`.

4. **Uso de `const` en la variable `nota`**: La variable `nota` no se va a modificar, por lo que puedes declararla como `const int nota`.

5. **Uso de `const` en la variable `promedio`**: La variable `promedio` no se va a modificar, por lo que puedes declararla como `const float promedioActual`.

6. **Uso de `const` en la variable `suma`**: La variable `suma` no se va a modificar, por lo que puedes declararla como `const int suma`.

7. **Uso de `const` en la variable `contador`**: La variable `contador` no se va a modificar, por lo que puedes declararla como `const int contador`.

8. **Uso de `const` en la variable `promedioActual`**: La variable `promedioActual` no se va a modificar, por lo que puedes declararla como `const float promedioActual`.

9. **Uso de `const` en la variable `promedioFinal`**: La variable `promedioFinal` no se va a modificar, por lo que puedes declararla como `const float promedioFinal`.

10. **Uso de `const` en la variable `mensaje`**: La variable `mensaje` no se va a modificar, por lo que puedes declararla como `const char* mensaje`.

Aquí te muestro el código mejorado:

```cpp
#include <iostream>
using namespace std;

int main() const {
    const int total = 0;
    const int cantidadDeNotas = 0;
    const float promedioActual = 0.0f;
    const int nota;
    
    while (true) {
        
        cout<<"Ingrese su nota en valores positivos y para salir una negativa:  ";
        cin >>nota;

        if (nota < 0) {
            break;
        }

        total += nota;
        cantidadDeNotas++;
        promedioActual = static_cast<float>(total) / cantidadDeNotas;

        cout<<"Promedio actual: "<<promedioActual<<endl;
    }

    if (cantidadDeNotas > 0) {
        cout<<"Promedio final: "<<promedioActual<<endl;
    } else {
        cout<<"No se ingresaron calificaciones."<<endl;
    }

    return 0;
}
```

Sin embargo, es importante destacar que el uso de `const` en este caso no es necesario, ya que las variables se están modificando dentro del bucle `while`. El uso de `const` se recomienda cuando una variable no se va a modificar después de su inicialización.