Skip to main content
adicionou 905 caracteres ao conteúdo
Fonte Link
Luiz Augusto
  • 2,5mil
  • 17
  • 16
  • 30

Sobre "TRIANGULO ACUTANGULO" é um pequeno erro de lógica, verifique neste trechoas estruturas de condição if/else, pois há alguns if sem o seu respectivo else, por mais que não seja obrigatório cada if possuir seu else, é uma boa pratica faze-lo.

Lembre-se das boas práticas de programação do Deitel:

bc=b+c; 
a2=a*a;
b2=b*b;
c2=c*c;
b2c2=b2+c2;
  1. Indentar ambos os comandos do corpo de uma estrutura if/else;
  2. Se existem vários níveis de indentação, cada nível deve ser indentado pelo mesmo espaço adicional

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?

  1. 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?
  1. Veja o quão mais limpo ficará seu código usando o Operador ternário;
  2. Ao nomear variáveis evite nomes tais como: bc; a2; c2; b2c2; Dê nomes significativos as variáveis.

FicaráPara sanar mais legível!dúvidas, leie esta resposta: Padrão de nomenclatura no código para o C#

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;

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?

Ficará mais legível!

Sobre "TRIANGULO ACUTANGULO" é um pequeno erro de lógica, verifique as estruturas de condição if/else, pois há alguns if sem o seu respectivo else, por mais que não seja obrigatório cada if possuir seu else, é uma boa pratica faze-lo.

Lembre-se das boas práticas de programação do Deitel:

  1. Indentar ambos os comandos do corpo de uma estrutura if/else;
  2. Se existem vários níveis de indentação, cada nível deve ser indentado pelo mesmo espaço adicional
  1. 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?
  1. Veja o quão mais limpo ficará seu código usando o Operador ternário;
  2. Ao nomear variáveis evite nomes tais como: bc; a2; c2; b2c2; Dê nomes significativos as variáveis.

Para sanar mais dúvidas, leie esta resposta: Padrão de nomenclatura no código para o C#

Fonte Link
Luiz Augusto
  • 2,5mil
  • 17
  • 16
  • 30

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