Pupils should be in the middle of the iris and be a circle. Eyeballs are round and eyelids are what gives our eyes shape, usually an oval shape. I recommend looking up pictures of eyes and tracing the shapes until you've got it down