Are you trying to match the picture? If you are, then a few things I have are that your shelves are too thick, and overall proportions are a bit off for each section.
Another piece of advice I have is to not be afraid to use more than one mesh. I remember starting out trying to make something like this one mesh, when you can make each shelf its own box, and even section it off into several pieces like the top molding could be its own mesh, the open section could be its own mesh, the base cabinetry can be its own mesh, and the cabinet doors could each be its own mesh. It just depends on your limitations.
Hey! This model isn't really ready for texturing. Will you be creating a high poly then baking a normal map to a low poly? You're going to have a very difficult time getting this to match the reference if you're not baking down details. You could cut down on a huge number of polygons just by building this mesh as if you were building it in real life. For instance, your shelves have an edge right through the middle of them. I understand this is might be for mirroring reasons, but now you should be deleting them if poly count is something to be worrying about.
You've utilized the extrude function a ton of times, unfortunately incorrectly. Check out my drawing below. The yellow outline is showing the curved shape of the molding in the reference, whereas you've just extruded a bunch of faces. Its the same with the bottom of the mesh as well. Your mesh is square on the bottom, but if you look at the bottom of the reference, the front of the shelf is pushed in. A lot of your extrusions should have a bit, if not a lot, of offset, instead of just being straight in or out. You're currently missing a lot of the round pieces (blue arrow) which will take away from the square blocky look it currently has. The inside back are of the shelf should have an extrusion to make the inside of the shelves look more interesting.
Replies
Another piece of advice I have is to not be afraid to use more than one mesh. I remember starting out trying to make something like this one mesh, when you can make each shelf its own box, and even section it off into several pieces like the top molding could be its own mesh, the open section could be its own mesh, the base cabinetry can be its own mesh, and the cabinet doors could each be its own mesh. It just depends on your limitations.
I'm not sure what using more than one mesh means, but I think the professor wants everything one object.
Thank you for your help!
You've utilized the extrude function a ton of times, unfortunately incorrectly. Check out my drawing below. The yellow outline is showing the curved shape of the molding in the reference, whereas you've just extruded a bunch of faces. Its the same with the bottom of the mesh as well. Your mesh is square on the bottom, but if you look at the bottom of the reference, the front of the shelf is pushed in. A lot of your extrusions should have a bit, if not a lot, of offset, instead of just being straight in or out. You're currently missing a lot of the round pieces (blue arrow) which will take away from the square blocky look it currently has. The inside back are of the shelf should have an extrusion to make the inside of the shelves look more interesting.