Thread: Char Arrays

I have been working with a program and am almost finished. I am at the last few parts which of course are going to give me the most trouble.

In this part of the code, the user enters a name in which to edit. The name is located in a file. The program is supposed to search for the name, and then prompt the user to enter what the name should be changed to. But before it gets to that point, I get an illegal operation. That nice little message box that termates programs. I have been working on it for 2 days now, commenting and testing, but I am unable to find out specifically what line it is because it seems to move. Here is the source code for the functions:

Code:

void editfile()
 {
  system("cls");
  printf("Enter the name of the contact you wish to edit: ");
  cin.ignore(800, '\n');
  cin.getline(search, 256);
  strcat(fcontact, search);
  
  system("cls");
  printf(" *****************************************************************");
  printf("\n|          What do you want to do with this contact?              |");
  printf("\n|          1. Edit the contact's name                             |");
  printf("\n|          2. Edit the contact's phone number                     |");
  printf("\n|          3. Delete the contact                                  |");
  printf("\n *****************************************************************");
  printf("\nSelection: ");
  cin >> input;
  switch(input)
  {
   case 1: changename();
           break;
    
   case 2: changenumber();
           break;
   
   case 3: deletecontact();
           break;
   
   default: fileoptions();
            break;
  }         
 }     
 
 void changename()
 {
  fin.open(profilename); 
  while(!fin.eof())
  {
   i=0;
   fin.getline(szEdit[i], 256);
   i++;
   if (stricmp(fcontact, szEdit[i]) == 0)
   {
    fin.close();
    j = i;
    system("cls");
    printf("Contact found!");
    Sleep(1500);
    system("cls");
    printf("Enter the new name: ");
    
    // Does not seem to make it to this point.
    
    cin >> name;
    strcat(econtact, name);
    strcpy(name, econtact);
    strcpy(szEdit[j], name);
    fin.close();
    fin.open(profilename);
    while(!fin.eof()) 
    {
     i = 0;
     fin.getline(szEdit[i], 256);
     i++; 
    }
    k = i;
    strcpy(szEdit[j], name);
    fin.close();
    fout.open(profilename);
    while(i<=k)
    {
     i = 0;
     fout << szEdit[i];
     i++;
    }
    system("cls");
    printf("Editing complete!");
    fout.close();
    Sleep(1500);   
   }
  }  
 }

I made a little comment for the part the program cannot seem to get to. Thanks for all your help!

Right, let me post all my variables, and yes, they are all global.

Code:

char search[256]
char fcontact[256] = "Contact: ";
int input;

char profilename[100];
char *szEdit[10000];
char name[256];
char econtact[256] = "Contact: ";

I made sure that if it failed it returned to the main menu, but it still pulled out the illegal error instead of returning to the main menu so it must not have failed.

Ok I did that but the error still exists. I did some tests and found that the line that it stopped at was the if statement:

Code:

if (stricmp(fcontact, szEdit[i]) == 0)

I am not sure if that is the actual problem, that is the line that the error appears at.

I am incrementing "i" so that each time the loop repeats it will place the next line into the next slot of the array. What I am trying to do is make it so a person can edit certain information in a file, the way I am performing this is by bringing in all the lines, edit the one line that they want, and then put it back with the changed information. That is why I am comparing the strings. If you know a better way to do this I am all ears. Thanks!

Oh ok, I see now, Ill try that and see if that works, Thanks!

I made the changes but still it gives the error, unless again I am missing something. Here is what it looks like with all the changes:

Code:

void changename()
 {
  fin.open(profilename); 
  if(fout.fail())
  {
   printf("Error, Returning to main menu");
   menu();
  }
  while(!fin.eof())
  {
   system("cls");
   i=0;
   for(i = 0; i < 500; i++)
   {
    szEdit[i] = new char[256];
   } 
   fin.getline(buffer, 256);
   strcpy(szEdit[i], buffer);
   if (stricmp(fcontact, szEdit[i]) == 0)
   {
    fin.close();
    j = i;
    system("cls");
    printf("Contact found!");
    Sleep(1500);
    system("cls");
    printf("Enter the new name: ");
    
    cin >> name;
    strcat(econtact, name);
    strcpy(name, econtact);
    strcpy(szEdit[j], name);
    fin.close();
    fin.open(profilename);
    if(fout.fail())
    {
     printf("Error, Returning to main menu");
     menu();
    }
    while(!fin.eof()) 
    {
     i = 0;
     fin.getline(szEdit[i], 256);
     i++; 
    }
    k = i;
    strcpy(szEdit[j], name);
    fin.close();
    fout.open(profilename);
    if(fout.fail())
    {
     printf("Error, Returning to main menu");
     menu();
    }
    while(i<=k)
    {
     i = 0;
     fout << szEdit[i];
     i++;
    }
    //system("cls");
    printf("Editing complete!");
    fout.close();
    Sleep(1500);   
   }
   else
   {
    i++;
   }
  }
 }

Heh, thanks for pointing that out, but that still does not fix the illegal error problem. Could it have something to do with my compiler? I am using the newest Bloodshed Dev C++. And I just tried the older one. And both failed.