I am overhauling some code, and have seen many maddeningly awkward constructs. Would you say this is one of them? I.e. is there a much more simple and clear way of doing it?