c++ - 我的程序运行良好，但我的老师希望它更有效率

Question

//#include conio.h   
//#include iomanip    
//#include iostream    
//#include string    

using namespace std;

const string Zo;
double Sp;
double Li;
double Ti;

void main()
{
    cout<<setiosflags(ios::fixed);
    cout<<setprecision (2);
    cout<<setw(22)<<"Speeding Ticket"<<endl;
    cout<<"Please Enter Your Speed :";
    cin>>Sp;
    cout<<"Speed Limit:";
    cin>>Li;
    cout<<"IF School Zone Enter (Yes/No):";
    cin>>"Yes"||"No";

    if(Zo=="Yes")
        Ti=30+6*(Sp-Li);
    else  
        Ti=30+3*(Sp-Li);
    if (Sp>=Li+30)
        Ti=Ti+100;

    cout<<"Your Speeding Ticket Is:"<<"$"<<Ti<<endl;
    getch();
}

这不是我的第一次尝试，第一个是 cin>>Zo 但老师说找到更好的方法，这样如果用户输入错误它就会知道。我是初学者，所以我尽可能简单地做。

Answer 1

我想你的老师想要更好的数学(这意味着你的应用程序有更好的逻辑)，一种可能的变体是:

const double additionalTicket = (Sp >= Li+30) ? 100 : 0;
const double schoolZoneMultiplier = (Zo == "Yes") ? 6 : 3; // be sure that you understand when to use strcmp and when ==
Ti = 30 + schoolZoneMultiplier*(Sp-Li) + additionalTicket;

这个变体做同样的事情，但是你看到了实际的公式，所以通常这样的代码更容易理解，而且这段代码的一部分可以由 CPU 并行执行

注意:您对 cin>>"Yes"||"No"; 有疑问 - 这没有意义