Простейшая программа,нуб)

[annum]

Доброго времени суток.Вообщем в С++ я вообще бревно,новичок,мы только начали изучать его. Надо вообщем написать программу,которая считывает значение х,а,б . И потом в зависимости от их значений высчитывает одну из трёх функций. На дефли это раз плюнуть,но с С++ я просто 0 ) помогите

#include <vcl.h>
#include <iostream.h>
#pragma hdrstop

//---------------------------------------------------------------------------

#pragma argsused
int x,a,b;
float z;
void log(float);
void sin(float);
int main(int argc, char* argv[])
{
cout<<"Enter A:";
cin>>a;
cout<<"Enter B:";
cin>>b;
cout<<"Enter X:";
cin>>x;
 if (x<=a)
{
z=log(x)+sin(x);
return 0;
}
 if ((a<x) and (x<b))
{
z=log(x)+cos(x);
return 0;
}
 if (x>=b)
{
z=tan(x);
return 0;
}
cout>>z;
cin.get();

        return 0;
}

ошибки такие
[C++ Error] Unit1.cpp(24): E2109 Not an allowed type
[C++ Error] Unit1.cpp(27): E2377 If statement missing )


я понимаю что несовместимость типов,но чёрт..не могу сообразить что куда.Помогите ) Заранее благодарен!

[Obey-Kun]

#include <iostream>
#include <cmath>

using namespace std;

int main()
{
    cout << "Enter A: ";
    int a;
    cin >> a;
    cout << "Enter B: ";
    int b;
    cin >> b;
    cout << "Enter X: ";
    int x;
    cin >> x;
    int z;
    if(x <= a) {
        z = log(x) + sin(x);
    } else if((a < x) && (x < b)) {
        z = log(x) + cos(x);
    } else {
        z = tan(x);
    }
    cout << z << endl;
    cin.ignore();
    cin.get();
    return 0;
}

1. Убрал return'ы (ведь тебе надо вывести z на экран, а ты завершал программу до этого).
2. Подключил мат. библиотеку.
3. Использовал нормальное написание include'а (без .h).
4. Вошёл в пространство имён std.
5. Вместо and надо использовать &&.
6. Если не выполены первые 2 условия, то третье получается автоматом.
7. Если выполнено первое условие, но второе и третье проверять не надо. Короче, надо использовать else.
8. <не-не-не>
9. Убрал лишние инклюды.
10. Убрал аргументы main.
11. Убрал лишние указания компилятору.

И не используй этот C++ builder, у него преимуществ никаких. А ещё он платный, а воровать плохо.

[Пепел Феникса]

по поводу 8 пункта.
имеет он такой метод
http://cplusplus.com/reference/iostream/istream/

[Obey-Kun]

Верно, имеет. Но делает не то, что надо .

[Пепел Феникса]

однако под паузу годится(по крайней мере что в gcc, что в MSVS)

[Obey-Kun]

И логичнее, кстати, делать не так

    if(x <= a) {
        z = log(x) + sin(x);
    } else if((a < x) && (x < b)) {
        z = log(x) + cos(x);
    } else {
        z = tan(x);
    }

а так

    if(x <= a) {
        z = log(x) + sin(x);
    } else if(x >= b) {
        z = tan(x);
    } else {
        z = log(x) + cos(x);
    }

Ибо тут как минимум писанины меньше. И шансов ошибиться тоже. И выполняется на пару тактов быстрее.

[Obey-Kun]

у меня cin.get() не паузует, ЧЯДНТ?

[Пепел Феникса]

наверно чтото в потоке остается после ввода чисел(чет не учел)
тогда clear get
ну или ваш вариант.

[Obey-Kun]

блин, а просто getline()-то нету, нужны параметры.
лучше таки get().
вот только почему оно у меня не работает, ума не приложу. поток-то чистый должен быть.

[Obey-Kun]

cin.ignore();
cin.get();

работает как надо