[VanQator]

#programowanie #cpp
Czy dobrze rozumiem, że nie mogę zwracać wskaźnika na tablicę, która została utworzona wewnątrz funkcji, bo po zakończeniu działania tej funkcji wskaźnik będzie wskazywał jakieś śmieci?

[MamCieNaHita]

@VanQator: jeżeli jest to tablica na stosie, to nie możesz. jeżeli to dynamicznie zaalokowana tablica, to możesz, ale musisz pamiętać, żeby ją zwolnić.
natomiast możesz zwrócić strukturę, która trzyma tablicę :P

[Hauleth]

@VanQator: nie, nie możesz zwrócić takiego wskaźnika, bo to UB. Często taki kod może nawet działać przez jakiś czas.

[s.....e]

@VanQator: móc możesz ale będzie to co piszesz.

[VanQator]

@MamCieNaHita: zrobiłem dynamicznie i też zwraca śmieci

[e.....n]

@VanQator: poka kod

[VanQator]

@eternitowy-Bogdan: zara zgithabuje

[VanQator]

@eternitowy-Bogdan: tutaj definicja funkcji: https://github.com/VanQator/algorithms/blob/master/main.cpp
wywołanie w main (na samym dole): https://github.com/VanQator/algorithms/blob/master/main.cpp
w pierwszej pętli działa fajnie, bo zwracam aktualny wynik z funkcji
w drugiej działa niefajnie

[luigi_pl]

@VanQator: No nie jest zaskakujące że masz śmieci na rezultacie quadraticEquation
dla wywołania quadraticEquation(1, 1, 1, 1)
delta jest mniejsza od zera, więc wpada w ostatni warunek, a że wartość res[0] i res[1] nie jest wyzerowana/ustawiona to masz to co było w pamięci w momencie alokacji

[VanQator]

@luigi_pl: Dżisas, przerbiając tablicę statyczną na dynamiczną zapomniałem zainicjalizować ją zerami ;)
Dzięki :D

[b.....n]

@MamCieNaHita: Bój się Boga człowieku, nie namawiaj do zarządzania nieswoimi zasobami.

[VanQator]

@b33zon: ale, że co?

[luigi_pl]

@VanQator: Druga sprawa to jak już ktoś wspomniał dynamicznie zaalokowaną pamięci powinna być posprzątana za pomocą delete
Trzecia rzecz przeanalizuj bardzo dokładnie co się dzieje w momencie jak wykonuje się ta linijka:
https://github.com/VanQator/algorithms/blob/master/main.cpp#L76

[VanQator]

@luigi_pl: No za każdym razem wywoływana jest funkcja, więc spada wydajność programu. Dlatego tak nie chciałem.

[VanQator]

@luigi_pl: a nie, czekaj, #!$%@?łem

[b.....n]

@VanQator: Kolega sugeruje, że można dynamicznie zaalokować pamięć, tylko trzeba pamiętać o jej zwolnieniu, to anty-pattern. Ten kto tworzy obiekt, powinien go niszczyć. No i rzecz jasna najlepiej użyć std::vector, ale o tym zapewne wiesz i wszyscy to mówią...

[VanQator]

@b33zon: właśnie chcę bez STL'a się teraz bawić, żeby takie niuanse w kodowaniu podłapać

[luigi_pl]

@VanQator: Neh, nie miałem na myśli problemu z wydajnością, lecz to, że za każdym razem jak wywołujesz quadraticEquation w 76. linijce, to powstaje nowa tablica, obliczenia są wykonywane na nowo, bierzesz z niej tylko jedną wartość w 76 i tak trzy razy :)

[q.....n]

@VanQator: Przekompiluj program z użyciem Memory, Address, Ubsan Sanitizers i takie błędy będziesz wyłapywał od razu.
Wystarczy dodać flagi do GCC lub Clang -fsanitize=address,memory,undefined

[VanQator]

@qarmin: dobra, następnym razem to zrobie