[stacktrace]

Mirki zacząłem pisać w #cpp, potrzebuję mieć w swoim programie kolejkowanie zdarzeń. Napisałem coś takiego: http://ideone.com/X75trz

Problem i program wydaje mi się prosty. Starałem się uprościć kod dlatego by milion klas nie przesłaniał problemu. Jeśli będzie trzeba to wytłumaczę swoje intencje.

Do Was mam prośbę o ocenę kodu, czy robię to dobrze ;)

[l.....l]

@stacktrace: Po pierwsze:

Nigdy, przenigdy, nie rób shared_ptr na "this"...!

To tylko na wstępie czytam dalej..

[Rincewind]

@stacktrace: @losiul: Do tego istnieje klasa

std::enable_shared_from_this

.

[l.....l]

@Rincewind: Tja, ale generalnie nawet to nie uchroni Cię przed używaniem delete na zmiennej na stosie ; ).

@stacktrace:

Źle używasz sharedptr'a. W C++ obiekty leżą w dwóch miejscach: na

[stacktrace]

@losiul: Co do sterty i stosu to na początku tak właśnie myślałem, ale potem popróbowałem i zobaczyłem, że program działa i zdarzenia są dostępne również po wykonaniu funkcji producer. Nie dawało mi to spokoju więc głównie dlatego postanowiłem zapytać o to. Chciałem uniknąć używania new.

Co do asynchroniczności masz rację, ale nie chciałem zaciemniać tego kawałka kodu.

Mój zamysł jest taki, że mam centralną kolejkę, która przechowuje wszystkie zdarzenia. Mam

[l.....l]

@stacktrace: nie wiem czy to dopasuje sie dobrze do Twoich wymagan, ale pomysl o tego typu rozwiazaniu:

Kolejka eventow zamknieta jest w klasie szablonowej, parametryzowanej typem eventa.

[stacktrace]

@losiul: Na początku tak kombinowałem tylko chciałem mieć w efekcie końcowym jedną kolejkę. Jednak mogłaby ona przyjmować do siebie tylko const Event & co powodowałoby rzutowanie w górę, a że to kopia to już w dół nie wrócę.

Wychodzi na to, że muszę skorzystać z Twojego pomysłu tylko jeszcze dopracować zarządzanie listą kolejek.

[l.....l]

@stacktrace: Zarządzanie? Najprostszy, choć nie do końca elegancki sposób widzę taki (EventQueue to klasa o której pisałem wyżej):

template

class EventListener: public