Нужно найти ошибку

@kasadi · Регистрация: 13.01.2014

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

Мой код

C++

#include <iostream>
#include <cmath>
using namespace std;
double ugol(int a, int n, double s) {
    s = (n*a*a) / (4 * (tan(360 / 2 * n)));
    return s;
}
int main() {
    int A, N;
    double S;
    cout << "Enter numbers of side \n";
    cin >> N;
    cout << "Enter length of the sides \n";
    cin >> A;
    cout << "Area of a regular polegon: " << ugol(A, N, S) << "\n";
    return 0;
}

Ошибка

Ошибка 1 error C4700: использована неинициализированная локальная переменная "S" g:\users\adilbek\documents\visual studio 2013\projects\consoleapplication47\consoleapplication47\исходный код.cpp 15 1 ConsoleApplication47

Добавлено через 17 секунд
Вроде все правильно, где моя ошибка?

@UnsKneD · 17.02.2014, 13:08

S не инициализирована, компилятор вам тоже самое говорит, если хотите инициализировать её в функции передавайте через указатель.

@kasadi · 17.02.2014, 13:19 **[ТС]**

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

S не инициализирована, компилятор вам тоже самое говорит, если хотите инициализировать её в функции передавайте через указатель.

А можете показать как именно?

@UnsKneD · 17.02.2014, 13:23

kasadi, перепутал, по ссылке надо передавать.

C++

#include <iostream>
#include <cmath>
using namespace std;
 
double ugol(int a, int n, double &s) {
    s = (n*a*a) / (4 * (tan(360 / 2 * n)));
    return s;
}
int main() {
    int A, N;
    double S;
    cout << "Enter numbers of side \n";
    cin >> N;
    cout << "Enter length of the sides \n";
    cin >> A;
    cout << "Area of a regular polegon: " << ugol(A, N, S) << "\n";
    return 0;
}

@chedman · 17.02.2014, 13:24

C++

#include <iostream>
#include <cmath>
using namespace std;
double ugol(double a, double n) {
    return  (n*a*a) / (4. * (tan(360. / 2. * n)));
 
}
int main() {
    double A, N;
 
    cout << "Enter numbers of side \n";
    cin >> N;
    cout << "Enter length of the sides \n";
    cin >> A;
    cout << "Area of a regular polegon: " << ugol(A, N) << "\n";
    return 0;
}

@Бендерродригез · 17.02.2014, 13:27

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

Мой код

Любопытства ради: а на фига и возвращать результат и по ссылке изменять?

@kasadi · 17.02.2014, 13:29 **[ТС]**

C++

#include <iostream>
#include <cmath>
using namespace std;
double ugol(int a, int n) {
    return (n*a*a) / (4 * (tan(360 / 2 * n)));;
}
int main() {
    int A, N;
    cout << "Enter numbers of side \n";
    cin >> N;
    cout << "Enter length of the sides \n";
    cin >> A;
    cout << "Area of a regular polegon: " << ugol(A, N) << "\n";
    return 0;
}

Не стал мучатся и написал вот так.
Но без толку, ответ выходит не правильный. Я правильно написал тангенс? Там должно было быть типо (360 градусов разделенная на 2*n)

@UnsKneD · 17.02.2014, 13:33

kasadi, что считаете то?

@Бендерродригез · 17.02.2014, 13:35

Если что, tan() работает с радианами, а не градусами.

@kasadi · 17.02.2014, 13:35 **[ТС]**

Ладно, выложу формулу, по которой и делалась программа.
http://www.fxyz.ru/%D1%84%D0%B... %BA%D0%B0/

@chedman · 17.02.2014, 13:38

360 замени на типа 3.14159*2 , двойки в дроби можно сократить
В проге используй тип double

@kasadi · 17.02.2014, 13:57 **[ТС]**

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

360 замени на типа 3.14159*2 , двойки в дроби можно сократить
В проге используй тип double

Ну 360 градусов - это 2 ПИ. Это значит что 360 - это 6.28. НИ 3.14 ни 6.28 не дали нужных результатов.

Добавлено через 11 минут
Ладно, жду Ваших советов.

@chedman · 17.02.2014, 13:59

У меня вот это работает

C++

#include <iostream>
#include <cmath>
using namespace std;
double ugol(double a, double n)
{
    return  (n*a*a) / (4. * (tan(M_PI / n)));
}
int main()
{
    double A, N;
 
    cout << "Enter numbers of side \n";
    cin >> N;
    cout << "Enter length of the sides \n";
    cin >> A;
    cout << "Area of a regular polegon: " << ugol(A, N) << "\n";
    return 0;
}

Покажи свой тестовый пример.

alsav22 · 17.02.2014, 13:59

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

НИ 3.14 ни 6.28 не дали нужных результатов.

Нужных - это каких? Пример ввода, и что должно получиться.
Это сделали?

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

В проге используй тип double

@kasadi · 17.02.2014, 14:19 **[ТС]**

C++

#include <iostream>
#include <cmath>
using namespace std;
double ugol(int a, int n) {
    return (n*a*a) / (4 * (tan(360 / 2 * n)));;
}
int main() {
    int A, N;
    cout << "Enter numbers of side \n";
    cin >> N;
    cout << "Enter length of the sides \n";
    cin >> A;
    cout << "Area of a regular polegon: " << ugol(A, N) << "\n";
    return 0;
}

Нужных-это чтобы программа высчитала площадь многоугольника.

Добавлено через 1 минуту

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

У меня вот это работает

C++

#include <iostream>
#include <cmath>
using namespace std;
double ugol(double a, double n)
{
    return  (n*a*a) / (4. * (tan(M_PI / n)));
}
int main()
{
    double A, N;
 
    cout << "Enter numbers of side \n";
    cin >> N;
    cout << "Enter length of the sides \n";
    cin >> A;
    cout << "Area of a regular polegon: " << ugol(A, N) << "\n";
    return 0;
}

Покажи свой тестовый пример.

А что за M_PI? и почему Вы его не объявили?

@chedman · 17.02.2014, 14:22

Ввел 4 и 4 получил 16, что верно для квадрата со стороной 4.

Добавлено через 2 минуты
M_PI - это Пи определено в заголовочном файле cmath моего компилятора mingw.

@kasadi · 17.02.2014, 14:43 **[ТС]**

Короче намучался, но получилось...

C++

#define M_PI 3.1415926535897932384626433832795
#include <iostream>
#include <cmath>
using namespace std;
double ugol(int a, int n) {
    return (n*a*a) / (4 * (tan(M_PI /n)));;
}
int main() {
    int A, N;
    cout << "Enter numbers of side \n";
    cin >> N;
    cout << "Enter length of the sides \n";
    cin >> A;
    cout << "Area of a regular polegon: " << ugol(A, N) << "\n";
    return 0;
}

@kasadi 0 / 0 / 0 Регистрация: 13.01.2014 Сообщений: 65
	17.02.2014, 13:35 [ТС]	10
	Ладно, выложу формулу, по которой и делалась программа. http://www.fxyz.ru/%D1%84%D0%B... %BA%D0%B0/ 0

@kasadi 0 / 0 / 0 Регистрация: 13.01.2014 Сообщений: 65
	17.02.2014, 13:57 [ТС]	12
	Сообщение от chedman 360 замени на типа 3.141592 , двойки в дроби можно сократить В проге используй тип double Ну 360 градусов - это 2 ПИ. Это значит что 360 - это 6.28. НИ 3.14 ни 6.28 не дали нужных результатов. Добавлено через 11 минут* Ладно, жду Ваших советов. 0

@UnsKneD алкокодер 157 / 153 / 41 Регистрация: 27.12.2012 Сообщений: 550
	17.02.2014, 13:08	2
	S не инициализирована, компилятор вам тоже самое говорит, если хотите инициализировать её в функции передавайте через указатель. 0

@kasadi 0 / 0 / 0 Регистрация: 13.01.2014 Сообщений: 65
	17.02.2014, 13:19 [ТС]	3
	Сообщение от UnsKneD S не инициализирована, компилятор вам тоже самое говорит, если хотите инициализировать её в функции передавайте через указатель. А можете показать как именно? 0

@Бендерродригез Сгибальщик 42 / 42 / 4 Регистрация: 18.05.2013 Сообщений: 220
	17.02.2014, 13:27	6
	Сообщение от kasadi Мой код Любопытства ради: а на фига и возвращать результат и по ссылке изменять? 0

@UnsKneD алкокодер 157 / 153 / 41 Регистрация: 27.12.2012 Сообщений: 550
	17.02.2014, 13:33	8
	kasadi, что считаете то? 0

@Бендерродригез Сгибальщик 42 / 42 / 4 Регистрация: 18.05.2013 Сообщений: 220
	17.02.2014, 13:35	9
	Если что, tan() работает с радианами, а не градусами. 0

@chedman 81 / 80 / 9 Регистрация: 30.10.2013 Сообщений: 251
	17.02.2014, 13:38	11
	360 замени на типа 3.14159*2 , двойки в дроби можно сократить В проге используй тип double 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	17.02.2014, 13:59	14
	Сообщение от kasadi НИ 3.14 ни 6.28 не дали нужных результатов. Нужных - это каких? Пример ввода, и что должно получиться. Это сделали? Сообщение от chedman В проге используй тип double 0

@chedman 81 / 80 / 9 Регистрация: 30.10.2013 Сообщений: 251
	17.02.2014, 14:22	16
	Ввел 4 и 4 получил 16, что верно для квадрата со стороной 4. Добавлено через 2 минуты M_PI - это Пи определено в заголовочном файле cmath моего компилятора mingw. 1