Skip to main content
6 de 6
se añadieron 12 caracteres en el cuerpo
Carlos Muñoz
  • 13.5k
  • 2
  • 47
  • 67

En tu código hay varias correcciones que hacer:

En primer lugar la propiedad Notasalumnos es de tipo double mientras que el campo _notasalumnos es de tipo double[,]. El que está mal es el valor de retorno de la propiedad que lo debes cambiar por doble[,]

public double[,] Notasalumnos
{
    get => _notasalumnos;
    set => _notasalumnos = value;
}

En segundo lugar, el método promedio_total necesita devolver un double pero en tu código estas ejecutando el return al pasar por la cooredenada [0,0]. Eso no solo hace que el resultado sea incorrecto devolviendo sólo la suma y no el promedio y sólo del primer elemento, sino que además no se puede alcanzar el final de la iteración y no podiendo devolver nada en ese caso.

Ya que la lógica está incorrecta y podemos aprovechar el foreach para recorrer todos los elementos del arreglo independientemente de la cantidad de dimensiones no necesitamos hacer el doble for y el código puede quedar así.

public double promedio_total(double[,] Notasalumnos)
{
    var suma = 0.0;
    foreach (var nota in Notasalumnos)
    {
        suma += nota;
    }

    return suma / Notasalumnos.Length;
}

Además no necesitas pasarle como parámetro al método promedio_total el arreglo de notas porque ya es una propiedad de la misma clase, simplemente quítalo.

public double promedio_total()
{
    var suma = 0.0;
    foreach (var nota in Notasalumnos)
    {
        suma += nota;
    }

    return suma / Notasalumnos.Length;
}

Hasta este punto el código ya está corregido y quedaría así:

namespace PROMEDIOCLASE
{
    class NOTASCLASE
    {
        private double[,] _notasalumnos = new double[10, 4];

        public double[,] Notasalumnos
        {
            get => _notasalumnos;
            set => _notasalumnos = value;
        }

        public double promedio_total()
        {
            var suma = 0.0;
            foreach (var nota in Notasalumnos)
            {
                suma += nota;
            }

            return suma / Notasalumnos.Length;
        }
    }
}

Pero quisiera sugerirte algunas cosas más:

Ya que el getter y setter no están haciendo nada mas que devolver el valor del campo subyacente se puede simplificar de esta manera. Ten en cuenta que ya no necesitamos el campo y la inicialización de la propiedad se puede hacer en la misma línea:

namespace PROMEDIOCLASE
{
    class NOTASCLASE
    
        public double[,] Notasalumnos { get; set; } = new double[10, 4];

        public double promedio_total()
        {
            var suma = 0.0;
            foreach (var nota in Notasalumnos)
            {
                suma += nota;
            }

            return suma / Notasalumnos.Length;
        }
    }
}

Finalmente tu sugiero seguir las convenciones de nomenclatura de clases, namespaces, campos, propiedades parámetros, etc.

Con lo cual el código quedaría así:

namespace PromedioClase
{
    class NotasClase
    {
        public double[,] NotasAlumnos { get; set; } = new double[10, 4];

        public double PromedioTotal()
        {
            var suma = 0.0;
            foreach (var nota in NotasAlumnos)
            {
                suma += nota;
            }

            return suma / NotasAlumnos.Length;
        }
    }
}
Carlos Muñoz
  • 13.5k
  • 2
  • 47
  • 67