I wouldn't say so at all. Poor eyesight carries on smartly. Baldness. I enjoy both.
But an old story about the controller code for a surface-to-air missile comes to mind.
Someone looking at the memory allocator spots an obvious resource leak: "This code is going to crash."
The reply was that, while the point was theoretically valid, it was irrelevant, since the system itself would detonate long before resource exhaustion became an issue.
So too prostate cancer back in the day: war, famine and plague were keeping the lifespan well below the threshold of every man's time bomb.
A better method would be to confine the program to monozygotic twin pairs of young women with spotless genetic heritage, and inseminate one twin with frozen sperm and the other with current sperm. The "current sperm" child (CS) could be closely monitored, and the "frozen sperm" (FS) child fitted with an explosive chastity device which, in the event that CS is found to have developmental issues, are remotely-detonated to ensure the tainted line does not persist.
Simple-as.
But you freeze the sperm at the peak of freshness. Then you wait and see how the donor does. If they live to a ripe old age without old age diseases, then go select their sperm. Otherwise, destroy it.
You can probably do this with eggs too. When a child is desired, select an egg and a sperm off the shelf, and there you go. Easy peasy. Your device ensures only munitions experts can procreate outside the system, and I think we'll need a lot of munitions experts in the ensuing generations.