3

Estoy empezando a programar con c++ y me ha surgido una duda al hacer un programa. He hecho una clase que pretende gestionar una matriz3d de datos. Para ello, he creado dentro un array lineal y métodos necesarios. Para gestionar los datos quiero crear un functor que devuelva una referencia a un voxel concreto de la matriz.

Por ejemplo a(1,2,3) = 3.0

No estoy del todo seguro si devolver una referencia esta del todo bien para sustituir funciones get y set. El programa va a realizar cálculos matemáticos y me gustaría que la forma que quede en el código sea lo más parecida a la expresión matemática.

añado parte del código:

class Grid {
  private:
     int xSize,ysize,zSize;
     double *matriz3D;

  public:
 ... constructores copy constructor move etc.. 

return double& operator()(int xPos, int yPos, zPos){
   if(xPos < 0 || xPos > xSize 
      yPos < 0 || yPos > ySize 
      zPos < 0 || zPos > zSize ){
       trow "Fuera de limites";
     }
   return matriz3d[zPos*xSize*ySize + yPos*xSize + xPos];
  }
 }
}

Cualquier comentario o sugerencia es bienvenido. muchas gracias

1
  • La pregunta es muy interesante a la vez que muy incompleta. Ayudaría mucho que compartieras el código de lo que llevas hecho. Mientras tanto puede que te sea útil esta pregunta. el 25 sep. 2019 a las 7:20

1 respuesta 1

1

Por lo que veo, el operador paréntesis (operator()) es justo lo que necesitas y funciona como pides. Aún así hay espacio para mejora:

  1. Haz que la clase sea plantilla: así podrás definir en tiempo de compilación tanto el tipo como los tamaños, las matrices no cambian de tamaño durante las operaciones matemáticas ¿verdad?.
  2. Evita la memoria dinámica: la memoria dinámica (la creada con el operador new es lenta, evítala si puedes), declara una formación de tamaño fijo dentro de tu clase.
  3. Crea un constructor: vale la pena poder dar valores iniciales a la matriz.
  4. La función de acceso necesita una versión constante: la versión constante equivale al get, la otra versión equivale al set.
  5. Al lanzar excepciones, no lances valores: lanza objetos excepción.
  6. La dimensiones no tienen signo: no puedes tener tamaños negativos, así que pasa los tamaños a entero sin signo y así además no necesitarás comprobar si el valor es negativo.

No estoy del todo seguro si devolver una referencia esta del todo bien para sustituir funciones get y set.

Es completamente correcto, y deseable, pero necesitas versiones de lectura (versión constante) y lectura/escritura (versión normal).

Con esas sugerencias tu clase podría parecerse a:

template <typename T, unsigned W, unsigned H, unsigned L>
struct Grid
{
    template <typename ... pack>
    Grid(pack ... valores) :
        datos{valores ...}
    {}

    T &operator()(unsigned w, unsigned h, unsigned l)
    {
        if (W < w || H < h || L < l)
            throw std::invalid_argument{"Fuera de limites"};

        return datos[w][h][l];
    }
    const T &operator()(unsigned w, unsigned h, unsigned l) const
    {
        if (W < w || H < h || L < l)
            throw std::invalid_argument{"Fuera de limites"};

        return datos[w][h][l];
    }
private:
    T datos[W][H][L]{};
};

Que puede usarse así:

using d3x3x3 = Grid<double, 3, 3, 3>;

d3x3x3 a
{
    0., 1., 2.,
    3., 4., 5.,
    6., 7., 8.,

    9., 10., 11.,
    12., 13., 14.,
    15., 16., 17.,

    18., 19., 20.,
    21., 22., 23.,
    24., 25., 26.,
};

std::cout << a(0, 0, 0) << '\n'; // Muestra 0
std::cout << a(1, 1, 1) << '\n'; // Muestra 13
std::cout << a(2, 2, 2) << '\n'; // Muestra 26

Y el operador paréntesis, sirve tanto para lectura como para escritura:

std::cout << a(0, 1, 2) << '\n'; // Muestra 5
a(0, 1, 2) = 3.;
std::cout << a(0, 1, 2) << '\n'; // Muestra 3

Puedes ver el código funcionando en Try it online!.

1
  • perfecto, tomo nota muchas gracias!!
    – theholy
    el 25 sep. 2019 a las 11:54

Tu Respuesta

Al pulsar en “Publica tu respuesta”, muestras tu consentimiento a nuestros términos de servicio, política de privacidad y política de cookies

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.