Shifting arrays K positions without using additional memory

I have been tasked with a program that shifts an array k position without additional memory. I don't know what kind of memory are they talking about, because I will obviously have to use one index value for the loop... Anyway, here is an example:
shiftArray(numbers, k) for 1 2 3 4 5 6 7 8 and k = 3 should give: 4 5 6 7 8 1 2 3.

The problem is simple, however, after constantly failing at minor things, I cannot think clear anymore. Here is my full code. The result for 1 2 3 4 5 6 7 8 is 5 2 3 4 5 6 7 8. I have discovered that the problem is in the if statement, but I cannot figure out why. I know the code is cluttered for the function, but that is only because I do not want to store the size into a variable, because of that memory condition...

If you cannot find a way around the program, could you give me another suggestion if there is a more efficient way? Thank you very much.

Cosmin

Code:

#include <iostream>

using namespace std;

void shiftArray(int numbers[], int k)
{
	int index;
	for(index = 0; index < (int)(sizeof numbers)/(sizeof numbers[0]);  index++)
	{
		if(index >= numbers[(int)(sizeof numbers)/(sizeof numbers[0]) - k])
		{
			numbers[index] = numbers[index % k];
		}
                                else
                                {
                                                numbers[index] = numbers[index + k];
                                }
	}
}

int main(int argc,char* argv) 
{
	int numbers[] = {1, 2, 3, 4, 5, 6, 7, 8};
	int k = 4;
	shiftArray(numbers, k);
	int i;
	for(i = 0; i < (int)(sizeof numbers)/(sizeof numbers[0]); i++)
	{
		cout << numbers[i] << " ";
	}
	cout << endl;

	return 0;
}

The shiftArray function cannot do sizeof(numbers). You need to pass the size of the array to the function:

Code:

void shiftArray(int numbers[], size_t array_size, int k)

Does "no additional memory" mean that you can't even use a single temporary variable?

Hahaha... For showing only, you can do;

Code:

for(int i=k;i<lengthOfArray;i++)
{
cout<<numbers[i];
}
for(int j=0;j<k;j++)
{
cout<<numbers[j];
}

But that's only for display. I know you actually mean, shifting, i am thinking .............

I did it somehow. But the problem now is that the last k numbers are replaced with the first k variables whose original values have been replaced...

Code:

#include <iostream>

using namespace std;

void shiftArray(int numbers[], int sizeofArray, int k)
{
	int index;
	for(index = 0; index < sizeofArray;  index++)
	{
		if(index >= sizeofArray - k)
		{
			numbers[index] = numbers[index % (sizeofArray - index)];
		}
		else
		{
			numbers[index] = numbers[index + k];
		}
	}
}

int main(int argc,char* argv) 
{
	int numbers[] = {1, 2, 3, 4, 5, 6, 7, 8};
	int k = 3;
	int sizeofArray = 8;
	shiftArray(numbers, sizeofArray, k);
	int i;
	for(i = 0; i < sizeofArray; i++)
	{
		cout << numbers[i] << " ";
	}
	cout << endl;

	return 0;
}

Also, isn't it another way to determine the size of an array on the fly so I do not use a storage variable?

I want to ask one thing, that you aren't allowed to use even a single variable except the specified ones?

Originally Posted by Mr.777

I want to ask one thing, that you aren't allowed to use even a single variable except the specified ones?

I suppose. But I have been thinking for more than two hours already and I have also asked an advice from an Informatics teacher and I have been told that it is impossible without having at least one variable and that is for the loop.

So I accept the use of a single variable... I am afraid I have no choice.

Mr. 777, is it possible without using one too? I would like your opinion concering the use of a variable, but I do not want to neglect the second approach as well.

I think you will have to use one temporary variable.

I have managed to ask informations. Yes, I am allowed to use temporary variables! Pfew. Now could you please tell me what is wrong in my problem so the thread can be closed?

Now simple do;

Code:

int temp=0;
for(int index=0;index<k;index++)
{
temp=numbers[k];
numbers[k]++=numbers[index];
numbers[index]=temp;
}

Originally Posted by Mr.777

Now simple do;

Code:

int temp=0;
for(int index=0;index<k;index++)
{
temp=numbers[k];
numbers[k]++=numbers[index];
numbers[index]=temp;
}

I think it should be numbers[k++], with the k between brackets. This code assign to the first k values the ones of the next k. However, the first three ones will be replaced after performing the code...

I am way too confuse after so many nerves and failures... Here is the problem:

I will be succesfully turning the first 5 numbers into the next three ones. So 1 becomes 4, 2 becomes 5, 3 becomes 6, 4 becomes 7, 5 becomes 8. But about the last three? How could I make them become 1 2 3 without using a second array to store these?

Yes my typing mistake. it will be numbers[k++]. Sorry.