Объем конуса

[Narsky]

Здравствуйте.
Задача:

Нахождение объема конуса. Написать программу, которая запрашивает у пользователя высоту и радиус основания конуса, рассчитывает объем и выводит результат вычисления на экран. При составлении программы учесть, что высота и радиус основания конуса – положительные величины.

Примечание: Все операции производятся в функции main(). Если в задании не указан явно тип операндов, предполагается использование вещественных чисел. В данном блоке заданий приведены условия, которые программа должна проверять при вводе пользовательских данных. Программа должна также выявлять и корректно реагировать на другие исключительные ситуации (например, деление на ноль и ситуацию переполнения). В последующих блоках заданий студентам предлагается определять эти условия самостоятельно, исходя из постановки задачи.

Код:

#include <iostream>
using namespace std;

#define Pi 3.14

int main()
{
double h, r, V;
cout << "Vvedite visotu:" << endl;
cin >> h;
if (h > 0)
{
cout << "Vvedite radius:" << endl;
cin >> r;
if (r > 0)
{
V = Pi * r^2 * h / 3;
cout << "Ob'em konusa raven:" << V << endl;
}
else cout << "Oshibka! Visota ne mozhet bit' otricatel'noi" << endl;
}
else cout << "Oshibka! Visota ne mozhet bit' otricatel'noi" << endl;
return 0;
}

Выдает ошибку: вложенный файл (выражение должно относиться к целочисленному типу). Как исправить?
Есть ли какие-нибудь возможные недочеты программы и её возможные улучшения?

[p51x]

^ - это не степень, для степени есть функция pow

[Narsky]

Хорошо, а что с числом pi?
И небольшой вопрос не в тему: пользователю нужно ввести шестизначное целое число, как сделать так, чтобы при вводе не шестизначного числа (пятизначного или семизначного например) выводилась ошибка?

[ura_111]

1) Попробуй написать просто const double Pi=3.14;

2) Любое шестизначное число находится в интервале 99 999 - 1 000 000.
Ну вот, и организуй проверку.

[p51x]

Хорошо, а что с числом pi?

https://msdn.microsoft.com/ru-ru/library/4hwaceh6.aspx

пользователю нужно ввести шестизначное целое число, как сделать так, чтобы при вводе не шестизначного числа (пятизначного или семизначного например) выводилась ошибка?

В лоб проверить длину полученной строки или перевести эту строку в число и сравнить диапазон.

[Narsky]

Решил проблему с pi, но что делать тут?

{
double r2 = pow ( r, 2 ) / 3;
V = M_PI* r2*h;
cout << "Ob'em konusa raven:" << V << endl;
}

Ошибка: операнд * должен быть указателем.

[Narsky]

Сообщение от p51x

https://msdn.microsoft.com/ru-ru/library/4hwaceh6.aspx


В лоб проверить длину полученной строки или перевести эту строку в число и сравнить диапазон.

Можно вопрос: чему не должно быть равно С? 12 не подходит

#include <iostream>
using namespace std;

int main(){
setlocale(LC_ALL, "Russian");
unsigned short int C, c1, c2, c3, c4, c5, c6;
cout << "Введите номер билета из 6 цифр" << endl;
cin >> C;
if (sizeof(C) != 12) {
cout << "Ошибка! Введено число не из 6 цифр" << endl;
system ("pause");
}
else {
c1 = C / 100000;
c2 = C % 100000 / 10000;
c3 = C % 10000 / 1000;
c4 = C % 1000 / 100;
c5 = C % 100 / 10;
c6 = C % 10;
if (c1 + c2 + c3 == c4 + c5 + c6) {
cout << "Поздравляем, это счастливый билет!" << endl;
}
else cout << "Увы, этот билет не счастливый." << endl;
}
return 0;
}

[p51x]

Дайте скрин с кодом и сообщением компилятора.

чему не должно быть равно С? 12 не подходит

Не, так не пойдет. Вы же уже ввели число, а не строку... Сравнивайте на диапазон.

[Narsky]

Вот.

[Narsky]

Сообщение от p51x

Дайте скрин с кодом и сообщением компилятора.


Не, так не пойдет. Вы же уже ввели число, а не строку... Сравнивайте на диапазон.

Не подскажете как?