Wskaźnik czy dobrze? Problem ze zwolnieniem pamięci;

pytanie zadane 5 września 2016 w C i C++ przez hubercik Użytkownik (560 p.)
zamknięte 5 września 2016 przez hubercik

Witam! :)

Czy kod tego o to "kalkulatora" na wskaźnikach jest dobry? Tzn. czy dobrze wykorzystałem wskaźniki :D




#include <iostream>
using namespace std;

int main()
{
    for(;;)
    {
    int x,y;
    int *e, *r;
    cout<<"Podaj pierwszą liczbę: ";
    cin>>x;
    e=&x;
    cout<<"Podaj drugą liczbę: ";
    cin>>y;
    r=&y;
        
    cout<<"Main menu:"<<endl;
    cout<<"1.Dodawanie"<<endl;
    cout<<"2.Odejmowanie"<<endl;
    cout<<"3.Mnożenie"<<endl;
    cout<<"4.Dzielenie"<<endl;
    cout<<"5.Wyjście"<<endl;
    cout<<"wybierz: ";
    int *w;
    int wybierz;
    cin>>wybierz;
    w=&wybierz;
    switch(*w)
    {
        case 1: cout<<"Suma="<<x+y<<endl;    break;
        case 2: cout<<"Różnica="<<x-y<<endl; break;
        case 3: cout<<"Iloczyn"<<x*y<<endl;  break;
        case 4: cout<<"Iloraz"<<x/y<<endl;   break;
        case 5: exit(0);
        
        default: { cout<<"Nie ma takiej opcji w menu, jestem zmuszony do zamknięcia programu!"<<endl; exit(0); } break;
            
    }
    
    }
   
    
    return 0;
}

Ps. Mam problem ze zwolnieniem pamięci w tym programie tzn. próbowałem na różne sposoby i coś mi nie wychodziło :D

Z góry dzięki za odpowiedź :D

komentarz zamknięcia: Priv - rozwiązanie

2 odpowiedzi

odpowiedź 5 września 2016 przez MichuDev Pasjonat (20,300 p.)

Nigdzie nie ma delete, ani funkcji zwalniającej pamięć z języka C.
Nie korzystasz dobrze ze wskaźników, ponieważ nie musisz do każdej zmiennej przypisywać wskaźnika, oraz w twoim programie nie widzę sensu użycia wskaźników.
Jeśli chcesz ocenę kodu, masz pytania ?
Pisz w komentarzach pod tą odpowiedzią. Chętnie odpowiem ;)

Pokaż 6 poprzednich komentarzy

komentarz 5 września 2016 przez MichuDev Pasjonat (20,300 p.)

Wszystko co jest w pamięci ma jakiś adres, jak go znasz to możesz się tam odwołać (adres jest przypisywany w czasie kompilacji).

Jeśli będziesz bardziej zaawansowany zrozumiesz, że jeśli piszesz program (nie system), to ten adres nie będzie normalnym adresem (system stosuje stronnicowanie i tłumaczy twoje adresy, na adresy wewnątrz strony).

komentarz 5 września 2016 przez hubercik Użytkownik (560 p.)

Czy ten kod jest teraz poprawny? Czy dla zmiennej "wybierz" trzeba tworzyć nowy wskaźnik? Jeśli nie to skąd program wie, którą wartość wywołać? --> mam na myśli to , że do *e są przypisane 2 zmienne, a gdy przypisze jeszcze adres zmiennej "wybierz" to tworzy się coś takiego : *w= adres_x, adres_y, adres_wybierz -- > skąd wskaźnik wie, by wywołać wartość wybierz? Już sam nie wiem... Chyba źle sobie wyobrażam jak to wygląda...

Ps. Przypisałem 3 zmienne do *e i działało ... /: switch wyrzucał wartość poprawną

kod:



#include <iostream>
using namespace std;

int main()
{
    for(;;)
    {
    int x,y;
        int *e;
    cout<<"Podaj pierwszą liczbę: ";
    cin>>x;
    e=&x;
    cout<<"Podaj drugą liczbę: ";
    cin>>y;
    e=&y;
        
    cout<<"Main menu:"<<endl;
    cout<<"1.Dodawanie"<<endl;
    cout<<"2.Odejmowanie"<<endl;
    cout<<"3.Mnożenie"<<endl;
    cout<<"4.Dzielenie"<<endl;
    cout<<"5.Wyjście"<<endl;
    cout<<"wybierz: ";
    int wybierz, *w;
    cin>>wybierz;
    w=&wybierz;
    switch(*w)
    {
        case 1: cout<<"Suma="<<x+y<<endl;    break;
        case 2: cout<<"Różnica="<<x-y<<endl; break;
        case 3: cout<<"Iloczyn="<<x*y<<endl;  break;
        case 4: cout<<"Iloraz="<<x/y<<endl;   break;
            
            
        case 5: exit(0);
        
        default: { cout<<"Nie ma takiej opcji w menu, jestem zmuszony do zamknięcia programu!"<<endl; exit(0); } break;
            
    }
        
    
    }
   
    
    return 0;
}

komentarz 5 września 2016 przez MichuDev Pasjonat (20,300 p.)

To jest ten sam kod, ale mógłbyś zmienić bloczek z kodem na C++ (masz actionscript). Czytaj prywatne wiadomości.

komentarz 5 września 2016 przez Sinnley Stary wyjadacz (12,810 p.)

Nie można do jednego wskaźnika przypisać kilka adresów na raz. Nie możesz też usunąć wskaźnika operatorem delete, ani w żadny inny sposób, chyba, że jest to nijako wskaźnik na wskaźnik.

Stosując operator delete usuwasz pamięć, którą za pomocą tego wskaźnika zaalokowałeś, jeśli tego nie robisz, tak jak w twoim przypadku, to nie masz czego usuwać. W twoim przypadku przypisujesz tym wskaźnikom adres zmiennych, które utworzyłeś statycznie, nie dynamicznie.

int * wskaznik;
wskaznik = new int;
*wskaznik = 7;
cout << "Wartosc pod wskaznikiem: " << *wskaznik;
delete wskaznik;

Powyzszy kod tworzy wskaznik na zmienna int. Potem dynamicznie alokuje pamięć na taego inta, przypisuje mu wartosc 7, wypisuje ją, a następnie zwalnia tą pamięć. Wkaznik jest statyczny, tak jak np. zmienne x,y w twoim programie, pozostaje wiec do konca jego dzialania i dopiero wtedy jest usuwany.