It's a pretty common thing people start out doing, no worries.
In general, I think a good tip for photography of any sort is that you should think about it in part just like an abstract painting. Think about how your retina and low level visual system see it, before you assign any meaning. The meaningful content matter (like "oh that's a person and a camera and a wall!") is at one level, but at another level it's just shapes, and edges, and contrast, and "lightest thing in the scene" and "darkest thing in the scene" and color etc.
The best photos will use all or many of those low level, sub-meaningful visual elements as tools, intentionally, to draw the viewer's eye to whatever meaningful thing they want the viewer to be paying most attention to, before they even know what it is their eye is being drawn toward.
Not only is this simply visually powerful and compelling, but it's also efficient in getting you to see the point of the photo in what might be very limited time (glancing over an advertisement or flipping through a bunch of photos)
So usually, in a portrait, this would mean that the person's eyes or at least face should be:
* comparatively bright, and/or
* comparatively high contrast, and/or
* have lines pointing toward them, and/or
* be color neutral or color-interesting in the photo, and/or
* Be the most in-focus thing in the image,
etc.