Wskaźnik do dynamicznej tablicy paczek

Wskaźnik do dynamicznej tablicy paczek
PP
  • Rejestracja:około 3 lata
  • Ostatnio:prawie 2 lata
  • Postów:19
0

Proszę o jakieś ukierunkowanie gdzie jest błędne myślenie.
Krótki opis o co chodzi, i co się dzieje przy poszczególnych zmianach.
Tuż pod mainem znajduje sie 6 elementowa tablica typu Paczka.
Klasa kurier posiada jako składnik m.in wskaźnik na obiekt typu Paczka. W tym przykładzie wskaźnik wskazuje na pierwszy element dynamicznej tablicy Paczek. Drugim składnikiem Kuriera jest int ilosc_ jako rozmiar tablicy.
Gdy zakomentuje linie 57-60 to wyswietla sie cout << --------12---------;
Natomiast gdy nie będzie zakomentowany to już się nie wyświetla '12' .
Sprawdzałem w debuggerze oraz operatorem&, adresy wskaźnika 'paczki_' w klasie kurier nie dzielą między sobą wspólnych adresów.

Szybki spis treści:
Linia 53: konstruktor kopiujacy
85: operator przypisania kopiującego
180-196: Wywołania które mają potencjalny wpływ na te problemy.

Będę wdzięczny za każdą pomoc :)

Ps. "Czemu nie używasz czegoś zamiast tego ?" - Bo tak nas na ten moment uczą i chciałbym to opanować, na wszystko będzie czas.

Kopiuj
#include <iostream>
using namespace std;
//Pojazd(const Pojazd& p)
//operator=(const pojazd& p)
class Paczka
{
private:
	string adres_;
	float wartosc_;
public:
	Paczka(){};
	~Paczka(){};
	Paczka (string adres_, float wartosc_)
	{
		this->adres_ = adres_;
		this->wartosc_ = wartosc_;
	}

	string &adres(){return adres_;}
	float &wartosc(){return wartosc_;}
	friend ostream& operator<<(ostream& os, Paczka& p);
};
ostream& operator<<(ostream& os, Paczka& p)
	{
		os << "adres dostawy: " << p.adres_ << '\t' << "wartosc paczki: " << p.wartosc_ << endl;
		return os;
	}
class Osoba
{
private:
	string imie_;
	string nazwisko_;
public:
	Osoba(){this-> imie_ = "Brak"; this-> nazwisko_ = "Brak";};
	~Osoba(){};
	Osoba(string imie_, string nazwisko_)
	{
		this->imie_ = imie_;
		this->nazwisko_ = nazwisko_;
	}
	friend ostream &operator<<(ostream& os, Osoba& o);
	friend bool operator==(const Osoba& a, const Osoba& b);
	string &imie(){return imie_;}
	string &nazwisko(){return nazwisko_;}
};
class Kurier : public Osoba
{
private:
    Paczka* paczki_;
    int ilosc_;
public:
    Kurier(){paczki_ = 0; ilosc_ = 0;}
    Kurier(Kurier& k) //KONSTURKTOR KOPIUJACY
    {
        this->paczki_ = new Paczka [k.ilosc_];
        this->ilosc_ = k.ilosc_;
        for(int i = 0; i < ilosc_; i++)
        {
            paczki_ = k.paczki_;
        }
    }
    ~Kurier(){if(paczki_!=0){delete [] paczki_;}}
    Kurier(Paczka* poczatek, Paczka* koniec)
    {
        ilosc_ = koniec - poczatek;
        paczki_ = new Paczka [ilosc_];
        for (int i = 0; i < ilosc_; ++i)
        {
            paczki_[i] = poczatek[i];
        }
    }
    float wartosc() // cout << k1.wartosc() << endl;
    {
            float tempWartosc = 0;
    if(paczki_ != 0)
        {
            for(int i = 0; i < ilosc_; ++i)
            {
                tempWartosc += paczki_[i].wartosc();
            }
            return tempWartosc;
        }
        else return 0;
    }
Kurier &operator=(Kurier& k) //OPERATOR PRZYPISANIA KOPIUJACEGO
{
    if(this != &k){delete [] paczki_;
    ilosc_ = k.ilosc_;
    paczki_ = new Paczka[ilosc_];
    for (int i = 0; i < ilosc_; ++i)
    {
        paczki_[i] = k.paczki_[i];
        cout << &paczki_[i] << " " << &k.paczki_[i] << " | ";
    }
    }
    return *this;
}//k1 = k1 + p[4];
Kurier &operator+(Paczka& p) //DODAWANIE PACZEK
{
    Paczka* temp = new Paczka[ilosc_+ 1];
    for (int i = 0; i < ilosc_; ++i)
    {
        temp[i] = paczki_[i];
    }
    temp[ilosc_] = p;
    delete [] paczki_;
    paczki_ = new Paczka[ilosc_+ 1];
    for (int i = 0; i < ilosc_+ 1; ++i)
    {
        paczki_[i] = temp[i];
    }
    delete [] temp;
    ilosc_++;
    return *this;
}
Paczka &operator[](int &num) //ZWRACA INFORMACJE O PACZCE SPOD PODANEGO INDEKSU
{
    if(num < 0 || num > ilosc_)
    {
        throw -10;
    }

    else{return paczki_[num];}
}
};
bool operator==(const Osoba& a, const Osoba& b)
    {return a.imie_ == b.imie_ && a.nazwisko_ == b.nazwisko_;}
