Dobra praktyka inicjalizacji tablicy char w constructorze

Question

W jaki sposób inicjalizujecie tablice char w konstruktorze, jeśli nie wiecie jakiej długości przyjdzie?

Pętla for + jakieś sizeof'y żeby sprawdzić długość czy jeszcze jakiś inny pomysł?

Answer 1

1. Nie używam raw-arrays, jeśli nie muszę... Od napisów jest std::string. Od liczb są inne kontenery, jak std::array czy std::vector.

2. Jeśli muszę, pomijam pierwsze zdanie punktu pierwszego.

 

...a od sprawdzenia długości napisu na char[], jest funkcja strlen z nagłówka cstring.

Comments

No tak oczywiście że są string'i, ale pomagam czasem rozwiązać jakieś zadania studenckie i tam stety czy nie prowadzący często ustalają jasne reguły gry. (tzn. jak robisz coś komuś, to lepiej żeby on już się nie kłócił ani sobie problemów nie robił (bo i po co?), tylko miał to cacy zrobione.

---

Może czas nauczyć wykładowców, żeby się ogarnęli i zaczęli uczyć dobrych praktyk. :)

Jeśli obiekt ma przechowywać w tablicy char napis, podany w konstruktorze, to robi się to jakoś tak:

#include <cstring>
#include <iostream>

class Foo
{
    char *string;
public:
    Foo(const char *string)
    {
        size_t length = strlen(string);
        this -> string = new char[length+1];
        strcpy(this -> string, string);
    }

    Foo(const Foo &f)
    {
        size_t length = strlen(f.string);
        this -> string = new char[length+1];
        strcpy(this -> string, f.string);
    }

    Foo& operator=(const Foo &f)
    {
        if(&f == this) return *this;

        delete[] this->string;
        size_t length = strlen(f.string);
        this -> string = new char[length+1];
        strcpy(this -> string, f.string);
        return *this;
    }

    ~Foo()
    {
        delete[] string;
    }

    void print() const
    {
        std::cout<<string<<"\n";
    }

    void modify()
    {
        string[0] = 'Z'; // jakaś operacja bez sensu...
    }
};

int main()
{
    char str[] = "Abc";
    Foo bar1(str);

    str[2] = 'C';
    Foo bar2(str);

    Foo bar3("Cba");
    Foo bar4 = bar2;

    bar1.print();
    bar2.print();
    bar3.print();
    bar4.print();

    std::cout<<"Po modyfikacji:\n";
    bar2.modify();
    bar2.print();
    bar4.print();

    std::cout<<"Przypisanie:\n";
    bar4 = bar2;
    bar2.print();
    bar4.print();

    return 0;
}

 

---

Masz mały błąd, dla cstringów trzeba zaalokować więcej pamięci(strlen + 1) i zakończyć tekst zerem('\0').

---

Masz rację, i to nie taki mały ten błąd. Moje niedopatrzenie. Poprawiam, żeby ktoś się nie sugerował tym potem, dziękuję bardzo. :) +1

Answer 2

Do przechowania tablicy wystarczy wskaźnik (dla tablicy jednowymiarowej) lub wskaźniki (dla tablicy wielowymiarowej).

Comments

Chodzi mi o stworzenie konstruktora który otrzymuje/zapisuje tablice charów, jaka jest najlepsza praktyka.

---

#include <iostream>

class Klasa
{
public:
	Klasa(char* tablica)
	{
		tablica_ = tablica;
	}
	void wypisz()
	{
		std::cout << tablica_ << std::endl;
	}
private:
	char* tablica_;
};

int main()
{
	char tablica[] = "Ala ma kota";
	Klasa obiekt(tablica);
	obiekt.wypisz();
	std::cin.get();
	return 0;
}

 

---

Patrycjerz, przemyśl swoje rozwiązanie. ;)

---

Chodziło o średnik, czyż nie? 

---

Nie, o rozwiązanie. W przypadku, który podajesz, kopiujesz tylko wskaźnik, nie wartości. To może powodować przykre konsekwencje, jeśli np. ów wskaźnik pokazywał na tablicę dynamiczną a pamięć została zwolniona.

---

Oczywiście, ale mój kod pokazuje jedynie sposób na przesłanie tablicy do klasy - zabezpieczenia oraz słuszność tego rozwiązania pozostawiam w gestii każdego z osobna. Osobiście użyłbym jakiegoś kontenera z biblioteki standardowej, ale jak już wcześniej mówiłem, to jest tylko przykład, w celach edukacyjnych.