c++ - 我的程序找到了三角形 C++ 的类型，但我在使用 if-else 语句时遇到了问题

Question

这是我正在处理的简单代码。该代码的目标是接受三个浮点值，然后计算出它是什么类型的三角形；等边、等腰、斜角、直角或这些的组合。

#include <iostream>
using namespace std;

int main(){

float side_a, side_b, side_c;
int equi, iso, sca, right_angle;
int equi_same, iso_same, sca_same, right_angle_same;


cin >> side_a >> side_b >> side_c;

if( (side_a == side_b) && (side_b == side_c) && (side_a == side_c ) ){

    cout << "\nThis is an equilateral triangle";

    equi = 1;
    iso = 0;
    sca = 0;
    right_angle = 0;
    equi_same = 1;
}

else if((side_a == side_b) || (side_b == side_c) || (side_b == side_c)){

    cout << "\nThis is an isosceles triangle";

    iso = 1;
    equi = 0;
    sca = 0;
    right_angle = 0;
    iso_same = 1;

}

else if( ( (side_a*side_a) == ( (side_b*side_b) + (side_c*side_c) ) ) ||
         ( (side_b*side_b) == ( (side_a*side_a) + (side_c*side_c) ) ) ||
         ( (side_c*side_c) == ( (side_a*side_a) + (side_b*side_b) ) )  ){


    cout << "\nThis is a right angled triangle";

    right_angle = 1;
    equi = 0;
    iso = 0;
    sca = 0;
    right_angle_same = 1;       

}

else{

    cout << "\nThis is a scalene triangle";

    sca = 1;
    equi = 0;
    iso = 0;
    right_angle = 0;
    sca_same = 1;

}


if( (equi = 1) && (equi_same != 1) ){
    cout << " and an equilateral triangle";
}

else if((iso = 1) && (iso_same != 1)){
    cout << " and an isosceles triangle";
}

else if((right_angle = 1) && (right_angle_same != 1)){
    cout << " and a right angled triangle";
}

else if((sca = 1) && (sca  != 1)){
    cout << " and a scalene triangle";
}

else{
    cout << "\n";
}

return 0;

}

我遇到困难的是如何让代码发现它也是其他三角形的组合。编写代码后，我意识到如果第一个语句为真，那么代码将不会检查其他 else if 语句。是否有任何我遗漏但未注意到的错误？如果不是，那么我应该如何更改我的代码？提前致谢!

编辑:在测试运行中，如果我输入相同的数字，输出将是:

It is an equilateral triangle and a right angled triangle

但是如果我输入满足其他要求的数字，输出:

It is a (depends on input) triangle and an equilateral triangle

也许我在直角部分或等边部分做错了什么或者我错过了什么？

Answer 1

你把它复杂化了。尝试创建不同的函数来确定三角形是否属于一个类型，并标记它，并使用 == 而不是 =:

我会怎么做:

在你的第一个测试中不要写else if-s，只写ifs。不输出结果，只设置标志。

稍后检查标志，并写入结果。

if(Equi(side1, side2, side3))
    equi = 1;
if(Iso(side1, side2, side3))
    iso = 1;
if(Sca(side1, side2, side3))
    sca = 1;
if(RightAngled(side1, side2, side3))
    rang= 1;

std::cout << "This is a " << equi == 1 ? "equilateral" : "" << ... << " triangle.";

编辑:使用using namespace std，您可以省略开头的std::。equi == 1 ? "equilateral": "" 是一个条件表达式:在?标记之前有一个条件，如果为真，则返回"equilateral": ""表达式的第一边，如果为假，返回一个空字符串(它是 ””)。所以它只会在设置了标志的情况下输出一些东西。查看http://www.cplusplus.com/forum/articles/14631/

例子:

// condition ? if_true_return_this : if_false_return_this
int x = true ? 1 : 2; //x will be 1
x = ((x == 2) ? 0 : 3); //x will be 3, since x == 2 is false, so the second in ... : ... is returned.

Equi(...) 和其他函数是函数，您可以用第一部分中的语句替换它们，我只是不想复制它以获得更紧凑的代码。

希望对你有帮助。