This is such a pervasive style choice on her part that I cannot imagine engaging with it as being "bad" at coming up with names
Like, you can fairly say it's not your bag, but the layers of on-the-nose, punny names are a feature, not a bug, of her creative process
I would describe it as stylistic choice that made some sense at the beginning when the first few books were aimed at younger kids, but is out of sync with the grown up themes of the other books.
Looking at the series as a whole, it does seem to have a lot of issues with Rowling being overly ambitious with worldbuilding a young kids’ fantasy series but being really green with the writing process. She locked in a bunch of details without seeing how it would all cohere or not by the end.
I don't think it holds up to the Tolkien-style analysis people want, but it's remarkable how well it handles the tone shift overall, how the changing relationship to magic comes to be a fairly deliberate metaphor for growing up, and how actually reading the books gives the *appearance* of cohesion
Tolkien if anything has the opposite problem - his protagonists and their POVs are *too* similar to the expected English reader given the world he's set up.
Though it's not really one in practice, both because he's a better worldbuilder, but also because the reader is obviously less familiar with Middle-Earth than they are with the Muggle "real world" in HP and so don't see any inconsistencies.