ostream &operator<<(ostream& os, Osoba& o)
{
	os << "imie: " << o.imie_ << '\t' << "nazwisko: " << o.nazwisko_ << endl;
	return os;
}

int main()
{
  Paczka p[] = {
    Paczka("Dabrowskiego 33", 123.00),
    Paczka("NMP 12", 40.00),
    Paczka("Armii Krakowej 4", 30.00),
    Paczka("Andersa 6", 51.00),
    Paczka("Kukuczki 13", 12.00),
    Paczka("Skrzyneckiego 5", 40.00),
  };

  Osoba o1("Jan", "Kowalski");
  cout << "---- 1 ----" << endl;
  cout << o1 << endl;

  {
      Osoba o2(o1);
      cout << "---- 2 ----" << endl;
      cout << o2 << endl;

      cout << "---- 3 ----" << endl;
      cout << boolalpha << (o1 == o2) << endl;

      Osoba o3;
      cout << "---- 4 ----" << endl;
      cout << o3 << endl;

      o3 = o2;
      cout << "---- 5 ----" << endl;
      cout << o3 << endl;

  }

  cout << "---- 6 ----" << endl;
  cout << o1 << endl;

  Kurier k1(p, p+3);//wskazniki (p+3 to pierwsza paczka z listy ktora nie ma byc 'kopiowana')
  cout << "---- 7 ----" << endl;
  cout << k1;
  cout << k1.wartosc() << endl;

 Kurier k2;
  cout << "---- 8 ----" << endl;
  cout << k2.wartosc() << endl;

  {
    Kurier k3(k1);
    cout << "---- 9 ----" << endl;
    cout << k3 << endl;

    k2 = k3;
    cout << "---- 10 ----" << endl;
    cout << k2.wartosc() << endl;

    k1 = k1 + p[4];
    k1 = k1 + p[5];

    cout << "---- 11 ----" << endl;
    cout << k1.wartosc() << endl;
  }

  cout << "---- 12 ----" << endl;
  cout << k2.wartosc() << endl;
 /* try{
    Kurier k4(p, p+3);
    cout << "---- 13 ----" << endl;
    for(int i=0; i<10; ++i){
      cout << k4[i] << ",";
    }
    cout << endl;
  }
  catch(...){
    cout << "---- 14 ----" << endl;
    cout << "Wyjatek" << endl;

  }*/
  return 0;
}

edytowany 2x, ostatnio: PaawllooPL
enedil
nadal nie rozumiem, w czym leży twój problem?
enedil
póki co opisałeś jakieś (być może dziwne) symptomy, ale ciężko stwierdzić co w tym jest nie tak
PP
Ciężko mi samemu na to odpowiedzieć, bo według mnie powinien działać ten program. Mówiąc szczerze to oczekuję analizy w zakresie przydzielania pamięci przy np. konstruktorze kopiującym, operatorze przypisania. Nie jestem w stanie określić konkretnej przyczyny problemu, bo jej nie widzę ;/ Innego wyjścia chyba nie ma.
enedil
nie chodzi mi o przyczynę, a o symptomy. Porównanie tego co oczekujesz, że się zdarzy, vs tego co się dzieje.
PP
użyłem operatora przypisania kopiującego (z k3 do k2 w linii 184), tuż pod tą instrukcją wyświetliłem wartość k2 i była taka sama jak k3. Oczekuję, że przy ponownym wywołaniu funkcji "wartosc" poza tymi nawiasami klamrowymi również dostanę tą wartość. A Jednak program już nie wyświetla nic po wyjściu z tej dodatkowej pary klamr (linie 179 i 193).
RE
  • Rejestracja:ponad 18 lat
  • Ostatnio:około 3 godziny
2

