Dziwny wynik podzczas przypisania tablicy.

Question

#include <iostream>
#include <time.h>
#include <string>

#define BL cout<<endl;

using namespace std;

struct text {

    int rozmiar = 100;
    char *wsk = new char[rozmiar];




};

void zapisz(text&);



int main()
{

    text tekst;

    zapisz(tekst);
    
        cout << tekst.wsk;

    

    
    return 0;
}


void zapisz(text &tekst) {

    char napis[20];
    cout << "Podaj tekst: ";
    cin >> napis;
    

    tekst.wsk = napis;



}

Answer 1

Żeby nie było żadnych wycieków pamięci i chcesz żeby pamięć tablicy była przydzielana dynamicznie zastosuj odpowiedni konstruktor który przydzieli pamięć i przypiszę do niej wartość, użycie operatora new w konstruktorze wymaga odpowiedniego destruktora z operatorem delete. Do zmieniania wartości tablicy możesz zdefiniować funkcje wewnątrz structa. Przecież jest on traktowany jako klasa z publicznym dostępem do pól i metod składowych. Przedmówca wytłumaczył już czemu wychodzą takie dziwne znaki, przez co nie muszę tego tłumaczyć. 

O to przykład jak można skonstruować taką struktura z dynamiczna tablica.

#include<iostream>
#include<cstring>

struct Text{
    char* wsk;
    Text(const char* tekst)
    {
        wsk = new char[strlen(tekst)+1];
        strcpy(wsk,tekst);
    }
    ~Text()
    {
        delete[] wsk;
    }


};

int main()
{
    Text tekst("TEKST KTORY CHCE ZAPISAC");
    std::cout << tekst.wsk;
    return 0;
}

 

Comments

Z tego co widzę, to tekst jest wskaźnikiem w tym konstruktorze, skoro tak, to w jaki sposób string "TEKST KTORY CHCE ZAPISAC" możemy wpisać do wskaźnika?

---

Funkcja strcpy :)

Answer 2

Ten kod nie powinien się nawet skompilować.

Comments

Poprawiłem, teraz się kompiluje. Tylko kompletnie nie wiem dlaczego to nie działa.

---

1. Nie używaj makr

2. Masz wyciek pamięci

3. W funkcji zapisz do tekst.wsk przypisujesz adres tablicy, która przestaje istnieś w chwili wyjścia z funkcji. UB

---

Tylko jak powinien wyglądać ten kod jeżeli chce tam ten napis co podalem przez cin dać do tej dynamicznej tablicy, a nie adres.