I like the general approach. It seems highly specific to the specific Model type though, and part of me wonders if isMyModelValid() should be a receiver method that accepts a Model. I suppose making it a receiver method doesn't really accomplish anything other than tie the code a bit more closely to the thing it's working on. I'm still unlearning decades of object-oriented bias.