undeclared temp

[lcmosley]

I am not really sure what type to declare "temp". Here's my code.

#include <fstream>
#include <iostream>
#include <iomanip>
#include <cstring>
using namespace std;

void main()
{
  struct{int chronum; char pname[25]; int next;} plist[50];
  int n=0, i, end, maxindex;

  ifstream fin;
  fin.open("prez.dat");
  fin >> plist[n].pname;

  while(!fin.eof())
  {
    plist[n].chronum = n+1;
    n++;
    fin >> plist[n].pname;
  }
  
  for(end=n-1; end>0; end--)
  {
    maxindex = 0;
    for(i=1; i<=end; i++)
    {
      if(strcmp(plist[i].pname, plist[maxindex].pname)>0)
        maxindex=i;
      temp = plist[maxindex];
      plist[maxindex]=plist[end];
      plist[end]=temp;
    }
  }
  
  cout << i << plist[i].chronum << setw(30) << left << plist[i].pname << plist[i].next << endl;
}

[Athar]

You shouldn't declare temp at all. Use std::swap (and while you're at it, std::sort).

[Pravesh Koirala]

or,, you can do like

#include <fstream>
#include <iostream>
#include <iomanip>
#include <cstring>
using namespace std;

void main()
{
  struct{int chronum; char pname[25]; int next;} plist[50],temp;
  int n=0, i, end, maxindex;

  ifstream fin;
  fin.open("prez.dat");
  fin >> plist[n].pname;

  while(!fin.eof())
  {
    plist[n].chronum = n+1;
    n++;
    fin >> plist[n].pname;
  }
  
  for(end=n-1; end>0; end--)
  {
    maxindex = 0;
    for(i=1; i<=end; i++)
    {
      if(strcmp(plist[i].pname, plist[maxindex].pname)>0)
        maxindex=i;
      temp = plist[maxindex];
      plist[maxindex]=plist[end];
      plist[end]=temp;
    }
  }
  
  cout << i << plist[i].chronum << setw(30) << left << plist[i].pname << plist[i].next << endl;
}

edit:added code tag,fixed [b] tag