Question

我是C ++的新手，我想做一个非常低的微型计算器，它可以使用数据结构的技术来计算电费。它是基于菜单的程序，可以每天从用户那里获取设备和时间。我正在使用一种结构和两个堆栈，一个可以存储用户对设备的选择，另一个可以存储每天消耗的时间。

    #include<iostream>
const int size=20;
struct bill
{
    float arr[size];
    int top;
}app,hrs;

void push(bill *ps, float x);
float pop(bill *ps);
float calculation(bill *app , bill *hrs);

using namespace std;
int main()
{
    float hours=0.0, result=0.0;
    int c=1 , choice=0;
    app.top=-1;
    hrs.top=-1;

        cout<<"***********APPLIANCES******WATTS************\n";
        cout<<"1.Color TV\t150W\n";
        cout<<"2.Light Bulb\t60W\n";
        cout<<"3.Celling Fan\t50W\n";
        cout<<"4.Cloth dryer\t2500W\n";
        cout<<"5.Coffee Maker\t1100W\n";
        cout<<"6.Desktop Computer\t275W\n";
        cout<<"7.Electric Heater\t2500W\n";
        cout<<"8.Electric Kettle\t2100W\n";
        cout<<"9.Food Blender\t350W\n";
        cout<<"10.Refrigerator\t275W\n";
        cout<<"11.Hair Dryer\t2150W\n";
        cout<<"12.Air Conditioner\t2500W\n";
        cout<<"13.Iron\t1000W\n";
        cout<<"14.Laptop\t75W\n";
        cout<<"15.LED LightBulb\t8.5W\n";
        cout<<"16.Microwave\t1150W\n";
        cout<<"17.Oven\t2150W\n";
        cout<<"18.Smart Phone charger\t5.5W\n";
        cout<<"19.Vacuum Cleaner\t450W\n";
        cout<<"20.Washing Machine\t500W\n";


        while(c!=0)
        {
            cout<<"Enter 0 for exit and 1 to continue";
            cin>>c;
            if(c==0)
            break;
            else if(c==1)
            {
            cout<<"Enter your choice ";
            cin>>choice;
                push(&app, choice); 
        cout<<"Enter hours/per day used";
            cin>>hours;
            push(&hrs, hours);
        }
        else cout<<"Invalid input"; 
        }
        result=calculation(&app, &hrs);
        cout<<"Total Electricity Bill is : "<<result;
    return 0;
}

float calculation(bill *a, bill *h)
{
    float units=0.0, kWh=0.0, temp=0.0, cost=0.0;
    float w=0.0,total_w=0.0, total_h=0.0;
    for(int i=0; i<size; i++){
    if(temp=pop(&app) == 1)
    w=150;
    else if(temp=pop(&app) == 2)
    w=60;
    else if(temp=pop(&app) == 3)
    w=50;
    else if(temp=pop(&app) == 4)
    w=2500;
    else if(temp=pop(&app) == 5)
    w=1100;
    else if(temp=pop(&app) == 6)
    w=275;
    else if(temp=pop(&app) == 7)
    w=2500;
    else if(temp=pop(&app) == 8)
    w=2100;
    else if(temp=pop(&app) == 9)
    w=350;
    else if(temp=pop(&app) == 10)
    w=275;
    else if(temp=pop(&app) == 11)
    w=2150;
    else if(temp=pop(&app) == 12)
    w=2500;
    else if(temp=pop(&app) == 13)
    w=1000;
    else if(temp=pop(&app) == 14)
    w=75;
    else if(temp=pop(&app) == 15)
    w=8.5;
    else if(temp=pop(&app) == 16)
    w=1150;
    else if(temp=pop(&app) == 17)
    w=2150;
    else if(temp=pop(&app) == 18)
    w=5.5;
    else if(temp=pop(&app) == 19)
    w=450;
    else if(temp=pop(&app) == 20)
    w=500;
    else 
    cout<<"Invalid Input";
    total_w=total_w+w;
    total_h=total_h+pop(&hrs);
}
kWh=total_w*total_h*30;
units=kWh/1000;
cost=units*9;
return cost;
}

void push(bill *ps, float x)
{
    if(ps->top!=size-1)
    {
        ps->top=ps->top+1;
        ps->arr[ps->top]=x;
    }
}

float pop(bill *ps)
{
        return ps->arr[ps->top--];
}

它会产生运行时错误，并且计算不正确

每当我们按0退出时，它将给出错误的输出可以提供帮助，让我摆脱这个问题。

Answer 1

在calculation函数中，每个 if将调用pop。除非您想连续多次调用pop，否则请一次调用它并将结果存储在用于检查的变量中。

我还建议您阅读有关switch语句的信息，因为它会使您的代码更简洁。

另一个问题是，由于operator precedence，像temp=pop(&app) == 2这样的表达式等于temp=(pop(&app) == 2)。这意味着您将比较结果pop(&app) == 2分配给temp，这将使temp成为1（如果比较为true）或{{1 }}（如果是0）。由于您（在当前代码中）不使用false没关系，但这是您将来需要牢记的。

Answer 2

您的代码中存在三个问题，首先是for函数中的calculation循环。它必须是从0到顶部而不是大小，因为大小必须在未初始化的位置上包含原始值。假设您有一个5的数组，请按照以下方式说：

int a[5];
a[0]=2;
a[1]=4;

现在您说添加所有位置并给出如下结果：

result=a[0]+a[1]+a[2]+a[3]+a[4];

位置0和1分别具有2和4，但其余位置具有垃圾。

代码中的第二个问题是重复的if语句中的pop函数调用。为此，您可以在循环语句开始时执行：

top=pop(a);

a将不会通过app，因为您在a中传递了应用。

最后再添加一条语句作为检查语句或安全控件w=0;，因为在您的else无效中，正在打印并且w仍具有其先前值，因此未提供正确的输出。

即使a的选项无效，您的小时数也必须包含在总小时数中，因此，要停止这种情况，您必须使用变量h，并且如果输入有效，则必须为其提供值，否则它必须为零。因此，您的代码将如下所示：

 float calculation(bill *a, bill *h)
 {
     float units=0.0, kWh=0.0, temp=0.0, cost=0.0;
     float w=0.0,total_w=0.0, total_h=0.0,hr=0.0;
     for(int i=0; i<size; i++){
         temp=pop(a);
         w=0.0;
         hr=pop(h);
         if(temp == 1)
             w=150;
         else if(temp == 2)
             w=60;
         else if(temp == 3)
             w=50;
         else if(temp == 4)
             w=2500;
         else if(temp == 5)
             w=1100;
         else if(temp == 6)
             w=275;
         else if(temp == 7)
             w=2500;
         else if(temp == 8)
             w=2100;
         else if(temp == 9)
             w=350;
         else if(temp == 10)
             w=275;
         else if(temp == 11)
             w=2150;
         else if(temp == 12)
             w=2500;
         else if(temp == 13)
             w=1000;
         else if(temp == 14)
             w=75;
         else if(temp == 15)
             w=8.5;
         else if(temp == 16)
             w=1150;
         else if(temp == 17)
             w=2150;
         else if(temp == 18)
             w=5.5;
         else if(temp == 19)
             w=450;
         else if(temp == 20)
             w=500;
         else{
             cout<<"Invalid Input";
             hr=0;
         }
         total_w=total_w+w;
         total_h=total_h+hr;
      }
      kWh=total_w*total_h*30;
      units=kWh/1000;
      cost=units*9;
      return cost;
   }

为什么此计算代码无法正确执行？

2 个答案: