use do instead of repeat