Problem z konstruktorami

Question

Cześć robie zadanie z ksiazki i przy kompilacji wywala mi blad

wiecie co zle napisalem z gory bardzo dziekuje za pomoc :)

#include <iostream>

using namespace std;
class Person {
    static const int LIMIT = 256;
    string lname;
    char fname[LIMIT];

public:
    Person() {
        lname = "";
        fname[0] = '/0';
    }
    Person(string & ln, const char * fn = "HejTy") {
        lname = ln;
        fn = fname;
    }
    Person(string & ln,  char * fn) {
        lname = ln;
        fn = fname;
    }

    void Show()  const {
        cout << fname << " " << lname<<endl;
    }

    void FormalShow() const {
        cout << lname << " " << fname<<endl;
    }

};
int main()
{
    Person one;
    Person two("Staszek");
    Person three("Jacek", "Placek");
    one.Show();
    one.FormalShow();
    two.Show();
    two.FormalShow();
    three.Show();
    three.FormalShow();
    return 0;
}

 

Comments

Myślę, że pomocne by było gdybyś napisał chociaż treść owego błędu. :)

Answer 1

fname[0]='/0' -> fname[0]='\0'

Nie wiem po co używasz const char*, skoro masz stringa... którego też używasz. 

Z innych rzeczy
 1. Używasz VLA, które raczej nie jest dobrą praktyką.
 2. W konstruktorach z argumentami przypisujesz do argumentu zmienną z klasy
 3. Nie widzę sensu robienia void jako const.
 4. W konstruktorze możesz użyć listy inicjalizacyjnej
 5. Kod Ci się nie skompiluje w przypadku np.
 

Person two ("Staszek")
Person three("Jacek", "Placek")

z uwagi na użycie r-val, gdzie konstruktor oczekuje l-val 
Poczytaj o Rule of Five (kiedyś Rule of Three, ale nagle BANG! i C++11)

Comments

To jeszcze dwa błędy zostały...

---

Dzięki, poprawiłem.

---

Robie tak bo mam tak  w książce a zadaniem było dopisać dwa konstruktory oraz metody show i formlshow

---

Doczepiłbym się do tego, że w kodzie nie ma VLA. Może i dodanie const do deklaracji funkcji nie jest koniecznie, aby kod działał, ale to dobra praktyka i pomaga wykrywać błędy w bardziej złożonych problemach.

Answer 2

Jakie środowisko (np. CodeBlocks)

Jakie błędy? :/