1.Add comments to explain what you are trying to do. 2.No "using namespace std;" 3.Clearly declare "private" inside of class. For example: use ```c++ class item{ private: int price; public: getPrice(); } ``` instead of class ```c++ class item{ int price; public: getPrice(); } ``` 4.Declare inheritance type(public, private, protected) when using inheritance. 5.If not finished with code, add a TODO as comments.