but adding that dependency to all those classes doesn't sit well with me.
Se ogni array è incapsulato all'interno di una classe di dati, ha senso aggiungere un operatore sovraccarico a quella classe per cercare gli elementi.
// This code only illustrate the overloaded operator.
// The code here is incomplete, and by itself is unfit
// for all purposes.
//
// (See below for a link to OpenCV's matrix source code.)
//
template <typename T>
class Array2
{
T& operator () (int x, int y) { return mData[y * mWidth + x]; }
const T& operator () (int x, int y) const { return mData[y * mWidth + x]; }
// If one really needs a function to just return the value of (y * mWidth + x), such functions can also be added to this class.
private:
int mWidth;
int mHeight;
T* mData; // make sure you know how to allocate, free, prevent memory leak, and prevent double-freeing of this pointer to array.
};
Una versione più completa sarà probabilmente simile a questa:
Se ci sono più classi, ed è prevedibile nel prossimo futuro che ogni classe possa avere presto dimensioni diverse (ad esempio, 2D e 4D), e se non si sta già utilizzando una classe per ogni tipo di matrice, si dovrebbe prendere seriamente in considerazione la possibilità di effettuare questa transizione.
Se, tuttavia, devi utilizzare una funzione gratuita (non un membro di alcuna classe), tieni presente questi aspetti:
-
Pensa a come WIDTH e HEIGHT sono passati alla funzione. Nell'esempio del codice (e anche nella risposta di @ Snowman), si presume che l'intero programma abbia un unico valore hardcoded per WIDTH e HEIGHT . Non appena inizi a utilizzare matrici di dimensioni diverse, vedrai che l'approccio di incapsulamento delle classi di dati è l'unico modo sensato per farlo.
-
Se hai bisogno di utilizzare matrici di dimensioni diverse, dai un nome distinto a quelle funzioni.
-
Evita le collisioni del nome della funzione. Se lo fai diventare un membro della classe, c'è meno preoccupazione per le collisioni dei nomi di funzioni. Altrimenti, puoi renderlo un membro di una classe statica, assegnare a tale funzione un prefisso specifico del progetto o metterla in uno spazio dei nomi.
Infine, va bene avere duplicati per pezzi così piccoli di codice. La prevenzione della duplicazione del codice dovrebbe essere soppesata rispetto al problema di aumentare le dipendenze (e quindi l'accoppiamento). In questo caso particolare, è opportuno lasciare che ogni classe abbia le funzioni di convenienza che usa.
Naturalmente, l'approccio incapsulato alla classe dei dati dovrebbe essere ancora l'approccio più favorevole, quando tutte le cose sono prese in considerazione.
Se nel futuro molto lontano inizi a considerare l'implementazione di SIMD e altre ottimizzazioni delle prestazioni, saranno necessarie ulteriori modifiche alla classe dati.