Is this the correct way of doing this?