How to stop returning the address in memory?

[barbes]

Can't seem to access the required elements in the following code. Pretty simplistic program, but hey... gotta start somewhere right?



#include <iostream.h>

float* mirror(float[], int);

void main(void)
{
const int size = 5;
float numbers[size] = {1.0, 2.0, 3.0, 4.0, 5.0};

cout << "The numbers before mirror() are..." << endl;

for(int i = 0; i < size; i++)
{
cout << numbers[i] << " ";

}
cout << endl;


cout << "The numbers after mirror() are..." << endl;

for(i = 0; i < size; i++)
{
cout << mirror(numbers, size) << " ";

}


}

float* mirror(float p[], int arraySize)
{
int i;

float* tempArray = new float[arraySize];

for(i = 0; i < arraySize; i++)
tempArray[i] = p[(arraySize - 1) - 1];

return tempArray;

}

[jfaust]

You can't send an array to cout.


/*...*/

cout << "The numbers after mirror() are..." << endl;

float* after = mirror(numbers, size);

for(i = 0; i < size; i++)
{
cout << after[i] << " ";
}

//don't forget this!
delete[]after;

/*...*/


Jeff

[jfaust]

Also, you should use <iostream> instead of <iostream.h>.

Once you feel comfortable with your example, take a look at std::vector. It's a much nicer way of handling arrays.

Jeff

[jfaust]

Also... (maybe I should read everything the first time around)


for(i = 0; i < arraySize; i++)
tempArray[i] = p[(arraySize - 1) - 1];


should be

for(i = 0; i < arraySize; ++i)
tempArray[i] = p[i];


You can also accomplish this much clearer by using std::copy.

Jeff

[barbes]

Thanx for all your help Jeff

Just with that last post..the actual function should return the reverse order of the array...hence (mirror), however I should of called the function reverseOrder or sumthing...

Thanx again..

[Yves M]

actually mirror is fine, but you put a 1 instead of an i :p


float* mirror(float p[], int arraySize)
{
int i;

float* tempArray = new float[arraySize];

for(i = 0; i < arraySize; i++)
tempArray[i] = p[(arraySize - 1) - i];

return tempArray;

}