No, that is not best-practice.
The rules with VLANs:
1/ Extend each VLAN on the fewest possible links
2/ Create the minimum possible number of VLANs on each switch.
The textbooks *still* demonstrate VLANs with examples such as "finance", "marketing", "sales", etc...
This is completely wrong.
Originally, VLANs were used for "security", to prevent some hosts from seeing some other hosts or servers, but ever since NT (25 years ago), you don't use VLANs to enforce access policies for resources any more.
VLANs are there to segment your network to make it easier to manage and troubleshoot and to mitigate some risks.
VLANs should be used geographically - networks should be segmented per switch or per wiring closet or per floor or per building, depending on what you are working with.