В чём ошибка условия if else в игре?

Question

[Игорь]

Лучше уменьшить размер экрана чтобы все полотно было видно.

По задумке:

1.Если объект не на камне то включается гравитация и он вниз падает.
на деле выходит что если нажать на пробел и прыгнуть то он зависает в воздухе а возле камня наоборот падает.

setInterval(function upDate(){                       ///ОБНОВЛЯЕМ ФУНКЦИИ И КОРДИНАТЫ
gravityIF=null;
    gamerCoord(); 	rockCoord();  //ОБНОВЛЯЕМ КООРДИНАТЫ КАМНЯ И ИГРОКА

    if (gamerObj.left>=rockObj.left && gamerObj.right<=rockObj.right ){   //ОБНАРУЖЕНИЕ СМЕШЕНИЯ ОБЪЕКТА под камнем
     
        console.log("пересекаемся"); gamer.style.top="-115px";

       return gravityIF=false;

     }
     else{        return gravityIF= true;       };

    gravityGamer(); //Вызываем функцию прыжка


},500);


function gravityGamer(){
	if((gamerObj.bottom<450)&&(gravityIF==true )){
		console.log("heyy");
		playerJump-=115;

		
		gamer.style.top="-115px";


		
	};
};

linewire.ru/t/Mario_game/Mario_game.html

Answer 1

[Дмитрий Беляев]

Бегло взглянув на код могу сказать, что эта строчкаgravityGamer(); //Вызываем функцию прыжканикогда не выполняется, так как в обоих ветках условия срабатывает return

Comments

[Игорь]

а как лучше оформить тогда?

[Дмитрий Беляев]

Игорь Николаевич: как минимум убрать в двух местах слово return, пользы от него никакой, а в Вашем случае еще и вред.
Как максимум, привести код к читаемому виду - ровные отступы, один оператор на строку, и тд
Вот такая конструкция бесполезна gravityIF==true полным аналогом будет просто проверка переменной в условии, а если нужно сравнить именно с true то нужен оператор ===
Половина пробелов пропущено, где-то в if подусловия взяты в скобки где-то нет, без видимой на то причины
Вообщем, приведите свой код в читаемый вид, тогда сами сможете видеть подобные ошибки