Логика PHP если статистика - PullRequest
0 голосов
/ 20 июля 2009

У меня есть этот код:

$distance= -1 ;//default
//distance calc
if($dgalaxy = $galaxy){
    if(($dx = $x) && ($dy = $y))
    {
            //inter planitary currently unknown formula
    }else{
            $distance = round(Sqrt(pow(($dx-$x),2)+pow(($dy-$y),2)));//interstllar
    }
}else{//intergalatic
    if ($galaxy < $dgalaxy){
            $distance = (((9-fmod($galaxy,10))*200)+2000+(fmod($dgalaxy,10)*200));//anti clockwise galactic
    }else{
            $distance = (((fmod($galaxy,10))*200)+2000+(fmod(9-$dgalaxy,10)*200));//clockwise galactic
    }
}

Должно возвращаться значение для каждого расчета, за исключением межпланетного предложения. для межзвездного (когда ($ dgalaxy = $ galaxy) и координаты x, y различаются) это запускает межзвездную формулу, и это прекрасно работает.

Но когда $ dgalaxy! = $ Galaxy, он возвращается с -1, что означает, что он не выполняет ни одного из вычислений галактического расстояния!

Что я сделал не так и как я могу это исправить?

Ответы [ 3 ]

14 голосов
/ 20 июля 2009

$dgalaxy = $galaxy - это назначение , а не сравнение . Используйте ==.

То же самое в другом месте.

4 голосов
/ 20 июля 2009

Вы используете = вместо == или ===:

$distance= -1 ;//default
                //distance calc
                // MISTAKE HERE. USE == INSTEAD of =
                if($dgalaxy == $galaxy){
                        if(($dx == $x) && ($dy == $y))
                        {
                                //inter planitary currently unknown formula
                        }else{
                                $distance = round(Sqrt(pow(($dx-$x),2)+pow(($dy-$y),2)));//interstllar
                        }
                }else{//intergalatic
                        if ($galaxy < $dgalaxy){
                                $distance = (((9-fmod($galaxy,10))*200)+2000+(fmod($dgalaxy,10)*200));//anti clockwise galactic
                        }else{
                                $distance = (((fmod($galaxy,10))*200)+2000+(fmod(9-$dgalaxy,10)*200));//clockwise galactic
                        }
                }
3 голосов
/ 20 июля 2009

Разве это не должно быть $ dgalaxy == $ galaxy? То есть проверка на равенство, а не присвоение?

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...