c++, pole deltoidu

Question

Siema wszystkim mam problem z skompilowaniem kodu, otóż program ma za zadanie obliczać pole deltoidu jak wpisze się jego wysokości. Będę bardzo wdzięczny jak pomożecie. PS jestem dopiero początkującym :)

#include <iostream>
using namespace std;
int longer,shorter,score;

int main()
{
    cout<<"Hi, this is program to calculate a deltoid field"<< endl;
    cout<<"Give the length of the longer height:"<< endl;
    cin>>longer>>endl;
    cout<<"Give the length of the short height:"<<endl;
    cin>>shorter>>endl;
    if(longer||short==0)
    {
        cout<<"Bzzz something is wrong"<<endl;
        cout<<"Give the length of the longer height:"<<endl;
        cin>>longer>>endl;
        cout<<"Give the length of the short height:"<<endl;
        cin>>shorter>>endl;

        score = (longer*shorter)/2;
    count<<"This is your score:"<<score endl;
    return 0;
    }
    score = (longer*shorter)/2;
    count<<"This is your score:"<<score endl;
    return 0;
}

 

Answer 1

Wyrażenie

if(longer||short==0)

zwraca prawdę zawsze, gdy longer będzie różne od zera. Poprawnie powinno być:

if(longer==0 || shorter ==0)

ponieważ każde wyraźenie sprawdzane jest osobno. Poza tym na końcu zamiast cout napisałeś count i w dwóch miejscach pod koniec brakuje << przed endl. Na przyszłość sprecyzuj, jakie błędy pokazuje kompilator.

Comments

#include <iostream>
using namespace std;
int longer,shorter,score;

int main()
{
    cout<<"Hi, this is program to calculate a deltoid field"<< endl;
    cout<<"Give the length of the longer height:"<< endl;
    cin>>longer>>endl;
    cout<<"Give the length of the short height:"<<endl;
    cin>>shorter>>endl;
    if(longer==0|| short==0)
    {
        cout<<"Bzzz something is wrong"<<endl;
        cout<<"Give the length of the longer height:"<<endl;
        cin>>longer>>endl;
        cout<<"Give the length of the short height:"<<endl;
        cin>>shorter>>endl;

        score = (longer*shorter)/2;
    cout<<"This is your score:"<<score endl;
    return 0;
    }
    score = (longer*shorter)/2;
    cout<<"This is your score:"<<score endl;
    return 0;
}

Niewiem gdzie wstawić << bo na końcu ma wyświetlać score

 

 

---

cin >> longer;

Wystarczy tak, przy cin nie używa się endl (bo nie wypisujemy nic na ekran), to samo przy wczytywaniu kolejnych zmiennych. A << wstawiasz zawsze przed endl, powinno wyglądać to tak:

cout << "This is your score:" << score << endl;

Poza tym powinieneś zmienić typ zmiennej score z int na float lub double, ponieważ przy dzieleniu przez dwa zawsze będzie zaokrąglone w dół, bo int może przechowywać jedynie liczby całkowite.

---

oki dzięki tylko jest mały problem, przy lini 11 wyskakuje czerony prostokąt 

#include <iostream>

using namespace std;

int longer,shorter,score;

int main()
{
    cout<<"Hi, this is program to calculate a deltoid field"<< endl;
    cout<<"Give the length of the longer height:"<< endl;
    cin>>longer>>;
    cout<<"Give the length of the short height:"<<endl;
    cin>>shorter>>;
    if(longer||shorter==0)
    {
        cout<<"Bzzz something is wrong"<<endl;
        cout<<"Give the length of the longer height:"<<endl;
        cin>>longer>>;
        cout<<"Give the length of the short height:"<<endl;
        cin>>shorter>>;

        score = (longer*shorter)/2;
    cout<<"This is your score:"<<score<<endl;
    return 0;
    }
    score = (longer*shorter)/2;
    cout<<"This is your score:"<<score<<endl;
    return 0;
}

 

---

Jeszcze raz porównaj z tym, co ci napisałem (początek poprzedniego komenarza). Usuń >> przed średnikiem.

---

A mógłbym zapytać dlaczego tak się dzieje, że nie wstawiamy >> przed średnikiem w lini 11, 13, 18 i 20 pytam z czystej ciekawości

---

Dlatego, że >> (przeładowanych operatorów przesunięcia bitowego) używasz w przypadku, gdy chcesz po nim wczytać kolejną liczbę, a średnika, kiedy nie chcesz już nic wczytywać. Tak więc przykładowo:

int x, y;
cin >> x >> y;

Tak samo w przypadku cout.