Instead of returning null would it better to throw an error here? fail-fast makes debugging easier