Thread: final exam in the morning

Code:

#include <iostream>

using namespace std;

int main()
{
	const int SIZE = 5;
	int values[SIZE], counter, temp;
	bool swap;

	
	for (counter=0; counter<SIZE; counter++)
	{
		cout<<"please input a number"<<endl;
		cin>>values[counter];
	}

	do
	{	swap = false;
		for (counter=0; counter<values[counter-1]; counter++)
		{
			if (values[counter+1]>values[counter])
			{
				temp=values[counter];
				values[counter+1]=values[counter];
				values[counter]=temp;
				swap=true;
			}
		}
		while(swap);
	}
	return 0;
}

it's giving me this error


" error C2059: syntax error : 'return "

please help

I didn't know for loops ended with a while()....

What, do you expect us to be phsychic? Show the new code!

>> Show the new code!
Maybe there is no new code. He just wanted to remind us that it doesn't work.

Anyway, I'm going to bed. My last advice: Check the part on do-while() loops

Originally Posted by Brad0407

>> Show the new code!
Maybe there is no new code. He just wanted to remind us that it doesn't work.

In which case ammanbesaw has not been reading anything we're written. And in which case ammanbesaw can figure out the problems without our help.

Look closer at your outer do-while loop... its quite easy to spot the mistake

Originally Posted by ammanbesaw

Code:

#include <iostream>

using namespace std;

int main()
{
	const int SIZE = 5;
	int values[SIZE], counter, temp;
	bool swap;

	
	for (counter=0; counter<SIZE; counter++)
	{
		cout<<"please input a number"<<endl;
		cin>>values[counter];
	}

	do
	{	swap = false;
		for (counter=0; counter<values[counter-1]; counter++)
		{
			if (values[counter+1]>values[counter])
			{
				temp=values[counter];
				values[counter+1]=values[counter];
				values[counter]=temp;
				swap=true;
			}
		}
		while(swap);
	}
	return 0;
}

it's giving me this error


" error C2059: syntax error : 'return "

please help

There it is.

Probably not a crash, but nevertheless not a good thing.

Change

Code:

		}
		while(swap);
	}

to

Code:

		}
	
	} while(swap);

The while statement should be after the closing bracket from do.



And I did get a crash in the middle of execution most likely because of what you guys just said (about out o' bounds array member)