Keine Ausgabe mit einem Rückgabefehler von 1 bei der Verwendung von Zeigern auf char in C++

Question

Ich habe gründlich nach einer Antwort ohne Erfolg gesucht. Ich habe angefangen mit C++ - Gaddis-Lehrbuch als Referenz anzufangen. Ich weiß, dass es eine einfache Antwort auf dieses Problem gibt, aber ich habe Probleme, es herauszufinden. Hier ist mein Code:

class Employee 
{ 
    private: 
    char *name; 
    int idNumber; 
    char *department; 
    char *position; 
    void initName(const char *n) 
{ 
    name = new char[strlen(n) + 1]; 
    strcpy(name, n); 
} 
void initDepartment(char *d) 
{ 
    department = new char[strlen(d) + 1]; 
    strcpy(department, d); 
} 

void initPosition(char *p) 
{ 
    position = new char[strlen(p) + 1]; 
    strcpy(position, p); 
} 

public: 

Employee() 
{ 
    strcpy(name, ""); 
    idNumber = 0; 
    strcpy(department, ""); 
    strcpy(position, ""); 

} 
Employee(char *nam, int num, char *dep, char *posit) 
{ 
    initName(nam); 
    idNumber = num; 
    initDepartment(dep); 
    initPosition(posit); 
} 

Employee(const char *nam, int num) 
{ 
    initName(nam); 
    idNumber = num; 
    strcpy(department, ""); 
    strcpy(position, ""); 
} 

~Employee() 
{ 
    delete [] name; 
    delete [] department; 
    delete [] position; 
} 

void setName(char *n) 
{ 
    name = new char[strlen(n) + 1]; 
    strcpy(name, n); 
} 

void setIdNumber(int num) 
{ 
    idNumber = num; 
} 

void setDepartment(char *d) 
{ 
    department = new char[strlen(d) + 1]; 
    strcpy(department, d); 
} 

void setPosition(char *p) 
{ 
    position = new char[strlen(p) + 1]; 
    strcpy(position, p); 
} 

const char * getName() const 
{ 
    return name; 
} 

int getIdNumber() const 
{ 
    return idNumber; 
} 

const char * getDepartment() const 
{ 
    return department; 
} 

const char * getPosition() const 
{ 
    return position; 
} 
}; 

int main(int argc, char** argv) 
{ 
const int SIZE = 50; 
const char name[SIZE] = "Mark Jones"; 
Employee employee(name, 3452); 
const char *ptr = NULL; 

ptr = employee.getName(); 

cout << ptr << endl; 

return 0; 
} 

Answer 1

Im Standardkonstruktors Sie leere Zeichenfolge in den Papierkorb Zeiger kopieren:

Employee() 
// name and all other members are not initialized 
{ 
    strcpy(name, ""); // << copy to unallocated memory 
    idNumber = 0; 
    strcpy(department, ""); // << and here 
    strcpy(position, ""); // << and here 
} 

Der richtige Standardkonstruktors würde sein:

Employee() 
: name(nullptr) // Initialize all members here 
, idNumber(0) 
, department(nullptr) 
, position(nullptr) 
{ 
    // Initialize c-strings 
    initName(""); 
    initDepartment(""); 
    initPosition(""); 
} 

Sie const alle benutzen zusammen nicht vergessen. Und in ähnlicher Weise andere Konstrukteure:

Employee(const char* nam, int num, const char* dep, const char* posit) 
: name(nullptr) // << Initialize all members here 
, idNumber(num) // << Initialize number with proper value 
, department(nullptr) 
, position(nullptr) 
{ 
    initName(nam); 
    initDepartment(dep); 
    initPosition(posit); 
} 

Employee(const char* nam, int num) 
: name(nullptr) // Initialize all members here 
, idNumber(num) // << Initialize number with proper value 
, department(nullptr) 
, position(nullptr) 
{ 
    initName(nam); 
    initDepartment(""); 
    initPosition(""); 
} 

Proper Funktionen Parameter:

void initName(const char *n) 
{ 
    name = new char[strlen(n) + 1]; 
    strcpy(name, n); 
} 
void initDepartment(const char *d) 
{ 
    department = new char[strlen(d) + 1]; 
    strcpy(department, d); 
} 

void initPosition(const char *p) 
{ 
    position = new char[strlen(p) + 1]; 
    strcpy(position, p); 
} 

Und vergessen Sie nicht Speicher in Setter zu löschen (sonst ist es ein memleak):

void setName(const char *n) 
{ 
    delete[] name; 
    name = new char[strlen(n) + 1]; 
    strcpy(name, n); 
} 

void setDepartment(const char *d) 
{ 
    delete[] department; 
    department = new char[strlen(d) + 1]; 
    strcpy(department, d); 
} 

void setPosition(char *p) 
{ 
    delete[] position; 
    position = new char[strlen(p) + 1]; 
    strcpy(position, p); 
} 

Diese sollte alle Ihre Laufzeitfehler beheben.

Answer 2

Verwenden Sie lieber die Elementfunktionen im Konstruktor. Die Zeiger werden nicht initialisiert, wenn der Konstruktor beginnt, aber die Mitgliederfunktionen wird sie initialisiert werden:

Employee(const char *nam,int num) 
{ 
    initName(nam); 
    idNumber = num; 
    //strcpy(department,""); 
    //strcpy(position,""); 
    initDepartment(""); 
    initPosition(""); 
}