Search:

A simple possible performance improvement - depending upon the compiler - is to look at the condition part of the for loops. In the for loop



for (trade2position = 1 + trade1position;...

Pleased to be of help. Hope it all goes OK for you now. Welcome to the world of c++!

It compiles without any warnings with my VS. Is the info generated correct and what you expected? How long is the program now taking to run?

Can you post your code again please after all the changes. I'll crank it through my VC compiler and see what comes out.

You would have

oneres.q = headerarray[q];
oneres.c = headerarray[c];

Rather than having three different results vectors - one each for float, int and string - just have one vector and put the data direct into this rather than into the three different and later on...

You mention that it is taking about 6 minutes to run. Roughly, do you know how much of this time is spent reading the csv file and building the arrays?

I also note in your code that you are not pre-reserving space for the vectors. If a vector does not have enough memory to store an extra element then it will re-allocate memory and copy existing...

In c++ with a multi condition statement, the conditions are only evaluated left to right as needed to correctly obtain the result. So in the case of &&, the left test if evaluted first. If it is...

Perhaps the easiest way is to define a struct to contain the 3 numbers and the two strings and then declare a vector of this struct. The example below illustrates a possible method



#include...