istream operator >> czy tak można?(new)

Question

1) czy można napisać operator>> w ten sposób;

#include <iostream>
struct in {
    double* var;
    ~in() {
        delete var;
    }
};
std::istream & operator >> (std::istream & is, in &v) {

    double * zmienna = new double;
    is>>*zmienna;
    v.var=zmienna;
    return is;
    //żeby nie tak
    // double zmienna;
    // v.set(zmienna);

}
using namespace std;

int main() {
    in obj;
    cin>>obj;
    cout <<*(obj.var)<< endl;

    return 0;
}

2) czy tak się stosuje? ewentualnie bez tego delete tylko smart ptr (w razie throw);

3) czy niema wycieku pamięci? 

Comments

Nie zaciemnia to kodu?

---

niby trochę, ja tak z ciekawości. Bo podobno stos jest ograniczony. A nie chcę do wszystkiego od razu map czy wektora stosować.

Answer 1

1. Zmienna var jest niezainicjalizowana. Jeżeli jej nie ustawisz to destruktor struktury może wywalić program.

2. Wyciek pamięci nastąpi jeżeli drugi raz prześlesz dane do struktury.

Po wprowadzeniu delikatnych zmian powinno działać bezproblemowo.

#include <iostream>
struct in {
    double* var=nullptr;
    ~in() {
        delete var;
    }
};
std::istream & operator >> (std::istream & is, in &v) {
    if (v.var == nullptr)
        v.var = new double;
    is>>*v.var;
    return is;
}
using namespace std;
 
int main() {
    in obj;
    cin>>obj;
    cout <<*obj.var<< endl;
 
    return 0;
}

 

Comments

Dzięki bardzo;

void set (double *ptrvar){var=ptr;} to nic nie zmienia ?

a jak lepiej przez kopię czy wskaźnik?

---

Powodujesz wyciek pamięci.

void set (double *ptrvar) {
    if (var != nullptr)
        delete var;
    var=ptr;
}

Wydaje mi się, że przekazanie wartości jest lepsze. Unikasz ryzyka związanego ze zwolnieniem pamięci w innym miejscu programu.
 

void set (double value) {
    if (var == nullptr)
        var = new double;
    *var = value;
}

 

---

Dzięki.