DaniWeb Forum Index > Software Development > C++

Class Fails at Runtime

Please support our C++ advertiser: Intel Parallel Studio Home

•

Join Date: Jan 2009

Posts: 151

Reputation: Phil++ is an unknown quantity at this point

Solved Threads: 3

Phil++

Offline

Junior Poster

Class Fails at Runtime

Nov 5th, 2009

Hey, I've created this class, it compiles OK but when I go to use the class it fails. It comes up with a pop-up saying something like 'Unexpected Error'

'Class'

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
class Person
{
   public:
          Person();
          Person(char theName[], float thePay, int theHours, bool Isworker);
          char* getName();
          float getPay();
          int getDepartment();
 
          void setName(char theName[]);
          void setPay(float thePay);
          void setHours(int theHours);
          void setSalaried(bool Isworker);
 
   private:
           char* name;
           float pay;
           int hours;
           bool worker;
};
class Person
{
   public:
          Person();
          Person(char theName[], float thePay, int theHours, bool Isworker);
          char* getName();
          float getPay();
          int getDepartment();
          
          void setName(char theName[]);
          void setPay(float thePay);
          void setHours(int theHours);
          void setSalaried(bool Isworker);
          
   private:
           char* name;
           float pay;
           int hours;
           bool worker;
};

'Functions'

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
#include <string>
#include <iostream.h>
#include "person.h"
 
Person::Person(){}
Person::Person(char theName[], float thePay, int theHours, bool Isworker)
{
   strcpy(name, theName);
   pay = thePay;
   hours = theHours;
   worker = worker;                      
}
 
char* Person::getName()
{
   return name;
}
 
float Person::getPay()
{
 
   return pay;     
}
 
int Person::getHours()
{
   return hours;     
}
 
void Person::setName(char theName[])
{
     strcpy(name, theName);
}
void Person:: setPay(float thePay)
{
     pay = thePayRate;
}
void Person::setHours(int theHours)
{
     hours = theHoursWorked;
}
void Person::setSalaried(bool Isworker)
{
   worker = Isworker;    
}
#include <string>
#include <iostream.h>
#include "person.h"

Person::Person(){}
Person::Person(char theName[], float thePay, int theHours, bool Isworker)
{
   strcpy(name, theName);
   pay = thePay;
   hours = theHours;
   worker = worker;                      
}

char* Person::getName()
{
   return name;
}

float Person::getPay()
{
      
   return pay;     
}

int Person::getHours()
{
   return hours;     
}

void Person::setName(char theName[])
{
     strcpy(name, theName);
}
void Person:: setPay(float thePay)
{
     pay = thePayRate;
}
void Person::setHours(int theHours)
{
     hours = theHoursWorked;
}
void Person::setSalaried(bool Isworker)
{
   worker = Isworker;    
}

Any ideas? Thanks.

Last edited by Phil++; Nov 5th, 2009 at 8:01 pm.

If you ask me questions through Private messaging I won't reply.

•

Join Date: Sep 2009

Posts: 576

Reputation: jonsca is on a distinguished road

Solved Threads: 77

jonsca

Offline

Posting Pro

Nov 5th, 2009

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
 
void Person:: setPay(float thePay)
{
     pay = thePayRate;
}
void Person::setHours(int theHours)
{
     hours = theHoursWorked;
}
void Person:: setPay(float thePay)
{
     pay = thePayRate;
}
void Person::setHours(int theHours)
{
     hours = theHoursWorked;
}

Have a problem here with those variable names. Also, does your compiler warn you about iostream.h? Should just be #include <iostream>

•

Join Date: Feb 2008

Posts: 638

Reputation: daviddoria is a jewel in the rough

Solved Threads: 46

daviddoria daviddoria is offline

Offline

Practically a Master Poster

Nov 5th, 2009

First of all, you should #include <iostream> instead of #include <iostream.h> . This is the "new" c++ way to do it.

Second, you have to #include <cstring> to use strcpy.

Third, you have not declared getHours in the header file. To show you this, the compiler says: "error: no 'int Person::getHours()' member function declared in class 'Person'".

Fix these things and let us know if you have any more problems.

David

•

Join Date: Oct 2009

Posts: 59

Reputation: pecet is an unknown quantity at this point

Solved Threads: 11

pecet

Offline

Junior Poster in Training

Nov 6th, 2009

what's more You try to copy string here:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
Person::Person(char theName[], float thePay, int theHours, bool Isworker)
{
strcpy(name, theName);
//...
}
Person::Person(char theName[], float thePay, int theHours, bool Isworker)
{
strcpy(name, theName);
//...
}