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;
}
}
}