Added the two remaining functions for the CreateMotif, a MusicalStrategy implementing the Strategy Pattern.
This is the first time I’ve tried to implement this pattern.
I’m worried it’s over-engineered this feature, but it’s a complicated feature no matter which way you cut it.
Live n’ learn, I guess.
Testing went not great. A bug I thought I had squashed last feature started popping up with regularity- where the composer is creating music to start in the past.
It’s an annoying issue because it’s essentially a context switch, making me the meme:
But wait, there’s more! The game also seems to be crashing- intermittently- here:
Leave the game to play long enough and it will suddenly crash complaining about pure virtual call being called or some such nonsense.
It’s not the first time I’ve seen this issue, but now it’s happening with more regularity.
Perhaps these are issues related to the new Strategy I’ve put in place, but they aren’t the issues I was expecting.
Anyways, I figure tomorrow I’ll be in a better place to start looking at these issues in earnest, as I’ll come in expecting them.
And I have to admit to myself that its still good progress to be able to start testing at all.
PS: Also I think I’m getting a cold. Which sucks.