You don't need methods to get and set the height, width and length of the box at all. in the constructor of the box, you already pass to the object those values. It would be better to just simply have a method called getVolume, and this method will know how to work with the height, width and length, and will just return the volume to you. You don't expose internal data of the object outside of the object, its the concept of "tell, don't ask", witch resumes to: "Procedural code gets information then makes decisions. Object-oriented code tells objects to do things."
if you want to learn more:
http://pragprog.com/articles/tell-dont-ask