#include <iostream>
#include<vector>
using namespace std;
class Punkt
{
int x, y;
public:
Punkt()
{
this->x = 0;
this->y = 0;
}
Punkt(int xx, int yy)
{
this->x = xx;
this->y = yy;
}
Punkt(const Punkt & wzorzec)
{
this->x = wzorzec.x;
this->y = wzorzec.y;
}
void przesun(int dx, int dy)
{
x += dx;
y += dy;
}
friend ostream & operator<<(ostream & ekran, Punkt & p);
};
ostream & operator<<(ostream & ekran, Punkt & p)
{
ekran << "(" <<p.x << ",";
ekran << p.y << ") " << endl;
return ekran;
}
class Linia
{
Punkt *p1, *p2;
public:
Linia() : p1(new Punkt()), p2(new Punkt(2,2))
{
}
Linia(Punkt *pierwszy, Punkt *drugi) : p1(pierwszy), p2(drugi)
{
}
Linia(const Linia & wzorzec)
{
*p1 = *(wzorzec.p1);
*p2 = *(wzorzec.p2);
}
void przesun(int dx, int dy)
{
p1->przesun(dx,dy);
p2->przesun(dx,dy);
}
void wypisz()
{
cout << "Linia:" << endl;
cout << "Punkt pierwszy: ";
(operator<<(cout,*p1));
cout << "Punkt drugi: ";
(operator<<(cout,*p2)) << endl;
}
~Linia()
{
delete p1;
delete p2;
}
};
int main()
{
Punkt *pierwszy = new Punkt(1,1);
cout << *pierwszy;
Punkt *drugi = new Punkt(2,2);
cout << *drugi;
Linia *pierwsza = new Linia(pierwszy,drugi);
Linia *druga = pierwsza;
pierwsza->przesun(2,2);
pierwsza->wypisz();
druga->wypisz();
return 0;
}
- Rejestracja:ponad 9 lat
- Ostatnio:ponad 9 lat
- Postów:27
0
C++ - Rejestracja:około 18 lat
- Ostatnio:12 miesięcy
2
Linia *druga = pierwsza;
Dlatego. Zadziałało tutaj kopiowanie, ale nie twojego obiektu, a wskaźnika. Używasz ich zdecydowanie zbyt dużo, ten program nie powinien zawierać ani jednego! Jest w nim ponadto kilka błędów - użycie niezainicjowanej pamięci, wycieki.
- Rejestracja:około 10 lat
- Ostatnio:3 minuty
- Lokalizacja:Łódź
- Postów:1402
1
Stworzyć drugi element, zamiast przypisywać adres? Na przykład:
Linia *druga = new Linia(*pierwsza);
oczywiście MUSISZ dodać zwalnianie pamięci i naucz sie tego juz teraz, żeby ktoś później po Tobie nie musiał sprzątać wycieków pamięci!
tadzik287
teraz program się zawiesza.
kaczus
Ponieważ używasz wskaźników, to musisz napisać odpowiedni konstruktor kopiujący. Sądziłem, że to jasne... Ogólnie zasada - jeśli nie umiesz operować na wskaźnikach, nie używaj ich zbyt często.
tadzik287
tak właśnie myślałem, że co z konstruktorem kopiującym jest nie tak, lecz nie wiem co w nim zmienic.
- Rejestracja:ponad 9 lat
- Ostatnio:ponad 9 lat
- Postów:27
0
Konstruktor kopiujący zmieniłem na inny, ale nadal dwie linie się przesuwają:
Linia(const Linia * wzorzec)
{
*p1 = *(wzorzec->p1);
*p2 = *(wzorzec->p2);
}
bo musisz miec kopie, ale nie adresów, tylko obiektów, a tu ogolnie masz krasz - bo przypisujesz w nieznane miejsce wartosci - ogólnie zarządzanie pamięcią jakie wymyśliłeś jest mało efektywne
- Rejestracja:około 10 lat
- Ostatnio:3 minuty
- Lokalizacja:Łódź
- Postów:1402
1
Przypadkowo nawet ma szanse zadziałać prawidłowo... Ale sam projekt jest podatny na to by się wywalić, przy ździebko innym kodzie.
Zobacz pozostałe 11 komentarzy
próbowałem pousuwać te wskaźniki i zrobić tak aby tylko w mainie tworzyć dynamiczne obiekty, ale się nie udało. Da się tak ?
Ogólnie to powinno robić się odwrotnie - na zewnątrz jak najmniej obsługi rzeczy dynamicznych, a jeśli trzeba to w środku - tak należy projektować, żeby później nie dziwić się, czemu sie wiesza, skąd wycieki pamięci itd. Jeszcze jak unieszczęśliwisz siebie tak skopanym kodem, pół biedy - sam będziesz się męczył, gorzej, jak ktos po tobie przejmie ten kod....
to ten kod jest aż tak bardzo zły ?
mógłbyś pokazać jak powinien ten kod wyglądać w klasie Punkt i Linia ?
Na początek pozbądź się wszelkich wskaźników w funkcjach udostępnianych na zewnątrz - jeśli zajdzie konieczność optymalizacji - zajmiesz się tym później.
- Rejestracja:ponad 9 lat
- Ostatnio:ponad 9 lat
- Postów:27
0
Bez wskaźników, tyle, że teraz nie mogę stworzyć obiektu dynamicznego linii, bo argumenty nie pasują do konstruktora.
#include <iostream>
#include<vector>
using namespace std;
class Punkt
{
int x, y;
public:
Punkt()
{
this->x = 0;
this->y = 0;
}
Punkt(int xx, int yy)
{
this->x = xx;
this->y = yy;
}
Punkt(const Punkt & wzorzec)
{
this->x = wzorzec.x;
this->y = wzorzec.y;
}
void przesun(int dx, int dy)
{
x += dx;
y += dy;
}
friend ostream & operator<<(ostream & ekran, Punkt & p);
};
ostream & operator<<(ostream & ekran, Punkt & p)
{
ekran << "(" <<p.x << ",";
ekran << p.y << ") " << endl;
return ekran;
}
class Linia
{
Punkt p1, p2;
public:
Linia() : p1(Punkt()), p2(Punkt(2,2))
{
}
Linia(Punkt & pierwszy, Punkt & drugi) : p1(pierwszy), p2(drugi)
{
}
Linia(const Linia & wzorzec)
{
this->p1 = Punkt(wzorzec.p1);
this->p2 = Punkt(wzorzec.p2);
}
void przesun(int dx, int dy)
{
p1.przesun(dx,dy);
p2.przesun(dx,dy);
}
void wypisz()
{
cout << "Punkt pierwszy: ";
(operator<<(cout,p1));
cout << "Punkt drugi: ";
(operator<<(cout,p2)) << endl;
}
};
int main()
{
Punkt *pierwszy = new Punkt(1,1);
Punkt *drugi = new Punkt(2,2);
Punkt *trzeci = new Punkt(3,3);
Punkt *czwarty = new Punkt(8,8);
Punkt *piaty = new Punkt(10,10);
Linia *pierwsza = new Linia(pierwszy,drugi);
Linia *druga = new Linia(drugi,trzeci);
Linia *trzecia = new Linia(trzeci,czwarty);
Linia *czwarta = new Linia(czwarty,piaty);
delete pierwszy,drugi;
return 0;
}
- Rejestracja:około 10 lat
- Ostatnio:3 minuty
- Lokalizacja:Łódź
- Postów:1402
0
Linia *pierwsza = new Linia(*pierwszy, *drugi);
Aczkolwiek dalej piszę, nie używaj wskaźników, jeśli nie umiesz....
Zobacz pozostały 1 komentarz
A musisz tworzyć obiekt dynamiczny? - jeśli tak, to trudno - ale wtedy trzeba nauczyć się najpierw wskaźników.
chyba aż takich tragicznych błędów z tymi wskaźnikami nie zrobiłem ?
Jak napisałem - w oczy rzucało się kilka, a sam miales problemy, których nie rozumiałeś...
Zrobiłeś tragiczne błędy. Nie zwolniłeś pamięci.
Gorszy był ten, w którym używał niezainicjowanej pamięci :).
- Rejestracja:ponad 13 lat
- Ostatnio:prawie 3 lata
1
Zamień:
Punkt *pierwszy = new Punkt(1,1);
Na:
Punkt pierwszy(1,1);
Lub:
Punkt pierwszy = {1,1};
Jeśli chcesz używać wskaźników:
- każdy "new" pakujesz do smart pointera lub kasujesz przez delete
- stosuj smart pointery (taka wersja pośrednia między alokacją dynamiczną a na stosie)
Przykład dla smart pointera:
auto pierwszy = make_shared<Punkt>(1,1);
No i od C++14 mamy również make_unique, czyli jeszcze jeden powód mniej, żeby używać gołego new.