Wzkaznik, referencja

pytanie zadane 26 grudnia 2016 w C i C++ przez qwerty Początkujący (380 p.)

Witam!

Czy mógłby mi ktoś pomóc, ten kod mi nie działa, nie mogę znaleźć błędu.

Z góry dziękuję za pomoc!

#include <iostream>
using namespace std;
void spr (string *wznaimie, string &imie)
{
    if(wznaimie == NULL)
    {
        cin>>imie;
        wznaimie = &imie;
    }
}
int main ()
{
    string imie;
    string *wznaimie = NULL;
    spr(wznaimie,imie);
    cout<<imie;
    cout<<*wznaimie;
    return 0;
}

5 odpowiedzi

odpowiedź 26 grudnia 2016 przez L33TT12 Gaduła (3,950 p.)
wybrane 26 grudnia 2016 przez qwerty

Najlepsza

Musisz pobrać referencję od wskaźnika, dodaj to do funkcji
void spr (string *&wznaimie, string &imie)
i wszystko zadziała.

komentarz 26 grudnia 2016 przez qwerty Początkujący (380 p.)

Dzieki wielkie działa!

odpowiedź 26 grudnia 2016 przez Knayder Nałogowiec (37,640 p.)

Musisz załączyć bibliotekę string.

#include <string>

komentarz 26 grudnia 2016 przez Knayder Nałogowiec (37,640 p.)

Ponieważ w niej znajduje się przeładowanie operatorów >> oraz << dla obiektów klas input, output stream i stringów.

komentarz 26 grudnia 2016 przez qwerty Początkujący (380 p.)

Wciąż nie działa ;C

komentarz 26 grudnia 2016 przez Knayder Nałogowiec (37,640 p.)

Chodzi ci o to, że kod ci się nie kompiluje, czy że nie działa poprawnie?

komentarz 26 grudnia 2016 przez qwerty Początkujący (380 p.)

Wpisuję imie, później wyswietla się imie, ale na końcu pojawia się komunikat, że program przestał działać (nie wyswietla *wznaimie).

odpowiedź 26 grudnia 2016 przez unknown Nałogowiec (39,560 p.)

Dereferencja NULL-pointera to UB. W 99,9% przypadków powoduje crasha'a.

komentarz 26 grudnia 2016 przez qwerty Początkujący (380 p.)

Co to ub?

komentarz 26 grudnia 2016 przez unknown Nałogowiec (39,560 p.)

undefined behavior

komentarz 26 grudnia 2016 przez qwerty Początkujący (380 p.)

Czyli, ze nie da się nic zrobić?

komentarz 26 grudnia 2016 przez unknown Nałogowiec (39,560 p.)

Można. Nie rób dereferencji na NULL-pointer'rze

odpowiedź 26 grudnia 2016 przez fenq80 Bywalec (2,940 p.)

W linijce 9 napisz *wznaimie = imie; ponieważ gdy wewnątrz tej funkcji przypiszesz nowy adres do tego wskaźnika wtedy ten wskaźnik nie będzie już wskazywał na miejsce w pamięci na które wskazuje zmienna wznaimie zadeklarowana w funkcji main. Wten sposób funkcja nie zmieni wartości wskazywanej przez wskaźnik wznaimie (tej w mainie) wiec w linijce 17 próbujesz wyświetlić pusty wskaźnik (to nie może się skończyć dobrze)

komentarz 26 grudnia 2016 przez qwerty Początkujący (380 p.)

Niestety również nie działa.

komentarz 26 grudnia 2016 przez fenq80 Bywalec (2,940 p.)

W funkcji zamiast 8 lini napisać wznaimie =new string (name) ;

odpowiedź 26 grudnia 2016 przez JAKUBW Nałogowiec (33,470 p.)

Przekazujesz wskaźnik który jest kopiowany. Przekaż orginał wskaźnika(wskaźnik do wskaźnika)

#include <iostream>
#include <string>

using namespace std;

void spr (string **wznaimie, string &imie)
{
    if(*wznaimie == NULL)
    {
        cin>>imie;
        *wznaimie = &imie;
    }
}
int main ()
{
    string imie;
    string *wznaimie = NULL;
    spr(&wznaimie,imie);
    cout<<imie<<endl;
    cout<<*wznaimie;
    return 0;
}