Não fiz nenhuma melhoria no seu código, porém darei algumas dicas para melhorá-lo
O saída, indesejada, que você estava encontrado quando a entrada era:
ValorUm=10;
ValorDois=5;
ValorTres=3
Saída:
NAO FORMA TRIANGULO
TRIANGULO OBTUSANGULO
Acontecia pois faltava delimitar o primeiro else
com chaves
#include <iostream>
using namespace std;
int main(){
double a=0, b=0, c=0, ValorUm=0, ValorDois=0, ValorTres=0, b2=0, a2=0, c2=0, bc=0, b2c2=0;
//aqui o usuário insere os valores...Usei as variaveis ValorUm...
//para depois inverter para a ordem "a" "b" "c" , os valores tem que
//ficar em ordem decrecente, o valor maior tem que ser a ..por
// exemplo //a== 10 , b == 5 , c == 3 do maior para o menor
cin>>ValorUm>>ValorDois>>ValorTres;
/*usei os if para inverter as posições, pelo oque testei o problema
nao esta nessa parte de trocar as posicoes*/
if(ValorUm<ValorDois&&ValorDois<ValorTres) {
c=ValorUm;
b=ValorDois;
a=ValorTres;
cout<<endl<<a<<endl<<b<<endl<<c<<endl;
}if(ValorUm>ValorDois&&ValorDois>ValorTres){
a=ValorUm;
b=ValorDois;
c=ValorTres;
cout<<endl<<a<<endl<<b<<endl<<c<<endl;
}if(ValorDois>ValorUm&&ValorTres<ValorDois&&ValorTres<ValorUm){
a=ValorDois;
b=ValorUm;
c=ValorTres;
cout<<endl<<a<<endl<<b<<endl<<c<<endl;
}if(ValorUm>ValorDois&&ValorUm<ValorTres){
a=ValorTres;
b=ValorUm;
c=ValorDois;
cout<<endl<<a<<endl<<b<<endl<<c<<endl;
}if(ValorUm>ValorDois&&ValorTres<ValorUm&&ValorDois<ValorTres){
a=ValorUm;
b=ValorTres;
c=ValorDois;cout<<endl<<a<<endl<<b<<endl<<c<<endl;
}
bc=b+c;
a2=a*a;
b2=b*b;
c2=c*c;
b2c2=b2+c2;
/*até onde sei é aqui , que está o problema o TRIANGULO OBTUSANGULO
e o TRIANGULO ACUTANGULO está com diversos erros*/
//a2 a*a b2 c2... b2c2 é b2+c2 e bc é b+c
if ((ValorUm>ValorDois+ValorTres) || (ValorUm==ValorDois+ValorTres)){
cout<<"NAO FORMA TRIANGULO"<<endl;
}
else{ **// FALTAVA ESTÁ CHAVE AQUI**
if(a2==b2c2)
{
cout<<"TRIANGULO RETANGULO"<<endl;
}
if(a2>b2c2){
cout<<"TRIANGULO OBTUSANGULO"<<endl;
}
if(a2 < b2c2){
cout<<"TRIANGULO ACUTANGULO"<<endl;
}
if(a==c && a!=b){
cout<<"TRIANGULO ISOSCELES"<<endl;
}
if(a==b && a!=c){
cout<<"TRIANGULO ISOSCELES"<<endl;
}
if(a == b && a == c ){
cout<<"TRIANGULO EQUILATERO"<<endl;
}
}**// FALTAVA ESTÁ CHAVE AQUI**
return 0;
}
Sobre "TRIANGULO ACUTANGULO" é um pequeno erro de lógica, verifique neste trecho:
bc=b+c;
a2=a*a;
b2=b*b;
c2=c*c;
b2c2=b2+c2;
DICAS
Por que fazer tantas estruturas de condições if/else? Seu código é pequeno, então por agora não terá problemas, mas e no futuro, e se tiver que implementar um código com milhares de condicionais, fará tantos if/else assim?
Opte fazer assim: ( Em C, pois não te darei o código pronto! :P)
if (a < b){ x = a; a = b; b = x; }
if (b < c){ x = b; b = c; c = x; }
Ficará mais legível!
Fonte: Tipos de Triângulos SOLUÇÃO EM C
Tipos de Triângulos SOLUÇÃO EM PYTHON