The only thing I would change is where the Netgear router is patched. I would patch it into one of your switches instead of the modem. As pictured, your wireless has no access to your LAN. If that was your goal, then you're fine... otherwise I'd enable AP mode and plug it into a switch.
You could go for a more intricate and arguably cleaner design by consolidating down to one switch, running extra cable to each room, possibly setting up VLANs, etc, but that involves time and money. Your current setup is completely functional, so If it's meeting your needs, there's nothing wrong with it.