Ciężko mi samemu na to odpowiedzieć, bo według mnie powinien działać ten program. Mówiąc szczerze to oczekuję analizy w zakresie przydzielania pamięci przy np. konstruktorze kopiującym, operatorze przypisania. Nie jestem w stanie określić konkretnej przyczyny problemu, bo jej nie widzę ;/ Innego wyjścia chyba nie ma.

a ja oczekuję że użyjesz:

  1. GDB
  2. Valgrind(i jego "podnarzędzia np. mem check")
  3. adress sanitizer

Do wszystkich 3 na yt instrukcje jak używać. Niestety nie mam czasu na analizę a i sam nauczysz się więcej. Jak będzie jakiś nians to wróć to się wtedy looknie.


We are the 4p. Existence, as you know it, is over. We will add your biological and technological distinctiveness to our own. Resistance is futile
enedil
  • Rejestracja:ponad 11 lat
  • Ostatnio:dzień
  • Postów:1027
4

Tu masz ten sam błąd co wyżej (zminimalizowałem przykład)

Kopiuj
#include <iostream>
using namespace std;

struct Paczka {
    string adres_ = "";
    float wartosc_ = 0;
};

class Kurier {
    Paczka* paczki_ = nullptr;
    size_t ilosc_ = 0;
public:
    Kurier(const Kurier& k) {
        paczki_ = new Paczka [k.ilosc_];
        ilosc_ = k.ilosc_;
        for (size_t i = 0; i < ilosc_; i++) {
            paczki_ = k.paczki_;
        }
    }
    ~Kurier() {
        if (paczki_ != nullptr) {
            delete [] paczki_;
        }
    }
    Kurier(Paczka* poczatek, Paczka* koniec) {
        ilosc_ = static_cast<size_t>(koniec - poczatek);
        paczki_ = new Paczka [ilosc_];
        for (size_t i = 0; i < ilosc_; ++i) {
            paczki_[i] = poczatek[i];
        }
    }
};

int main() {
    Paczka p[] = {
        Paczka{"Dabrowskiego 33", 123.00},
        Paczka{"NMP 12", 40.00},
        Paczka{"Armii Krakowej 4", 30.00},
    };

    Kurier k1(p, p+3);
    Kurier k2(k1);
    return 0;
}
RE
  • Rejestracja:ponad 18 lat
  • Ostatnio:około 3 godziny
5

w ogóle odpaliłem ten program, i chyba nie czytasz co tam ci wychodzi na wyjściu i to chyba dużo podpowie. Jak to puścisz w GDB to bajka powinna być

Kopiuj
Program returned: 139
free(): double free detected in tcache 2

do tego weź sobie odpal statyczną analizę kodu jak cppcheck czy clang-tidy.
edit:
4p_debug.png


We are the 4p. Existence, as you know it, is over. We will add your biological and technological distinctiveness to our own. Resistance is futile
edytowany 2x, ostatnio: revcorey
Zobacz pozostałe 3 komentarze
PP
Robiłem już debuggerem w Codeblocks, pierwszy raz w życiu używałem. Właśnie instaluje VM z linuxem i spróbuję się zagłębić w valgrinda.
RE
msvc ma chyba swoje jakieś własne toole. A tu narzędzie online https://www.onlinegdb.com/online_c++_compiler i wedit co pokazuje.
RE
kontynuuj dalej :)
PP
Czyli z tego wynika, że jednak są jakieś wspólne adresy ?
RE
spójrz na przykła @enedil ale najlepiej zrób tak odpal debugger na swoim, ustaw pułapkę od lini 136 i idź krok po kroku. zobaczysz co jak się przypisuje i jakie ma adresy. valgrindteż to zauważy czy tam narzędzie z msvc do analizy pamieci.
IN
  • Rejestracja:ponad 2 lata
  • Ostatnio:około 2 lata
  • Postów:60
0

Dałeś 25 cout w main a prawie żadnego w funkcjach. Program wg mnie, powinien na początkowym etapie (kiedy jeszcze się uczysz) generować logi po każdym działaniu.

