Employee.cpp

// payment function
#include<iostream>
#include<string>
#include<conio.h>
#include "Employee.h"
using namespace std;
float Employee::payment()
{
return pay*hours+pay*1.5*exhours;
}
// set function
void Employee::set()
{
	cout<<"Enter ID : ";
	cin>>id;
	cout<<"Enter pay per hour : ";
	cin>>pay;
	cout<<"Enter hours :";
	cin>>hours;
	cout<<"Enter extra hours : ";
}
//show function
void Employee::show()
{
cout<<"ID : \n"<<id;
cout<<"Payment : \n"<<payment();
}

Employee.h

class Employee
{
private :
int hours,exhours,id;
float pay;
public :
float payment();
void show();
void set();
};

main.cpp

#include<iostream>
#include<string>
#include<conio.h>
#include "Employee.h"
#include "Employee.cpp"
using namespace std;
void main ()
{
int i;
Employee *a;
a=new Employee;
for(i=0;i<2;i++)
a.set();
for(i=0;i<2;i++)
{
a.show();
cout<<"\n\n";
}
}

נתחיל מה main
1) ה include ל string מיותר
2) conio.h ספרייה של בורלנד לא שייכת לאף תקן לא C ולא C++ אין לי מושג למה אתה עושה לה INCLUDE, ככלל אצבע אל תשתמש בה.
3) לא עושים INCLUDE לקובץ CPP - מספיק לקובץ H
4) הפונקציה הראשית צריכה להיות מסוג int ולהחזיר למערכת הפעלה 0 במידה והתוכנית הצליחה.
5) שאתה מגדיר מצביע אתה פונה אליו עם אופרטור חץ a->show() (לדוגמא) או עם כוכבית ואז נקודה.
6) שאתה מסיים עם מצביע תמיד תמיד למחוק אותו- delete a
7) ב C++ תקני לא צריך להשתמש ב\n אלא ב endl
לדוגמא std::cout << "hello" << endl;

לגבי המחלקה עצמה
1) נהוג קודם כל לרשום את המתודות ורק אז את ה DATA MEMBERS וגם לתת להם שמות עם קידומת m_ כדי שתדע שהם שייכים למחלקה.
2) נהוג לא להשתמש ב float מטעמי דיוק. יש להשתמש ב double
3) כל ההערות שהערתי על הקובץ הראשי חלות גם פה