As Marcelo Soares wrote above there is nothing wrong with your setup, however it could be changed in some way to better comply with best practices.
One common setup is:
vSwitch0:
Management - one IP subnet - vmnic0 active, vmnic1 standby (if having two physical networks cards you could use one vmnic from second card)
vMotion - separate IP subnet, separate VLAN if possible. vmnic1 active, vmnic0 passive
vSwitch1:
VM network, vmnic2 and vmnic3 active.
This physically separates the different network types and makes the setup more "clean", but makes one vmnic less for VM traffic. Look at the performance counters and see if you often consumes all three. If not you will likely be fine with two.
You could also look at how your physical switches connect to the ESXi hosts. Do you have one or several? That is an important part of a redundant vSphere cluster.