illegal Else ???

can any one see my mistakes for illegal else and

Quote:

---

//chech if there an Appointmen on the same Date and Time!

---

seems not working

Code:

---

void addAppointmenRecords( )
{ 
        int number, i;
        system("cls");        //clear screen

                cout<<"\nHow many Appointmen you wish to add? ";
                cin>>number;                  //read number
                cin.get();            //read newline character left in the buffer

        if( (number + currentSize ) <= listSize)        //There is still room in the array       
          for( i = 1; i<=number; i++)
          {
                        cout<<"\nEnter Person name: ";
                        getline(cin, AppointmenList[currentSize].name);
                        cout<<"Enter Appointmen Descriptions: ";
                        getline(cin, AppointmenList[currentSize].description);
                        cout<<"Enter Appointmen date: ";
                        cin>>AppointmenList[currentSize].appdate.day;
                        cin>>AppointmenList[currentSize].appdate.mounth;
                        cin>>AppointmenList[currentSize].appdate.year;
                        cin.get();                                                        //read a character
                        cout<<"Enter Appointmen time: ";
            getline(cin, AppointmenList[currentSize].time);
                        cout<<endl;
                        currentSize += 1; //update CurrentSize
                       
                        for(int k=0; k<currentSize; k++)  //chech if there an Appointmen on the same Date and Time!
                        if (AppointmenList[currentSize].appdate.day == AppointmenList[k].appdate.day &&
                                AppointmenList[currentSize].appdate.mounth == AppointmenList[k].appdate.mounth &&
                                AppointmenList[currentSize].appdate.year == AppointmenList[k].appdate.year &&
                                AppointmenList[currentSize].time == AppointmenList[k].time)
                        cout<<"\nGot a match\n";
                                cout<<"Appointmen Date and Time already filled"<<endl;
                                currentSize-=1;
                                break;
                        else
                       
                                cout<<"Appointmen accepted"<<endl;
                        currentSize+=1;               
                }
        else
                cout<<"Overflow!!!! Appointmen List is full"<<endl;
                cout<<"\nPress any key to continue"<<endl;
                cin.get();    //read a character
}

---

Yeah. You're assuming that indenting code is enough to tell the compiler that code should be grouped. It is not.

The code;

Code:

---

if (X())
    a();
    b();
else
    c();

---

actually looks to the compiler like

Code:

---

if (X())
    a();

b();
else
c();

---

The call of b() is not one of the statements affected by the if() statement. And the else keyword does not match anything.

You need to learn to use braces {} to group lines of code, not indenting with whitespace. Because the compiler ignores whitespace (except in character strings).

You have several examples of this sort of thing in your code needing fixing.

thaks grumpy i fixed like this, im not getting any error or worning now but i still have problem.
what is wrong with my

Quote:

---

//chech if there an Appointmen on the same Date and Time!

---

Code:

---

void addAppointmenRecords( )
{ 
        int number, i;
        system("cls");        //clear screen

                cout<<"\nHow many Appointmen you wish to add? ";
                cin>>number;                  //read number
                cin.get();            //read newline character left in the buffer

        if( (number + currentSize ) <= listSize)        //There is still room in the array
        {
          for( i = 1; i<=number; i++)
          {
                        cout<<"\nEnter Person name: ";
                        getline(cin, AppointmenList[currentSize].name);
                        cout<<"Enter Appointmen Descriptions: ";
                        getline(cin, AppointmenList[currentSize].description);
                        cout<<"Enter Appointmen date: ";
                        cin>>AppointmenList[currentSize].appdate.day;
                        cin>>AppointmenList[currentSize].appdate.mounth;
                        cin>>AppointmenList[currentSize].appdate.year;
                        cin.get();                                                        //read a character
                        cout<<"Enter Appointmen time: ";
                                                getline(cin, AppointmenList[currentSize].time);
                        cout<<endl;
                        currentSize += 1; //update CurrentSize
                       
                        for(int k=0; k<currentSize; k++)  //chech if there an Appointmen on the same Date and Time!
                        if (AppointmenList[currentSize].appdate.day == AppointmenList[k].appdate.day &&
                                AppointmenList[currentSize].appdate.mounth == AppointmenList[k].appdate.mounth &&
                                AppointmenList[currentSize].appdate.year == AppointmenList[k].appdate.year &&
                                AppointmenList[currentSize].time == AppointmenList[k].time)
                        {
                                                cout<<"\nGot a match\n";
                                cout<<"Appointmen Date and Time already filled"<<endl;
                                currentSize-=1;
                        }
                        else
                        {
                                cout<<"Appointmen accepted"<<endl;
                                currentSize+=1;
                        }
        }
}
        else
        {
                cout<<"Overflow!!!! Appointmen List is full"<<endl;
                cout<<"\nPress any key to continue"<<endl;
                cin.get();    //read a character
        }
}

---

What is wrong with what? If you at least explain what sort of problem we are looking for, we can help a bit better?

--
Mats

sorry matsp
this is my entry

Quote:

---

How many Appointmen you wish to add? 2

Enter Person name: zafer
Enter Appointmen Descriptions: lunch
Enter Appointmen date: 12 12 2009
Enter Appointmen time: 12:30

Appointmen accepted

Got a match
Appointmen Date and Time already filled

Enter Person name: zafer
Enter Appointmen Descriptions: dinner
Enter Appointmen date: 12 12 2009
Enter Appointmen time: 20:30
Appointmen accepted

Got a match
Appointmen Date and Time already filled

---

its not checking if there an Appointmen on the same Date and Time!
and

Quote:

---

Appointmen accepted

---

and

Quote:

---

Got a match
Appointmen Date and Time already filled

---

fallowing each other. i couldent work out.