Почему не работает код (Не бойтесь, 20 строк) на php?

Question

[Mansur05]

Точнее не делает то что я хочу.
Вот сам код

<?php
function abcx ($a, $b, $c) {
    $D = $b*$b-4*$a*$c;
    if ($D>0) {
        $D = sqrt($D);
        $x == ($b + $D)/2*a;
        $x2 == ($b - $D)/2*a;
        echo "Два корня, x1 = ", $x, " x2 = ", $x2;
    }else if ($D=0) {
        $x == $b/(2*$a);
        echo "Один корень, x =", $x;
    }else if ($D<0) {
        echo "Нет корней";
    }
}
echo(abcx($a=2, $b=-4, $c=3));
?>

Компилится, но результаты не выводит (корни уравнения)

Answer 1

[Алексей Уколов]

else if ($D=0)
Используйте сравнение (===), а не присваивание (=).

$x == $b/(2*$a);
Используйте присваивание (=), а не сравнение (==).

abcx($a=2, $b=-4, $c=3);
При такой записи вы создаётё вне функции переменные $a, $b и $c, а потом передаёте в неё их значения. Переменные при это продолжают болтаться снаружи и никак не связаны с одноименными локальными переменными функции. Это не ошибка, код будет работать, но действие бессмысленное и потенциально вредное. Вызывайте функцию как обычно: abcx(2, -4, 3);

Судя по количеству ошибок на такой маленькой площади, вам нужно вернуться к учебникам - вы не понимаете, что и зачем происходит в коде, а без этого понимания программировать бесполезно.

P.S. Что же вы лесенку-то не достроили? :)

$D = sqrt($D);
$x == ($b + $D)/2*a;
$x2 === ($b - $D)/2*a;

Comments

[Ульрих]

Невнимательность

[Алексей Уколов]

Ульрих: да нет, судя по коду, это не невнимательность, а непонимание на каком-то глубинном уровне.

[Mansur05]

Спасибо, да php практически не знаю (самый минимум) и использовал знания из других языков

Answer 2

[Александр Шаповал]

Почитайте как правильно составлять условия что-бы всегда находить ошибку: Условия_Йоды

Это не обязательно, сам грешу и редко использую этот принцип. Но тогда ошибки сами вылезут наружу.

И еще одно, постарайтесь разобраться чем сравнение '==' отличается от сравнения '==='. Жить в дальнейшем станет чуток легче.

Желаю творческих успехов! :)

Comments

[Алексей Уколов]

Условия Йоды ухудшают читаемость кода и с легкостью заменяются хорошей IDE, которая о таких проблемах сразу информирует.

[ellrion]

Алексей Уколов: "ухудшают читаемость кода" - это спорное субъективное утверждение. а вот польза в снижении ошибки объективна. И какая же это IDE информирует о присвоении в условных операторах? вот в йода стайле да проинформирует даже без запуска, а так с чего бы ей беспокоиться?)

[Алексей Уколов]

ellrion: PhpStorm, например. Беспокоиться ей с того, что это распространённая ошибка, которую имеет смысл отловить на ранних этапах.
Про "ухудшают" это не субъективное утверждение, иначе а) все бы изначально так и писали и б) для такого стиля не придумывали бы специальное название, которое подчёркивает его неествественность. Оно "спорное" в том смысле, что кому-то всё-равно, а кому-то ухудшают. Ни разу не видел, чтобы кто-то сказал "так читабельнее".

[Александр Шаповал]

Алексей Уколов: использую PhpStorm, не припомню случая когда она уберегла меня от присвоения в условном операторе или от не читаемого кода Йоды. Может быть потому что я не до конца разобрался в её настройках. Соглашусь что ухудшается читаемость, однако если весь код построен не на условных операторах - это капля в море

[Алексей Уколов]

Александр Шаповал: код, который весь построен на условных операторах имеет огромную цикломатическую сложность и требует рефакторинга.
Что касается PhpStorm: Settings -> Editor -> Inspections -> PHP -> Probable Bugs -> Assignment in condition.

Using assignments in conditional expressions may cause errors that are hard to detect and is therefore considered bad programming practice.

[Алексей Уколов]

В общем, для меня условия Йоды являются явным признаком плохо пахнущего кода: либо они используются без необходимости просто потому что, либо код можно и нужно отрефакторить так, чтобы их использование не требовалось.

[Александр Шаповал]

Алексей Уколов: Да, подсветило ошибку. Благодарю за помощь

Answer 3

[Barmunk]

Выше написали, что в коде очень много ошибок, чтобы их понять прочтите это:

php.net/manual/ru/language.operators.php
php.net/manual/ru/functions.returning-values.php

$x = ($b + $D)/2*a;

Смущает a, у вас это не переменная, поэтому по типам она приведется в булев тип и станет 1 или 0
поэтому это тоже стоит почитать php.net/manual/ru/language.types.php

Answer 4

[Сергей Зеленский]

<?php
function abcx ($a, $b, $c) {
    $D = $b*$b-4*$a*$c;
    if ($D > 0) {
        $D = sqrt($D);
        $x == ($b + $D)/2*a;
        $x2 == ($b - $D)/2*a;
        return "Два корня, x1 = ", $x, " x2 = ", $x2;
    }else if ($D == 0) {
        $x == $b/(2*$a);
        return "Один корень, x =", $x;
    }else if ($D<0) {
        return "Нет корней";
    }
}
echo abcx($a=2, $b=-4, $c=3);
?>