Wskaźnik do stringa

Question

Cześć wszystkim ;)

Mam do wykonania zadanie ósme.

Napisałem taki kod:

#include <iostream>
#include <cstdlib>
#include <string>

using namespace std;

int main()
{
    struct pizzas
    {
        string name;
        int diameter;
        float weight;
    };
    pizzas j;

        cout<<"Podaj nazwe firmy: ";
        cin.clear();
        getline(cin, j.name);
        cout<<"Podaj srednice pizzy: ";
        cin>>j.diameter;
        cout<<"Podaj wage pizzy: ";
        cin>>j.weight;

        cout<<"Dodano pizze! "<<endl;


        pizzas *poiter=&j;
        new (poiter) pizzas;
        cout<<endl<<poiter->name<<endl;
        cout<<poiter->diameter<<endl;
        cout<<poiter->weight<<endl;

        delete poiter;

    return 0;
}

Niestety string w przeciwieństwie do intów nie zostaje wypisany na końcu programu. Czy ktoś mógłby mi podpowiedzieć co robię źle?

Nie wiem też czy poprawnie wykonuje tę część zadania "use new to allocate a structure instead of declaring a
structure variable", ale to jedyne co mi przychodzi do głowy :)

 

Z góry dziękuję za wszelką pomoc.

Comments

Proszę umieszczać kod w specjalnych bloczkach.

---

Dzięki, będę o tym pamiętał.

Answer 1

pizzas *poiter=&j;
new (poiter) pizzas;

Używasz tu placement new do utworzenia nowego obiektu dokładnie w miejscu obiektu j. Czyli faktycznie nadpisujesz obiekt którego pola wypełniałeś cin-em.

Niestety string w przeciwieństwie do intów nie zostaje wypisany na końcu programu. 

Bo został nadpisany. Na jego miejscu powstał nowy string (stworzony z konstruktorem domyślnym, a więc pusty).  Inty są poprawnie wypisywane,b o to typy podstawowe i nie mają własnego konstruktora (nadającego jakąś wartość "na start"), a klasie `pizza` też nie napisałeś żadnego konstruktora, więc pamięć w której są inty pozostaje nieruszona.

delete poiter;

Program ci się nie crashuje? Użyłeś placement new, a nie new. Tzn. nie zaalokowałeś pamięci na stercie, dlatego też nie potrzebujesz jej zwalniać. Nie możesz tak robić.

• new - prosisz system o dodatkową pamięć dla programu i tworzysz na niej obiekt
• placement new - podajesz konkretny adres pamięci i każesz stworzyć tam obiekt. Ofc to musi być adres pamięci należącej już do programu.

 Nie wiem też czy poprawnie wykonuje tę część zadania "use new to allocate a structure instead of declaring a  structure variable", ale to jedyne co mi przychodzi do głowy :)

 Chodziło raczej o najzwyklejsze zaalokowanie obiektu za pomocą new:

pizzas *j = new pizzas;

// cin, cin, cin

delete j;

PS:

1. Dlaczego 'pizzas'? (dlaczego liczba mnoga?)
2. Zdajesz się nie zauważać literówki w kodzie, dlatego wypomne :P 'pointer' zamiast 'poiter' powinno być.

Comments

pizzas zostało jeszcze z zadania 7 w wersji w której  zamknąłem to wszystko w pętli i dodawałem kilka pizz :D natomiast faktycznie nie zauważyłem braku "n" w "poiter"

Brawo za spostrzegawczość ;)

Wracając do programu, nadal nie wiem jak to ugryźć

---

Jak teraz czytam treść zadania, to nie wiem czy masz mieć możliwość dodania wielu pizz i one powinny być gdzieś zapisywane czy po prostu cały czas 'pracujemy' na aktualnie podanej...

---

Pracujemy na aktualnie podanej, ale chciałem sobie zmodyfikować trochę program.

---

Hm no to ten kod co wyżej podałem jest ok (ten z "// cin, cin, cin"). Napisze cały:

pizzas *p = new pizza;

std::getline(std::cin, p->name);
std::cin >> p->diameter;
std::cin >> p->weight;

std::cout << p->name << '\n' << p->diameter << '\n' << p->weight << '\n';

delete p;

---

#include <iostream>
#include <cstdlib>
#include <string>

using namespace std;

int main()
{
    struct pizzas
    {
        string name;
        int diameter;
        float weight;
    };
    pizzas j;


        cout<<"Dodano pizze! "<<endl;

        pizzas *poiter =new pizzas;
        poiter=&j;
        cout<<"Podaj nazwe firmy: ";
        cin.clear();
        getline(cin, j.name);
        cout<<"Podaj srednice pizzy: ";
        cin>>j.diameter;
        cout<<"Podaj wage pizzy: ";
        cin>>j.weight;
        //new (poiter) pizzas;
        cout<<endl<<poiter->name<<endl;
        cout<<poiter->diameter<<endl;
        cout<<poiter->weight<<endl;

        delete poiter;

    return 0;
}

Mam coś takiego. Niby działa, ale wysypuje się przy 2 i każdym kolejnym "odpaleniu".

 

Przy Twoim kodzie mam błąd

"error request for member 'name' in 'p' which is of pointer type" to samo w linijce wpisującej  diameter i weight

---

Przy Twoim kodzie mam błąd

Sorka, edytowałem. Była kropka zamiast ->. 

Mam coś takiego. Niby działa, ale wysypuje się przy 2 i każdym kolejnym "odpaleniu".

Niepotzrebny ci obiekt j. W 20 linijce tworzysz obiekt operatorem new i nic więcej nie potrzebujesz.
Na dodatek masz wyciek pamięci - 21 linia - tracisz adres otrzymany przez new i próbujesz dealokować pamięć ze stosu programu (34 linia) - stąd crash. 

---

Wszystko działa jak należy.

Dzięki wielkie za pomoc.