The variable `exerciseId` is always specified (all the callers use it), but I'm not certain that it's impossible for `exerciseId` to be set as `undefined` (though there is no evidence of that in the issues), or at the very least for `problems.length` to be zero. I can set this to else `if (problems.length > 0)` or simply `else` if that would be clearer/better.