Błąd przy łancuchy i alokacji pamięci miejscowej

Question

Witam mam problem z łańcuchem znaków i alokacją miejscową pamięci oto kod:

#include <iostream>
#include <new>

struct chaff
{
        char dross[20];
        int slag;
};
int main()
  {      char tab[20], tab_2[20];
        chaff * p1, *p2;
        p1 = new(tab) chaff; //miejscowe przypisanie pamieci
        p1->dross = "ciag znaków";
}

Comments

Ten kod nie ma prawa się skompilować. Wyrażenia muszą się znajdować w funkcjach.

---

Kod został poprawiony

Answer 1

p1->dross = "ciag znaków";

Tablica charów to nie std::string. Takie przypisanie nie ma sensu. Użyj funkcji strcpy.

Comments

użyłem w dalszej części na kopie.

#include <iostream>
#include <new>
#include <cstring>
struct chaff
{
        char dross[20];
        int slag;
};

int main()
{
        char tab[30], tab_2[30];
        chaff * p1, *p2;
        p1 = new(tab) chaff; //miejscowe przypisanie pamieci
        strcpy(p1->dross, "ciag znaków");
        std::cout << "Przed kopiowaniem: " << p1->dross << "\nadres \"p1.dross\": " << &p1->dross << std::endl;
        strcpy (tab, p1->dross);
        std::cout << "SKopiowane: " << tab << "\nadres \"tab\": " << &tab << std::endl;

        p2 = new chaff;
        strcpy(p2->dross, "asfasfafs");
        std::cout << "\n\nPrzed kopiowanie: " << p2->dross << "\nadres \"p2.dross\": " << &p2->dross << std::endl;
        strcpy(tab_2, p2->dross);
        std::cout << "Skopiowane: " << tab_2 << "\nades \"tab_2\": " << &tab_2 << std::endl;
        delete [] p2;
        return 0;
}

zadziałało, jednak dalej mam błąd

warning: placement new constructing an object of type ‘chaff’ and size ‘224’ in a region of type ‘char [30]’ and size ‘30’ [-Wplacement-new=]
  p1 = new(tab) chaff; //miejscowe przypisanie pamieci

Dlaczego, sprawdzałem to kilku krotnie i powinno się kompilować

---

strcpy (tab, p1->dross);

Oba fragmenty pamięci nakładają się na siebie. W tym przypadku będzie undefined behaviour.

delete [] p2;

Tej wersji delete używa się dla pamięci zaalokowanej new[].

---

Dziękuję, udało mi się naprawić ten błąd. Nie wiedziałem, że to może spowodować błąd kompilacji dziękuję jeszcze raz.

---

To nie błąd kompilacji, to ostrzeżenie. Tak w ogóle, jeśli dobrze rozumiem komunikat, typ chaff nie powinien mieć rozmiaru aż 224. Trudno mi cokolwiek na ten temat powiedzieć.

---

Jeśli chcesz uniknąć częściowo problemów z pilnowaniem wielkości buforów, możesz zrobić tak:

        char tab[sizeof(chaff)];
        char tab_2[sizeof(chaff::dross)];

 

Answer 2

Kod jest prawie poprawny (choć bezsensowny, ale mniejsza o to). Otóż nie zmieści ci się obiekt struktury chaff w pamięci o rozmiarze 20 charów. Potrzebowałbyś 24-elementową tablicę dla architektury 32-bitowej, a 28 dla 64-bitowej.

PS Jak zauważył słusznie kolega Qwerty96, powinieneś użyć również funkcji strcpy.