Kopiuj
Kurier &operator+(Paczka& p) //DODAWANIE PACZEK
{
    Paczka* temp = new Paczka[ilosc_+ 1];
    for (int i = 0; i < ilosc_; ++i)
    {
        temp[i] = paczki_[i];
    }
    temp[ilosc_] = p;
    delete [] paczki_;
    paczki_ = new Paczka[ilosc_+ 1];
    for (int i = 0; i < ilosc_+ 1; ++i)
    {
        paczki_[i] = temp[i];
    }
    delete [] temp;
    ilosc_++;
    return *this;
}```

Tutaj nie masz ani jednego.

PP
Funkcja main to kod testujący, którego nie mogę zmienić. Co do generowania logów pewnie masz rację. Na ten moment jak wrócę odpalę valgrinda i będę naprawiać:)
IN
Pewnie mam rację. Wolisz tak jak piszą chłopaki, gapić się w debugera? btw Bardzo sympatyczny projekt. Podoba mi się. Aż mnie korci, żeby to próbować skompilować. Mógłbyś dodać jakiś timer, i żeby to działało w pętli, i losowo dodawało paczki, po jakimś losowym czasie usuwało, wyświetlało ile jest ich w danej chwili na stanie itp. Żeby coś się działo.
IN
Teraz nie mam nic pod ręką akurat ale w wolnym czasie przygotuję przykład kodu jak powinien wyglądać w trakcie uczenia, bo to może się wielu ludziom przydać i znacznie uprościć życie.
PP
Chetnie bym zobaczyl taki mini poradnik :)
RE
" Wolisz tak jak piszą chłopaki, gapić się w debugera?" - ktoś by mógł powiedzieć że to jedna z podstawowych umiejętności programisty... Do tego nauka valgrind i sanitaziera. Teraz będzie wypisywał adresy, co może sprawdzić debuggerem itd. no cóż niektóre rzeczy można robić dłużej.
IN
Właśnie coś mi się przypomniało, zobacz, że niektóre linuxy mają możliwość pokazywania logu uruchomieniowego o różnych poziomach szczegółowości, i tam wybierasz sobie jak dużo informacji zobaczysz. Coś takiego mam na myśli.
RE
dalej co zmienia dodanie loggera(pomijam kwestię linuksowego syslog.h bo masz inne multiplatformowe loggery jak np. spdlog), kiedy szybciej i łątwiej zrobić to gdb, sanitaizerem, valgrind?
IN
@revcorey: preferujesz cudze rozwiązania od własnych narzędzi. Twój wybór. Ja akurat lubię własne rozwiązania, ciągle wpadam na fajne pomysły jak je można ulepszyć.
RE
linuxowe logger to nie moje narzędzie, podobnie spdlog. Zakłądamże system operacyjny + libki napisałeś sam? podążając twoją logiką.
enedil
@recovery: ja się nie mogę zgodzić - im bardziej skomplikowane problemy, tym mniej człek ma ochotę używać gdb (co innego sanitizery).
IN
@revcorey: napisałem tylko przykład z linuxa który można zaimplementować we własnym programie. Ten wydał mi się najbardziej dający do myślenia. Zacznę tłumaczyć po swojemu, przyczepisz się, podam przykład, też się przyczepisz. Jakiej metody bym nie użył jest wg Ciebie niewłaściwa. Masz problem w tym, że dzielę się, wg mnie interesującymi pomysłami?
RE
im więcej wyrzyganych logów tym trudniej się połapać i jeszcze człowiek na platformie wbudowanej tęskni za gdb którego tam nie odpali, no i po raz kolejny logger to nie twoje narzędzie jak możesz go używać? Co więcej w tak banalnym programie gdb czy inny jest idealny.
enedil
@revcorey: gdb wkurza bo nie ma opcji prostego wrócenia do wcześniejszego stanu. I trzeba się stepować i na żywo analizować, zamiast móc ogarnąć całościowo co się psuje.
RE
@enedil: wszystko zależy. Kiedyś debuggowałem WPE i co mi po logach(jak to nie mój soft a dojść tam do czegoś...) dopiero jakaś próba odpalenia gdb trochę rozjaśniła. W każdym razie wszystko zależy od błędu. Ale już rzygam logami czasami.
enedil
Może w tak maluśkich progarmach jeszcze da się używać gdb do normalnego analizowania.
enedil
gdb jeszcze ma ten problem, że zbyt często już doświadczałem p this - optimized out
enedil
ogólnie uważam, że ludzie co się bronią przed debuggerem bo go nie znają są kiepskimi programistami, ale też ludzie co się bronią przed logami, bo znają debugger, to też są kiepscy programiści
RE
@enedil: ja go i tak zazwyczaj nie dam radę odpalić, za mało pamięci albo appka dostawała takiej czkawki że stałą się nie używalna. Ale tu podałem gdb bo nadaje się idealnie i uważam że programista powinien znać podstawy debugowania.
IN
Widać jak te Twoje narzędzia które proponujesz "gdb, sanitaizerem, valgrind" działają. Właśnie te napisane własnoręcznie mogą być bardzo precyzyjne, kwestia pomyślunku. No ale, to doszedłeś do wniosku który dla mnie był oczywisty - co własne narzędzie to własne. Trzeba tylko wiedzieć dokładnie co się chce. Ty krytykujesz cudze rozwiązania i brzydzisz się tworzeniem własnych. Dokąd zmierzasz?
enedil
@recovery: w jakim sensie, nie mogłeś odpalić na gdb, bo za mało pamięci? Gdb nie ma praktycznie narzutu pamięciowego (chyba, że mówimy o mikrokontrolerac)?
enedil
Ja osobiście raczej częściej pracuję nad programami co mają dostępny 1TB ramu niż 500kB, to może zmieniać perspektywę.
RE
@enedil: gdb jakieś tam wymaganie pamięciowe ma, ale jak jest duży memory leak to mogłem mieć crashe ze wzlgędu na ogólna zmienność w całym środowisku tzn. Masz gdb, jest leak pamięć nie zwolniona a inne procesy pracują niezależnie i nagle pyk brak pamięci przy debugowaniu bo jakiś proces zechciał pamięci a jak miałem gdb i namierzałem. Taki sprzęt embeeded co mu już brakuje mu po latach rozwoju(mips). Ale i tak najgorsze były czkwaki. Więc i tak wracało się do dodawania dupa logów, co do sanitaizerów to nawet pamiętam że tam toolchain z broadcom go nie zapewniał :)
enedil
ah, no to współczuję koledze w embedded
enedil
no to tak jak mówiłem, inna specyfika pracy
RE
najgorsze to są te toolchainy bo ich nie ruszysz masz gcc 4.8 i jazda. Całe szczęście już pracuję na projekcie z gcc 9 xD
enedil
@recovery: widzisz, a my na clangu 15 jedziemy.
_13th_Dragon
@infinityhost, Twierdzisz że dasz rady napisać coś lepszego niż zespół mistrzów, którzy zęby zjedli w tym temacie?
enedil
@infinityhost: pokaż mi proszę, jak zastąpisz address sanitizer "własnym rozwiązaniem", na potrzeby mini programiku
IN
@enedil: nic Ci nie będę pokazywał. Nie za to mi płacą.
IN
@enedil: masz swojego address sanitizer i się tego trzymaj. Jak nie masz wyobraźni do czegoś więcej to nic nie zmieniaj. Ja zmiany akurat lubię.
IN
Nikt komu pisanie sprawia radość i ma z tego frajdę, nie pyta "po co". Wy to robicie cały czas. Taka różnica między nami.
IN
@_13th_Dragon: dokładnie tak, kiedy starałem się pomóc temu chłopakowi poprzez podanie mu pomysłu na zebranie informacji z tego jak działa jego program, żeby mógł się upewnić czy krok po kroku się dzieje to co powinno, to zaczęliście wyśmiewać ten pomysł. To bliżej wam do mędrców czy debili wg tej Twojej teorii? xD
_13th_Dragon
Ja nie pisałem address sanitizera, ale to ty się z nimi porównujesz, kiedy nawet nie jesteś w stanie zawalczyć o własną koncepcje! Kiedy ci nawet pozwolono ustalać zasady gry!
enedil
@infinityhost: logi mają sens do debugowania logiki, a nie błędów zarządzania pamięcią
IN
@enedil: to czy wyświetlenie zawartości pamięci nie pokaże czy wszystko jest ok czy nie? (jeśli się nie uda też wskaże dokładnie źródło problemu - metode jej alokacji). Czego więcej potrzebujesz?
enedil
Wyświetlanie zawartości pamięci nie musi pokazać wszystkiego, bo zazwyczaj błędy zarządzania pamięcią wiążą się z UB, a to znaczy że twój kod który loguje wskaźniki może równie dobrze wyświetlać nie to co powinien, bo optymalizacja kompilatora coś namieszała
enedil
Już nie mówiąc o tym, że dużo ciężej się połapać jak masz np. kilkaset alokacji, w międzyczasie jakieś zwolnienia pamięci, jak jeszcze włączysz w działanie konstruktory przenoszące a nie kopiujące to jeszcze mniej się staje jasne co odpowiada za dany kawałek pamięci.
IN
@enedil: a to ciekawe. Ale chyba jak testujesz program to dokładnie wiesz co powinno być wyświetlone. Jak masz kilkaset alokacji to masz okazję tworzyć dopasowane metody kontrolujące to. "Mniej się staje jasne co odpowiada za dany kawałek pamięci"?. Jak stracę kontrolę nad tym na czym operuję to odinstaluję kompilatory.
enedil
"jak testujesz, wiesz co dokładnie powinno być wyświetlone" - chyba że masz np. testy losowe (nie wierzę w poprawność żadnego softu który nie był losowo testowany)
enedil
A co do odinstalowania kompilatora, to polecam to zrobić czym prędzej, bo z takim podejściem nie wyobrażam sobie programować niczego jakkolwiek bardziej skomplikowanego.
IN
@enedil: w matematyce masz ekstrema. To chyba nad tym da się zapanować. Co Ci tam losowego może wyskoczyć? Nazwisko człowieka będzie się składało z nie-liter? Może mi trudno Ciebie zrozumieć, bo jeszcze nigdy nie straciłem kontroli nad tym co piszę.
_13th_Dragon
@infinityhost, tak wiemy o extremach a ty jesteś jedną z nich czyli taką która sporo poniżej zera absolutnego
PP
  • Rejestracja:około 3 lata
  • Ostatnio:prawie 2 lata
  • Postów:19
0

Okej, valgrind mi za bardzo nie pomógł, bo po prostu nie umiem z niego jeszcze korzystać. Debugger nakierował mnie na linie 53 gdzie jest konstruktor kopiujący. Aż głupio się przyznać ale w linii 59 brakuje indeksów ;p więc cały czas była przepisywana pierwsza paczka z jednego kuriera do pierwszej paczki drugiego kuriera. Chociaż nie potrafię sobie zobrazować dlaczego spowodowało to wyciek.

_13th_Dragon
  • Rejestracja:ponad 19 lat
  • Ostatnio:3 miesiące
0

operator=?


Wykonuję programy na zamówienie, pisać na Priv.
Asm/C/C++/Pascal/Delphi/Java/C#/PHP/JS oraz inne języki.
Zobacz pozostałe 2 komentarze
_13th_Dragon
No widzisz, jesteś mądrzejszy niż twoja własna ocena tego :)
PP
Nie rozumiem czy to był atak w moją stronę? Spytałem normalnie, bo nie wiem czy zadałeś zwykłe pytanie do czego służy, czy też zasugerowałeś żeby przy nim popracować ;p
_13th_Dragon
Nie atak, raczej gratulacje, z delikatną nutka: - "a sam wcześniej nie mogłeś się domyślić?" :P
PP
Przepraszam, jest już późno i nie zrozumiałem :D Odkryłem również powód dla którego zakomentowałem blok try-catch. Mianowicie operator[] sprawdzał przez zwróceniem elementu tablicy czy wartość indeksu jest większa od ilosc_. A ilosc jest przeciez o 1 większa, bo indeksowanie idzie od 0. Masakra jakie dwie proste rzeczy niszczyły mi program :)
_13th_Dragon
Odrobinę pokory, prawdziwe mistrzowie wpadali w tak głupie pułapki - jestem temu świadkiem. Więc nie musisz tym się za bardzo przyjmować. Doświadczenie wszystko naprawi - gwarantuję!
PP
  • Rejestracja:około 3 lata
  • Ostatnio:prawie 2 lata
  • Postów:19
0

Jakby ktoś mógł mi powiedzieć czemu brak indeksu w 59 linii spowodował wyciek pamięci, to byłbym wdzięczny :) W moim rozumowaniu, zostaje przydzielona pamięć na 'n' elementów, ale używany jest tylko pierwszy. Co nie zmienia faktu, że dostęp to całego obszaru pamięci nadal jest dostępny poprzez wskaźnik typu paczka, więc przypuszczam, że powinien zostać usunięty wraz z destruktorem.

edytowany 1x, ostatnio: PaawllooPL
_13th_Dragon
Może by tak wkleiłeś kod po poprawkach?
PP
Za chwile, blokada czasowa na wstawianie kolejnych odpowiedzi.
enedil
  • Rejestracja:ponad 11 lat
  • Ostatnio:dzień
  • Postów:1027
0
Kopiuj
    Kurier(const Kurier& k) {
        paczki_ = new Paczka [k.ilosc_];
        ilosc_ = k.ilosc_;
        for (size_t i = 0; i < ilosc_; i++) {
            // tutaj nadpisujesz przed chwilą zaalokowany wskaźnik, przez co nigdy go nie zwolnisz
            paczki_ = k.paczki_;
        }
    }

ale większy zarzut to oczywiście, że potem zarówno k jak i podmiot danego konstruktora będzie zwalniać tę samą pamięć.

edytowany 1x, ostatnio: enedil
Zobacz pozostały 1 komentarz
enedil
no ale ja właśnie komentuję jak to się zdarzyło, że miałeś wyciek w wersji błędnej (gdzie zapomniałeś dać indeksy)
PP
Czy podczas zastosowania "cout" kompilator automatycznie dodaje tak jakby indeks[0], albo po prostu wyłuskuje zawartość z tablicy?
enedil
Nie. A czemu pytasz?
PP
Jednak nie ważne, byłem pewny, że wyświetlała mi się zawartość tablicy przy wpisaniu cout << paczki_; Dlatego spytałem, ale jednak adres wyświetla normalnie.
PP
Za dużo na dzisiaj chyba, dzięki za pomoc :)
enedil
  • Rejestracja:ponad 11 lat
  • Ostatnio:dzień
  • Postów:1027
0

Dodałbym jeszcze, że masz jeszcze jednen problem:

Kopiuj
Kurier k1(p, p+3);
cout << k1 << "\n";
k1 + p;
cout << k1 << "\n";

Wyjaśnij mi proszę, dlaczego uważasz za rozsądne, że te dwie wartości będą różne.

Już nie wspominając o tym że ta implementacja dodawania jest całkiem niewydajna - zawsze używa czasu proporcjonalnego do liczby dotychczasowych paczek.

edit:
chodziło mi o taki przykład:

Kopiuj
  Kurier k1(p, p+3);
  cout << k1.wartosc() << "\n";
  k1 + p[0];
  cout << k1.wartosc() << "\n";
edytowany 1x, ostatnio: enedil
Zobacz pozostałe 11 komentarzy
PP
Nie no, jak dla mnie wszystko tutaj gra. W linii 98 zaczyna się definicja przeciążonego operatora dodawania jak by coś.
enedil
Ja po prostu stwierdzam, że imo k+p[0] łamie oczekiwania programisty który czyta Twój kod - ja bym oczekiwał że taki plus wyprodukuje jakieś wyrażenie, bez zmieniania wartości żadnej ze stron.
PP
Aaaaa, no to tak. To jest tylko kod testujący, dostałem taki na laboratoriach, więc pod niego muszę się dostosować.
enedil
Ok. No to współczuję.
PP
Może nie tyle co muszę, bo powiedział gość, że można sobie zmienić fragmenty, które są nie istotne dla całej idei programu. Ale o tym po prostu nie pomyślałem, żeby "upięknić" program.
Kliknij, aby dodać treść...

Pomoc 1.18.8

Typografia

Edytor obsługuje składnie Markdown, w której pojedynczy akcent *kursywa* oraz _kursywa_ to pochylenie. Z kolei podwójny akcent **pogrubienie** oraz __pogrubienie__ to pogrubienie. Dodanie znaczników ~~strike~~ to przekreślenie.

Możesz dodać formatowanie komendami , , oraz .

Ponieważ dekoracja podkreślenia jest przeznaczona na linki, markdown nie zawiera specjalnej składni dla podkreślenia. Dlatego by dodać podkreślenie, użyj <u>underline</u>.

Komendy formatujące reagują na skróty klawiszowe: Ctrl+B, Ctrl+I, Ctrl+U oraz Ctrl+S.

Linki

By dodać link w edytorze użyj komendy lub użyj składni [title](link). URL umieszczony w linku lub nawet URL umieszczony bezpośrednio w tekście będzie aktywny i klikalny.

Jeżeli chcesz, możesz samodzielnie dodać link: <a href="link">title</a>.

Wewnętrzne odnośniki

Możesz umieścić odnośnik do wewnętrznej podstrony, używając następującej składni: [[Delphi/Kompendium]] lub [[Delphi/Kompendium|kliknij, aby przejść do kompendium]]. Odnośniki mogą prowadzić do Forum 4programmers.net lub np. do Kompendium.

Wspomnienia użytkowników

By wspomnieć użytkownika forum, wpisz w formularzu znak @. Zobaczysz okienko samouzupełniające nazwy użytkowników. Samouzupełnienie dobierze odpowiedni format wspomnienia, zależnie od tego czy w nazwie użytkownika znajduje się spacja.

Znaczniki HTML

Dozwolone jest używanie niektórych znaczników HTML: <a>, <b>, <i>, <kbd>, <del>, <strong>, <dfn>, <pre>, <blockquote>, <hr/>, <sub>, <sup> oraz <img/>.

Skróty klawiszowe

Dodaj kombinację klawiszy komendą notacji klawiszy lub skrótem klawiszowym Alt+K.

Reprezentuj kombinacje klawiszowe używając taga <kbd>. Oddziel od siebie klawisze znakiem plus, np <kbd>Alt+Tab</kbd>.

Indeks górny oraz dolny

Przykład: wpisując H<sub>2</sub>O i m<sup>2</sup> otrzymasz: H2O i m2.

Składnia Tex

By precyzyjnie wyrazić działanie matematyczne, użyj składni Tex.

<tex>arcctg(x) = argtan(\frac{1}{x}) = arcsin(\frac{1}{\sqrt{1+x^2}})</tex>

Kod źródłowy

Krótkie fragmenty kodu

Wszelkie jednolinijkowe instrukcje języka programowania powinny być zawarte pomiędzy obróconymi apostrofami: `kod instrukcji` lub ``console.log(`string`);``.

Kod wielolinijkowy

Dodaj fragment kodu komendą . Fragmenty kodu zajmujące całą lub więcej linijek powinny być umieszczone w wielolinijkowym fragmencie kodu. Znaczniki ``` lub ~~~ umożliwiają kolorowanie różnych języków programowania. Możemy nadać nazwę języka programowania używając auto-uzupełnienia, kod został pokolorowany używając konkretnych ustawień kolorowania składni:

