Come @Stefan ha intuito in un commento alla risposta di @ CaptainGiraffe, si guadagna un bel po 'usando un vettore di strutture invece di una struttura di vettori. Il codice corretto ha questo aspetto:
#include <vector>
#include <cmath>
#include <iostream>
#include <time.h>
class FloodIsolation {
public:
FloodIsolation() :
h(0),
floodedCells(0),
floodedCellsTimeInterval(0),
qInflow(0),
qStartTime(0),
qEndTime(0),
lowerFloorCells(0),
cellLocationX(0),
cellLocationY(0),
cellLocationZ(0),
levelOfCell(0),
valueOfCellIds(0),
h0(0),
vU(0),
vV(0),
vUh(0),
vVh(0),
vUh0(0),
vVh0(0),
ghh(0),
sfx(0),
sfy(0),
qIn(0),
typeInterface(nEdges, 0),
neighborIds(nEdges, 0)
{
}
~FloodIsolation(){
}
void Update() {
h = h + 1;
floodedCells = !floodedCells;
floodedCellsTimeInterval = !floodedCellsTimeInterval;
qInflow = qInflow + 1;
qStartTime = qStartTime + 1;
qEndTime = qEndTime + 1;
lowerFloorCells = lowerFloorCells + 1;
cellLocationX = cellLocationX + 1;
cellLocationY = cellLocationY + 1;
cellLocationZ = cellLocationZ + 1;
levelOfCell = levelOfCell + 1;
valueOfCellIds = valueOfCellIds + 1;
h0 = h0 + 1;
vU = vU + 1;
vV = vV + 1;
vUh = vUh + 1;
vVh = vVh + 1;
vUh0 = vUh0 + 1;
vVh0 = vVh0 + 1;
ghh = ghh + 1;
sfx = sfx + 1;
sfy = sfy + 1;
qIn = qIn + 1;
for(int j = 0; j < nEdges; ++j) {
++typeInterface[j];
++neighborIds[j];
}
}
private:
static const int nEdges = 6;
bool floodedCells;
bool floodedCellsTimeInterval;
std::vector<int> neighborIds;
double valueOfCellIds;
double h;
double h0;
double vU;
double vV;
double vUh;
double vVh;
double vUh0;
double vVh0;
double ghh;
double sfx;
double sfy;
double qInflow;
double qStartTime;
double qEndTime;
double qIn;
double nx;
double ny;
double floorLevels;
int lowerFloorCells;
bool flagInterface;
std::vector<int> typeInterface;
bool floorCompleteleyFilled;
double cellLocationX;
double cellLocationY;
double cellLocationZ;
int levelOfCell;
};
int main() {
std::vector<FloodIsolation> isolation(20000);
clock_t start = clock();
for (int i = 0; i < 400; ++i) {
if(i % 100 == 0) {
std::cout << i << "\n";
}
for (auto &f : isolation)
f.Update();
}
clock_t stop = clock();
std::cout << "Time: " << difftime(stop, start) / 1000 << "\n";
}
Compilato con il compilatore da VC ++ 2015 CTP, utilizzando -EHsc -O2b2 -GL -Qpar
, ottengo risultati come:
0
100
200
300
Time: 0.135
La compilazione con g ++ produce un risultato leggermente più lento:
0
100
200
300
Time: 0.156
Sullo stesso hardware, utilizzando il compilatore / JVM da Java 8u45, ottengo risultati come:
0
100
200
300
Time: 181
Questo è circa il 35% più lento della versione da VC ++ e circa il 16% più lento della versione da g ++.
Se aumentiamo il numero di iterazioni al 2000 desiderato, la differenza scende solo al 3%, suggerendo che parte del vantaggio del C ++ in questo caso è semplicemente un caricamento più veloce (un problema perenne con Java), non proprio nell'esecuzione stessa. Ciò non mi sorprende in questo caso: il calcolo misurato (nel codice pubblicato) è così banale che dubito che la maggior parte dei compilatori possa fare molto per ottimizzarlo.
std::vector<bool>
usa un bit per elemento per risparmiare spazio, il che porta a molti spostamenti di bit. Se vuoi la velocità, dovresti starne alla larga. Usastd::vector<int>
invece.