別の3つのint変数の合計の値でint変数を宣言しました。この変数を印刷すると、大きな負の数が表示されます

Question

#include <iostream> 
#include <math.h> 

using namespace std; 

int main() 
{ 
    int l,b,h; 
    int s; 
    s=(l+b+h); 
    float ar=s*(s-l)*(s-b)*(s-h); 
    float area; 
    int ch; 
    cout<<"How do you want to calculate the area?"<<endl; 
    cout<<"1) simple formula"<<endl<<"2) heron's formula"<<endl; 
    cin>>ch; 
    if(ch==1){ 
     cout<<"Enter the sides of the triangle."<<endl; 
     cin>>l>>b>>h; 
     area=0.5*(b*h); 
     cout<<"Area of the triangle is = "<<area<<endl; 
    } 
    else if (ch==2){ 
     cout<<"Enter the sides of the triangle."<<endl; 
     cin>>l>>b>>h; 
     cout<<s<<endl<<l+b+h<<endl; 
     cout<<"The calculated area of the triangle is = "<<sqrt(ar)<<endl; 
    } 
    return 0; 
} 

これはl + b + hの正しい値を表示しますが、巨大な負の数を表示します。sのデータ型を変更しようとしましたあまりにも。これは私のほとんどすべてのプログラムで起こります。

Answer 1

sは一度計算されます（初期化されていない値を読むことでUBなので）。

あなたが代わりにラムダを作成することがあります。

auto s = [&](){ return l + b + h; }; 
auto ar = [&](){ return s() * (s() - l) * (s() - b) * (s() - h); }; 

、その後

cout << "Enter the sides of the triangle." << endl; 
cin >> l >> b >> h; 
cout << s << endl << l + b + h << endl; 
cout << "The calculated area of the triangle is = " << sqrt(ar) << endl; 

それとも単に値

cout << "Enter the sides of the triangle." << endl; 
cin >> l >> b >> h; 
const int s = l + b + h; 
const int ar = s * (s - l) * (s - b) * (s - h); 

cout << s << endl << l + b + h << endl; 
cout << "The calculated area of the triangle is = " << sqrt(ar) << endl;