Heya, so I think I found what's wrong and fixed it accordingly. So, in your Crayons class you have statements like color=color and length=length, but these statements are pretty reduntant. What you want to do is to do this.length and this.color in order to specify that the length you are referring to is the one in the parameter and the color you're specifying is the color in set in the parameter of the class.
JamesCherrill commented: You are right - but after 6 years it's probably too late! +15