please help with c ++?

Both of the above answers are rather useless. One tells you to read a book, the other advises with C-style preprocessor constants and structs. Give me a moment to read through your longass post and I'll give you a real answer.

EDIT:

1. Don't get into the bad habit of using global variables. Pass the local variables into the function by value to protect them from being changed by the function, or pass them by reference if you want the function to change them.

2. It's good practice to initialize your variables to a default value so you don't run the risk of trying to access a variable of undefined value.

3. You don't really have any input validation. What you have is an if statement that checks for one unwanted condition and displays a message if the input matches this undesired value. If statements should be used to test for a DESIRED value, because there can be an infinite amount of unwanted values. Undesired values should be dealt with by the else statement.

4. After accepting the input, if it's invalid you do nothing with it and let the program end (return 0) from your if statement. You should only have ONE statement in main that returns 0. Anything else should be within functions or loops that exit under appropriate circumstances (they will eventually hit the return statement anyway), but nothing should end the program except the end of main.

5. Your for loop is again not using input validation, but exiting the program if one particular undesired condition is met.

6. Because I told you not to use global variables, this means you'll have to change your functions that access global variables. Also, it doesn't really matter which naming convention you follow, but you should at least follow one pattern for naming your variables, functions, preprocessor constants, etc., to make them easier to identify. For simplicity, I'm going to use lowercase variable names and TitleCase function names.

7. You mismatched your array name in your sortArray function. In one place you refer to it as sales, in another place you refer to it as array (the local parameter name). You probably did this as a direct result of your bad global variable habits.

Below is a link to a text file with a modified version of your code, with the above-mentioned corrections.

Ugh, this is way too long a question to answer...

Read the book "Accelerated C++", it has pretty much EXACTLY the same problem (maybe that's whey they got it from?) and teaches you how to build the program step by step across the first few chapters of the book.

you can make a struct:

#define maxN 10000 //(or whatever the maximum number of students is)

struct data

{

string name;

int score;

} arr[maxN];

for the sorting function, you can use the simple bubble sort

//the following sort function will sort according to the students' scores in descending order

bubbleSort(student data[])

{

int i, j;

for(i=0;i<nStudents;i++)// nstudents is a variable = to the number of students

for(j=i+1;j<nStudents;j++)

if(data[i] < data[j]) swap(data[i], data[j]);

}

for scanning, you can do that:

int nStudents = 0, i;

for(i=0;i<nStudents;i++)

cin >> arr[nStudents].name >> arr[nStudents++].score;

This code will print 2 random integers on the stdout. printf("%d %d",i,j) will print a million 2 and printf("%d %d",j,i) will print 2 a million(because of the fact this is the output you asked!).