Reading the code once again. I do not think that we need a state pattern. What is needed is just a list of (form time) and iterate over them. May be I'm wrong but it looks like we do not need three classes for that. Le 24/11/15 19:51, stepharo a écrit :
I was looking at the LoadingMorph BTW: it would be good that QA does not refer to Komitter. And package it as a separate project. QA should not dependent on Komitter because we can remove Komitter. And it makes no sense!
And it uses a state pattern, which is ok but there is not needed to create a new state all time.
SmallCogInitialState >> nextState
^ SmallCogState2 new
nextState
^ SmallCogState3 new
We can have a state pattern without creating garbage all the time.
Now I wonder who is using this LoadingMorph? Stef