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

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 ahcer forhacer el doble for y el código puede quedar así.

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 ahcer for y el código puede quedar así

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í.

se eliminaron 19 caracteres en el cuerpo
Origen Enlace
Carlos Muñoz
  • 13.5k
  • 2
  • 47
  • 67

En primer lugar la propiedad double Notasalumnos es de tipo double por lo mientras que debes retornar un double no unel campo _notasalumnos es de tipo double[,]. En realidad elEl que está mal es el valor de retorno de la propiedad porque el campo subyacente es de tipoque lo debes cambiar por doble[,]

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;
}
namespace PROMEDIOCLASE
{
    class NOTASCLASE
    {
        private double[,] _notasalumnos = new double[10, 4];

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

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

            return suma / Notasalumnos.Length;
        }
    }
}
namespace PROMEDIOCLASE
{
    class NOTASCLASE
    
        public double[,] Notasalumnos { get; set; } = new double[10, 4];

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

            return suma / Notasalumnos.Length;
        }
    }
}

En primer lugar la propiedad double Notasalumnos es de tipo double por lo que debes retornar un double no un double[,]. En realidad el que está mal es el valor de retorno de la propiedad porque el campo subyacente es de tipo doble[,]

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

    return suma / Notasalumnos.Length;
}
namespace PROMEDIOCLASE
{
    class NOTASCLASE
    {
        private double[,] _notasalumnos = new double[10, 4];

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

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

            return suma / Notasalumnos.Length;
        }
    }
}
namespace PROMEDIOCLASE
{
    class NOTASCLASE
    
        public double[,] Notasalumnos { get; set; } = new double[10, 4];

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

            return suma / Notasalumnos.Length;
        }
    }
}

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 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;
}
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;
        }
    }
}
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;
        }
    }
}
se eliminaron 19 caracteres en el cuerpo
Origen Enlace
Carlos Muñoz
  • 13.5k
  • 2
  • 47
  • 67

Nota, voy a ir editando para ver como se llega de tu código a esto

public double[,] Notasalumnos
{
    get => _notasalumnos; //aquí me sale un error que dice: no se puede convertir implícitamente el tipo 'double[*,*]' en double.
    set => _notasalumnos = value; //aquí me sale el mismo error
}

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:

CorrigiendoFinalmente tu sugiero seguir las convenciones de nomenclatura de clases, simplificando y mejorando un poconamespaces, campos, propiedades parámetros, etc.

Con lo cual el código se podría llegar a estoquedarí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;
        }
    }
}

Nota, voy a ir editando para ver como se llega de tu código a esto

public double[,] Notasalumnos
{
    get => _notasalumnos; //aquí me sale un error que dice: no se puede convertir implícitamente el tipo 'double[*,*]' en double.
    set => _notasalumnos = value; //aquí me sale el mismo error
}

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:

Corrigiendo, simplificando y mejorando un poco el código se podría llegar a esto:

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;
    }
}
public double[,] Notasalumnos
{
    get => _notasalumnos;
    set => _notasalumnos = value;
}

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:

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;
        }
    }
}
se añadieron 537 caracteres en el cuerpo
Origen Enlace
Carlos Muñoz
  • 13.5k
  • 2
  • 47
  • 67
Loading
se añadieron 537 caracteres en el cuerpo
Origen Enlace
Carlos Muñoz
  • 13.5k
  • 2
  • 47
  • 67
Loading
Origen Enlace
Carlos Muñoz
  • 13.5k
  • 2
  • 47
  • 67
Loading