Grouping

Quote:

---

Originally Posted by adak

so is that persons matrix number bk08110110 or is it just 08110110?

---

bk08110110

OK, so if (strcmp(string1, string2) == 0) or equals > 0 (first string is greater than the 2nd one, or equals < 0, (second string is greater than the first string).

Sure, but can you put that into the Sort function and make it work?? :p :p :p

Give it a try. I'll check back in a bit.

After #include

Code:

---

void Sorter(void);

---

inside main(), after calculation

Code:

---

Sorter();

---

outside main()

Code:

---

void Sorter(void)  {
  int i, j; 
  Record *ptr1;
  Record *ptr2;
  Record *ptrtemp; 

  for(i = 0; i < 10-1; i++)  {
      for(j = i+1; j < 10; j++)  {
        ptr1 = &records[i];
        ptr2 = &records[j];
        if (strcmp(ptr1->student[10], ptr2->student[10]) < 0 )  {
            strcpy(*ptrtemp, *ptr1);
            strcpy(*ptr1, *ptr2);
            strcpy(*ptr2, *ptrtemp);
          }
      }
  }
}

---

error C2440: 'function' : cannot convert from 'Record' to 'char *'


i tried putting the

Code:

---

void Sorter(void)  {
  int i, j; 
  Record *ptr1;
  Record *ptr2;
  Record *ptrtemp; 

  for(i = 0; i < 10-1; i++)  {
      for(j = i+1; j < 10; j++)  {
        ptr1 = &records[i];
        ptr2 = &records[j];
        if (strcmp(ptr1->student[10], ptr2->student[10]) < 0 )  {
            strcpy(*ptrtemp, *ptr1);
            strcpy(*ptr1, *ptr2);
            strcpy(*ptr2, *ptrtemp);
          }
      }
  }
}

---

inside main() after the Sorter(); but it gives tons of error

Code:

---

void Sorter(void)  {
  int i, j; 
  Record *ptr1;
  Record *ptr2;
  Record *ptrtemp; 

  for(i = 0; i < 10-1; i++)  {
      for(j = i+1; j < 10; j++)  {
        ptr1 = &records[i];
        ptr2 = &records[j];
        if (strcmp(ptr1->student, ptr2->student) > 0 )  {
            strcpy(*ptrtemp, *ptr1);
            strcpy(*ptr1, *ptr2);
            strcpy(*ptr2, *ptrtemp);
          }
      }
  }
}

---

Quote:

---

error C2440: 'function' : cannot convert from 'Record' to 'char *'

---

student[10] is the char in the 11th spot of the student array. You don't want that. You want just "student", which is the address of the entire char array. I fixed it, above.

WAIT!! NO STRCPY() EITHER!! Let the pointers do their magic!!

What a spoilsport!! :p :p :p

i change already but i get this error cannot convert from 'Record' to 'char *'

You took out the strcpy() calls? And returned the assignment operator: =

so what i have 2 do? change it back to

v

Code:

---

oid Sorter(void)  {
  int i, j;  //these are just used for counting, nothing fancy
  Record *ptr1;
  Record *ptr2;
  Record *ptrtemp; 

  for(i = 0; i < 10-1; i++)  {
      for(j = i+1; j < 10; j++)  {
        ptr1 = &records[i];
        ptr2 = &records[j];
        if(ptr1->total < ptr2->total)  {
            *ptrtemp = *ptr1;
            *ptr1 = *ptr2;
            *ptr2 = *ptrtemp;
        }
      }
  }
}

---

but change the total to student?

Yepper!

WAIT! Now you've taken out the strcmp() !!

We need strcmp() to compare what the two ptr->student things are, and tell us which is greater.

no more error in the compiling but i get debug error it says the variable ptrtemp is being use without initialiased

warning C4700: uninitialized local variable 'ptrtemp' used

so how to initialized ptrtemp?

I don't get any warnings, so I'm not sure.

After the strcmp() line, you should have three lines of code for the ptrs:

Code:

---

*ptrtemp = *ptr1;
*ptr1 = ptr2;
*ptr2 = ptrtemp;

---

Maybe you have a typo or something?

Make sure ptrtemp is NOT mentioned in any other place except where it's declared, and here.

We could add

Code:

---

ptrtemp = NULL;

---

where it's created.

Quote:

---

Originally Posted by Adak

I don't get any warnings, so I'm not sure.

After the strcmp() line, you should have three lines of code for the ptrs:

Code:

---

*ptrtemp = *ptr1;
*ptr1 = ptr2;
*ptr2 = ptrtemp;

---

Maybe you have a typo or something?

Make sure ptrtemp is NOT mentioned in any other place except where it's declared, and here.

We could add

Code:

---

ptrtemp = NULL;

---

where it's created.

---

hey thx alot for helping=]

You're welcome. :)