Найти ошибки в задаче с перегрузками функций

@Gole_Zahra · Регистрация: 23.03.2016

Author24 — интернет-сервис помощи студентам

Помогите, пожалуйста, определить, где здесь ошибка

Никак не могу найти..
Буду очень благодарна!

C++

#include <iostream>
#include <math.h>
using namespace std;
 
float ploshad(int a,int b,int c)
{   float p=(a+b+c)/2.0;
return sqrt(p*(p-a)*(p-b)*(p-c));}
 
float ploshad(int a,int b)
{   return a*b;}
 
float ploshad(int a) 
{   return a*a;} 
 
int main() {
int n,a,b,c;
float s;
 cout<<"введите 1 для нахождения площади обычного треугольника, 2 - для прямоугольника, 3 - для квадрата: ";
cin>>n;
 switch (n)
 {
case 1: cout<<"Введите 3 стороны: ";
cin>>a>>b>>c;
s=ploshad(a,b,c);
break;
 case 2: cout<<"Введите длину и ширину прямоугольника: ";
cin>>a>>b;
s=ploshad(a,b);
break;
case 3: cout<<"Введите сторону квадрата: ";
cin>>a;
s=ploshad(a);
break;
}
 cout<<"Площадь равна "<<s<<"\n";
system("pause");
return 0; 
}

@skales007 · 25.05.2016, 07:50

Gole_Zahra, а в чем заключается ошибка?

@0x90h · 25.05.2016, 07:52

А зачем две отдельных функции для квадрата и прямоугольника? Квадрат является частным случаем прямоугольника.

Ваш switch не содержит ветви default. В ответ на приглашение можно ввести не только 1,2,3, а все, что угодно, например букву. Ввод пользователя всегда надо проверять.

В результате не будет выполнена не одно из ветвлений, а сразу произойдет вывод переменной s, которая ничем не инициализирована. Получите "мусор" на экран - случайное значение ячейки памяти.

@skales007 18 / 18 / 9 Регистрация: 09.07.2015 Сообщений: 91 Записей в блоге: 2
	25.05.2016, 07:50	2
	Gole_Zahra, а в чем заключается ошибка? 0

@0x90h 683 / 458 / 160 Регистрация: 01.10.2015 Сообщений: 1,264
	25.05.2016, 07:52	3
	А зачем две отдельных функции для квадрата и прямоугольника? Квадрат является частным случаем прямоугольника. Ваш switch не содержит ветви default. В ответ на приглашение можно ввести не только 1,2,3, а все, что угодно, например букву. Ввод пользователя всегда надо проверять. В результате не будет выполнена не одно из ветвлений, а сразу произойдет вывод переменной s, которая ничем не инициализирована. Получите "мусор" на экран - случайное значение ячейки памяти. 0