```javascript
document.write('Hello World');
```

Możesz zaznaczyć również już wklejony kod w edytorze, i użyć komendy  by zamienić go w kod. Użyj kombinacji Ctrl+`, by dodać fragment kodu bez oznaczników języka.

Tabelki

Dodaj przykładową tabelkę używając komendy . Przykładowa tabelka składa się z dwóch kolumn, nagłówka i jednego wiersza.

Wygeneruj tabelkę na podstawie szablonu. Oddziel komórki separatorem ; lub |, a następnie zaznacz szablonu.

nazwisko;dziedzina;odkrycie
Pitagoras;mathematics;Pythagorean Theorem
Albert Einstein;physics;General Relativity
Marie Curie, Pierre Curie;chemistry;Radium, Polonium

Użyj komendy by zamienić zaznaczony szablon na tabelkę Markdown.

Lista uporządkowana i nieuporządkowana

Możliwe jest tworzenie listy numerowanych oraz wypunktowanych. Wystarczy, że pierwszym znakiem linii będzie * lub - dla listy nieuporządkowanej oraz 1. dla listy uporządkowanej.

Użyj komendy by dodać listę uporządkowaną.

1. Lista numerowana
2. Lista numerowana

Użyj komendy by dodać listę nieuporządkowaną.

* Lista wypunktowana
* Lista wypunktowana
** Lista wypunktowana (drugi poziom)

Składnia Markdown

Edytor obsługuje składnię Markdown, która składa się ze znaków specjalnych. Dostępne komendy, jak formatowanie , dodanie tabelki lub fragmentu kodu są w pewnym sensie świadome otaczającej jej składni, i postarają się unikać uszkodzenia jej.

Dla przykładu, używając tylko dostępnych komend, nie możemy dodać formatowania pogrubienia do kodu wielolinijkowego, albo dodać listy do tabelki - mogłoby to doprowadzić do uszkodzenia składni.

W pewnych odosobnionych przypadkach brak nowej linii przed elementami markdown również mógłby uszkodzić składnie, dlatego edytor dodaje brakujące nowe linie. Dla przykładu, dodanie formatowania pochylenia zaraz po tabelce, mogłoby zostać błędne zinterpretowane, więc edytor doda oddzielającą nową linię pomiędzy tabelką, a pochyleniem.

Skróty klawiszowe

Skróty formatujące, kiedy w edytorze znajduje się pojedynczy kursor, wstawiają sformatowany tekst przykładowy. Jeśli w edytorze znajduje się zaznaczenie (słowo, linijka, paragraf), wtedy zaznaczenie zostaje sformatowane.

  • Ctrl+B - dodaj pogrubienie lub pogrub zaznaczenie
  • Ctrl+I - dodaj pochylenie lub pochyl zaznaczenie
  • Ctrl+U - dodaj podkreślenie lub podkreśl zaznaczenie
  • Ctrl+S - dodaj przekreślenie lub przekreśl zaznaczenie

Notacja Klawiszy

  • Alt+K - dodaj notację klawiszy

Fragment kodu bez oznacznika

  • Alt+C - dodaj pusty fragment kodu

Skróty operujące na kodzie i linijkach:

  • Alt+L - zaznaczenie całej linii
  • Alt+, Alt+ - przeniesienie linijki w której znajduje się kursor w górę/dół.
  • Tab/⌘+] - dodaj wcięcie (wcięcie w prawo)
  • Shit+Tab/⌘+[ - usunięcie wcięcia (wycięcie w lewo)

Dodawanie postów:

  • Ctrl+Enter - dodaj post
  • ⌘+Enter - dodaj post (MacOS)