[TymRazemNieBedeBordo]

Odnośnie mojego problemu z tego posta: https://www.wykop.pl/wpis/33780461/chciej-stworzyc-archiwum-zip-z-jednym-pliczkiem-ea/

Generalnie, problem polegał na tym, że nie było problemu ( ͡° ͜ʖ ͡°) Metoda Dispose() działała jak należy, więc czemu się nie zapisywało?

Ano dlatego, że przed tamtą metodą na start programu odpalałem inną, która wczytywała plik i go rozpakowywała. Doszedłem do tego, że mimo wywołania Dispose() plik ciągle był w pamięci. Dlaczego? Ano dlatego, że w ciągu kilku sekund po wczytaniu pliku od razu chciałem go nadpisać.

Co powodowało, że Garbage Collector nie mógł zadziałać bo miał za malo czasu by zebrać plik.

Teraz przed każdym wczytaniem, manipulacją pliku ręcznie wywołuje GC.Collect(). I działa.

Czy to jest rzeczywistość programisty? Gdzie głowimy się nad problemem kilka godzin, a okazuje się, że jedna linijka rozwiązuje problem? ( ͡° ʖ̯ ͡°)

#naukaprogramowania #csharp

[b.....a]

@TymRazemNieBedeBordo: robisz to zle.

[TymRazemNieBedeBordo]

@bacteria: co?

[b.....a]

@TymRazemNieBedeBordo: robisz to zle i zostalo Ci to wytlumaczone w ostatnim poscie.

Naprawde rzadko kiedy trzeba recznie wywolywac GC.Collect(). W 99% przypadkow jest gdzie widze taki kod jest fundamentalnie blednych.

[C.....y]

@TymRazemNieBedeBordo: Robisz to katastrofalnie, nie źle.

[TymRazemNieBedeBordo]

@bacteria: @Czesiowcy: nie użyłem tagu #naukaprogramowania by się pochwalić tylko by czegoś się nauczyc. słowa "zostało ci to już wytłumaczone" "robisz to katastrofalnie" to domena elektrody, odeślijcie mnie jeszcze do google i dajcie warna.

Co robię źle? Użycie dispose? Nie używam go przy using jak mi zostało wytłumaczone. Użycie streamu? Będe nad tym myslał, jeśli zrobię resztę logiki w kodzie

[b.....a]

@TymRazemNieBedeBordo: nie p-----l tylko czytaj ze zrozumieniem. W ostatnim poscie wyrzuciles z kodu blok using co bylo bledem. Teraz brniesz w kolejny spaghetti code zamiast doczytac, ewentualnie dopytac i zrozumiec.

[C.....y]

@TymRazemNieBedeBordo: Istnieje bardzo niewiele przypadków, gdzie manualne wywołanie GC.Collect() ma jakikolwiek sens i ten tutaj się do nich nie kwalifikuje. GC.Collect() to blokująca i bardzo zasobożerna operacja, lecąca po wszystkich generacjach GC; w kodzie produkcyjnym mogłaby stanowić bardzo wąskie gardło. Operujesz na plikach, gdzie będzie - pod spodem - w użyciu sporo niezarządzanych referencji, Dispose() zapewne je zamyka. Jeżeli jednak problemem jest wielokrotne tworzenie ZipFile, to dlaczego po prostu nie

[TymRazemNieBedeBordo]

W ostatnim poscie

@bacteria: ale mirko to nie jest moja kontrola wersji

stnieje bardzo niewiele przypadków, gdzie manualne wywołanie GC.Collect() ma jakikolwiek sens i ten tutaj się do nich nie kwalifikuje. GC.Collect() to blokująca i bardzo zasobożerna operacja, lecąca po wszystkich generacjach GC; w kodzie produkcyjnym mogłaby stanowić bardzo wąskie gardło. Operujesz na plikach, gdzie będzie - pod spodem - w użyciu sporo niezarządzanych referencji,Dispose() zapewne je zamyka. Jeżeli jednak problemem

[C.....y]

@TymRazemNieBedeBordo: Jasne, ale jeśli to dobrze rozumiem, usuwasz z pamięci Zipa tylko po to, żeby stworzyć następnego. Dlaczego nie operujesz na jednym, jeśli to jedna ścieżka i jeden plik?

[s.....k]

@TymRazemNieBedeBordo: jesteś dobrym programistą, nie słuchaj ich ( ͡° ͜ʖ ͡°)

[TymRazemNieBedeBordo]

@Czesiowcy: Faktycznie, teraz myślę, że mógłbym stworzyć publiczną klasę ZipFile i za każdym razem gdy jest potrzebna po prostu przesłać do danej metody, by ta manipulowała plikami. Wtedy nie musiałbym korzystac z GC i byłoby to jasniejsze.

[C.....y]

@TymRazemNieBedeBordo: Nie wiem, jaki masz dokładny kod, ale tak, stworzenie tego albo jako pola, albo jako zmiennej wew. jednej metody, która wywołuje po kolei mniejsze metody manipulujące Zipem brzmi jak całkiem sensowny refaktor.

[TymRazemNieBedeBordo]

@Czesiowcy: dziękuje za czas i wytłumaczenie :)

[C.....y]

@TymRazemNieBedeBordo:

Luzik.