Homework 5, Feedback.

• If you type make, and anything but the prompt shows up, don't turn it in. It is worth 0 and there is no redo.
• If you have two colorT's compare them, not the strings.
  • String comparison is much more expensive.
  • And this is the reason you built a colorT.

  • string color1 = ColorTToString(player.favoriteColor);
    string color2 = ColorTToString(balloon.color);
    
    if (color1 == color2) {
    ...
    } 

• Don't do this

  • cout << "You found a " << ColorTToString(balloon.color) << " balloon in " << balloon.location << " worth " << balloon.value << " points." << endl;
            

  • Do this

  • cout << "You found a " << ColorTToString(balloon.color) 
         << " balloon in " << balloon.location << " worth "
         << balloon.value << " points." << endl; 

• Don't do this

  • void PrintBallloon(BalloonT balloon) {
        ColorT tmpColor;
        string tmpLocation;
        int tmpPonts;
    
        tmpPoints = balloon.points;
        tmpColor = balloon.color;
        tmpLocation = balloon.location;
    
        cout << tmpPoints  << " " ...
    
    } 

  • do This

  •     cout << balloon.points  << " " << balloon.location ...