I think the windows aren't too bad – it would be very odd if a ship that size didn't have
any – but there's a
lot of surface detail on the
Enterprise-D that detracts somewhat from the overall design, especially when compared to the comparatively smooth hulls of the original and refit 1701, or the
Excelsior. Things like the transporter emitters and especially the lifeboats just break up those clean, smooth lines, and feel a bit, dare I say, low-tech. Why do they need to be on the outside in the 24th century, the era of "technology unchained", when 22nd and 23rd century starships managed perfectly well with them hidden behind hull plates?
Just look how visually noisy all those square hatches are. Worse, the transporter emitters at the front of the saucer look like angry eyebrows!
Also, the 4ft filming model with its extra hull detailing didn't help – I remember as a kid seeing it in "Yesterday's Enterprise" and thinking that it was a deliberate modification to the model to make it look more heavily armoured. The detailing on the hull plates was too heavy, even for low-res CRT TVs.