2016-05-31 55 views
0

我目前有一个问题,即我的程序只执行第一个 if语句,它将'w'乘以3.5,即使'w'不符合要求。如果我把4的输入,输出变成14(4 * 3.5),它应该真的是34(4 * 8.5),因为重量在3到10磅之间。我确信修复很简单,但我似乎无法找到它!程序只执行第一条IF语句

#include <iostream> 
using namespace std; 

int main() 
{ 
float w, price; 
cout << "Enter weight of package : "; 
cin >> w; 

if (0 < w <= 1) 
{ 
    price = 3.5 * w; 
} 
else if (1 < w <= 3) 
{ 
    price = 5.5 * w; 
} 
else if (3 < w <= 10) 
{ 
    price = 8.5 * w; 
} 
else if (10 < w <= 20) 
{ 
    price = 10.5 * w; 
} 
else if (20 < w <= 30) 
{ 
    price = 12.5 * w; 
} 
else if (30 < w) 
{ 
    cout << "The package cannot be shipped" << endl; 
} 
else 
    cout << "Invalid input" << endl; 
cout << "Weight : " << w << " lbs" << endl; 
cout << "Shipping cost : $" << price << endl; 
} 

回答

3

你可能打算用:

if (0 < w && w <= 1) 

相反的:

if (0 < w <= 1) 

您当前的方法(使用您的例子w = 4)的计算结果是这样的:

if ((0 < 4) <= 1) 

哪简化为:

if (1 <= 1) 

其中评估结果为true

根据C++标准(4.7节,第4章):

如果源类型是bool,值false被转换到零和 值true被转换为一个。

+1

啊我明白了!我已经完全忘记了使用'&&',以及我的方法会如何解释。非常感谢你 – Gizmoh

3

if(3<w<20)是错误的说法。你应该有if(3<w&&w<20)

2

a < w < b不会产生你所期望的效果,但实际上结果为(a < w) < b),其中a < b将有一个真/假值。我建议像这样重写你的代码:

if (w <= 0) 
{ 
    cout << "Invalid input" << endl; 
} 
else if (w <= 1) 
{ 
    price = 3.5 * w; 
} 
else if (w <= 3) 
{ 
    price = 5.5 * w; 
} 
else if (w <= 10) 
{ 
    price = 8.5 * w; 
} 
else if (w <= 20) 
{ 
    price = 10.5 * w; 
} 
else if (w <= 30) 
{ 
    price = 12.5 * w; 
} 
else 
{ 
    cout << "The package cannot be shipped" << endl; 
} 

cout << "Weight : " << w << " lbs" << endl;