0

Здравствуйте! В упор не могу понять, почему значение переменной s не соответствует ожидаемому?

#include <stdio.h>              
#include <locale.h>
#include <math.h>

int main(void)

{
    signed int a1, a2, b1, b2, c1, c2;
    double s;

    //получаем значения переменных из stdin

    s = 0.5 * ((a1 - c1)*(b2 - c2) - (b1 - c1)*(a2 - c2));
    if (s = 0.0) {
        //это условие никогда не выполняется — почему?
    }

    // дальше какой-то код

    return 0;
}

Спасибо.

5
  • Что значит мусор? Вещественные типы так не сравнивают. Тем более у тебя даже не сравнение (==), а присваивание. 15 сен 2015 в 13:26
  • @АнтонСазонов, нашёл ошибку, спасибо. забыл. 15 сен 2015 в 13:27
  • В будущем, пожалуйста, указывайте точно: ожидал ______, получил ______. Выражение «мусор в переменную» абсолютно непонятно, оттого и минусы.
    – Nick Volynkin
    15 сен 2015 в 13:58
  • @NickVolynkin, Хорошо, учту, спасибо. Минусы как-то переживу. 15 сен 2015 в 14:31
  • Кстати, а в чем смысл умножать число на 0,5, если мы его потом только сравниваем с нулем?
    – Nick Volynkin
    15 сен 2015 в 16:08

3 ответа 3

13

У Вас стандартная ошибка с присваиванием вместо сравнения

if (s = 0.0) {...}

а должно быть

if (s == 0.0) {...}

и как уже справедливо заметили, сравнивать вещественные так действительно нельзя

1
  • Вообще, нельзя, но в данном случае можно. Оно вообще половина целого.
    – Qwertiy
    15 сен 2015 в 14:43
8

Если хочешь сравнить с нулём, делай так:

if ( fabs( s ) <= eps ) ...

, где eps - какое-то очень маленькое число, точность сравнения.

7

Застраховать себя от таких ошибок чтобы, Условия Йоды использовать должны вы:

if (0 == s) ...

Числа с плавающей точкой сравнивать чтобы, используй эпсилон, молодой джедай.

0

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.