Форум программистов, компьютерный форум CyberForum.ru

Найдите ошибку - C++

Восстановить пароль Регистрация
 
Кооошечка
 Аватар для Кооошечка
0 / 0 / 0
Регистрация: 02.03.2010
Сообщений: 21
27.09.2010, 16:34     Найдите ошибку #1
Даны действительные числа x1, x2, x3, y1, y2, y3. Принадлежит ли начало координат треугольнику с вершинами (x1, y1), (x2, y2), (x3, y3)?
Проверьте пожалуйста код...какие б значения я не вводила всегда пишет принадлежит, не могу найти ошибку
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
#include <iostream>
int main()
{
    float x1,x2,x3,y1,y2,y3,s1,s2,s3,S;
    std::cout << "x1=";
    std::cin >> x1;
    std::cout << "y1=";
    std::cin >> y1;
    std::cout << "x2=";
    std::cin >> x2;
    std::cout << "y2=";
    std::cin >> y2;
    std::cout << "x3=";
    std::cin >> x3;
    std::cout << "y3=";
    std::cin >> y3;
    s1 =abs(((y2)*abs(x2)+(y3+y2)*abs(x3-x2)-(y3)*abs(x3))/2);
    s2 =abs(((y1)*abs(x1)+(y3)*abs(x3)-(y1+y3)*abs(x3-x1))/2);
    s3 =abs(((y1+y2)*abs(x2-x1)+(y2)*abs(x2)-(y1)*abs(x1))/2);
    S  =abs(((y1+y2)*abs(x2-x1)+(y3+y2)*abs(x3-x2)-(y1+y3)*abs(x3-x1))/2);
    std::cout<<"s1="<<s1<<" s2="<<s2<<" s3="<<s3<<" S="<<S;
    if ((S-s1-s2-s3)<0.00001) std::cout<<"prinadlejit";
    else std::cout<<"ne prinadlejit"<<"\n";
    system("pause");
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
27.09.2010, 16:34     Найдите ошибку
Посмотрите здесь:

C++ Найдите ошибку!
C++ Найдите ошибку
Найдите ошибку C++
Найдите ошибку C++
C++ Найдите ошибку!
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Dexter
 Аватар для Dexter
284 / 144 / 16
Регистрация: 13.10.2009
Сообщений: 164
27.09.2010, 17:16     Найдите ошибку #2
Немного не понимаю что принимают значения s1,s2,s3 и S, но вот вкратце что может помочь:
1. Строка 22: if ((S-s1-s2-s3)<0.00001) - если значение будет отрицательное, то будет правда, как понимаю там порог для ошибки, потому надо брать его по модулю.
2. abs - возвращает абсолютное значение для 32битного int, потому нужно использовать fabs(), или преобразовать явно в флоат.

Если я правильно понял принцип, то s1,s2,s3 - площади треугольников в которых две вершины из нашего треугольника и третья 0, а S - общая площадь треугольника.
Если это так, то проще было бы писать так:

Код
s1 =fabs(x2*y3-x3*y2)/2;
s2 =fabs(x1*y3-x3*y1)/2;
s3 =fabs(x2*y1-x1*y2)/2;
S  =fabs(x1*(y2-y3)+x2*(y3-y1)+x3*(y1-y2))/2;
Кооошечка
 Аватар для Кооошечка
0 / 0 / 0
Регистрация: 02.03.2010
Сообщений: 21
27.09.2010, 17:39  [ТС]     Найдите ошибку #3
Цитата Сообщение от DexterUa Посмотреть сообщение
Немного не понимаю что принимают значения s1,s2,s3 и S, но вот вкратце что может помочь:
1. Строка 22: if ((S-s1-s2-s3)<0.00001) - если значение будет отрицательное, то будет правда, как понимаю там порог для ошибки, потому надо брать его по модулю.
2. abs - возвращает абсолютное значение для 32битного int, потому нужно использовать fabs(), или преобразовать явно в флоат.

Если я правильно понял принцип, то s1,s2,s3 - площади треугольников в которых две вершины из нашего треугольника и третья 0, а S - общая площадь треугольника.
Если это так, то проще было бы писать так:

Код
s1 =fabs(x2*y3-x3*y2)/2;
s2 =fabs(x1*y3-x3*y1)/2;
s3 =fabs(x2*y1-x1*y2)/2;
S  =fabs(x1*(y2-y3)+x2*(y3-y1)+x3*(y1-y2))/2;
Спасибо, дельный совет, но ни чего не поменялось, всё равно пишет принадлежит в любом случае
Dexter
 Аватар для Dexter
284 / 144 / 16
Регистрация: 13.10.2009
Сообщений: 164
27.09.2010, 17:41     Найдите ошибку #4
а поменяли ли вы if ((S-s1-s2-s3)<0.00001) на if (fabs(S-s1-s2-s3)<0.00001)?
Кооошечка
 Аватар для Кооошечка
0 / 0 / 0
Регистрация: 02.03.2010
Сообщений: 21
27.09.2010, 17:54  [ТС]     Найдите ошибку #5
Цитата Сообщение от DexterUa Посмотреть сообщение
а поменяли ли вы if ((S-s1-s2-s3)<0.00001) на if (fabs(S-s1-s2-s3)<0.00001)?
Огромнейшее спасибо я поменяла но не правильно написала написала abs)) теперь всё прекрасно работает. Очень Вам презнательна
Yandex
Объявления
27.09.2010, 17:54     Найдите ошибку
Ответ Создать тему
Опции темы

Текущее время: 17:36. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru