Как вызвать функцию внутри функции?

Question

[Tizi]

как вызвать функцию goUser которая находится в функции start_game ?

Скрин

код

$(document).ready(function() {
	////////
	// переменные
	////////

	var startGame = 0;

	////////
	// События 
	////////

	$('.btn_start').on('click', start_game );
	$(document).on('keydown', goUser);

	////////
	// Начало игры 
	////////

	function start_game(event) {
		
		if( $('.user_name').val() != '') {

			startGame = 1;

			$('.start_game_box').addClass('hidden');
			$('.user').removeClass('hidden');

			if (startGame = 1) {
				function goUser(event) {

					///////
					// Определяем текущие коардинаты персонажа
					///////

					var pz_user = $('.user').offset(),
					top = pz_user['top'],
					left = pz_user['left'];

					///////
					// Вверх
					///////

					if (event.which === 38) {
						var top = top -5;
						$('.user').css('top', top + 'px')
					}

					///////
					// Вниз
					///////

					if (event.which === 40) {
						var top = top +5;
						$('.user').css('top', top + 'px')
					}

					///////
					// В лево
					///////

					if (event.which === 37) {
						var left = left -5;
						$('.user').css('left', left + 'px')
					}

					///////
					// В право
					///////

					if (event.which === 39) {
						var left = left +5;
						$('.user').css('left', left + 'px')
					}
				}
			}

		} else {
			$('.errors').html('<p class="errors_p">Введите имя!</p>');
		}
	}
});

P.s: Вообще правильным решением будет вызывать ее так /?

Comments

[JimmDiGreez]

on('keydown', goUser) сделай внутри start_game, а не вне его. (например, первой строчкой после if (startGame == 1), можно даже имя функции не давать, прямо так впихнуть, если нигде больше не пригодится.
А еще лучше goUser вообще в start_game не засовывать. Если только это не объект планируется быть. А если так - то название не говорящее и тд и тд. В общем правильная организация кода от кривых костылей тебя спасет, а сейчас выглядит не логично.

[Tizi]

JimmDiGreez, Принцип понял, я вот переменную меняю
0 = игра не запущена
1 = игра начата

как я буду обращаться к переменной если я из функции start_game вынесу goUser

[JimmDiGreez]

Tizi, Посмотри в сторону объектов. (В твоем случае его назвать можно Game). Сделай ему метод goUser, сделай ему метод start, сделай ему поле isGameStarted и развлекайся с логически более чистым кодом.
Если же делать бех объекта - то по сути тоже самое, но isGameStarted (твоя старая startGame) может в глоабльном скопе тогда находиться, а это не кошерно в целом, но может ничем тебе и не помешать.
Если же продолжить работу с текущей функцией, то привязку обработчиков так же вноси в нее, как я и говорил.
Во всех трех случаях у тебя будет доступно то, что тебе нужно. Вопрос в читаемости, как по мне.

[Tizi]

JimmDiGreez, нарыл в интернете вот это по объектам. это ?/ мне просто останется переделать это под себя?

P.s: я в jq еще слабоват, сори если глупый вопрос.

[JimmDiGreez]

Tizi, это относится к javascript, а не к jquery.
по каким-то причинам все ссылки на learn.javascript.ru, где неплохая инфа, у меня перестали открывать, поэтмоу скидываю что следующее нашел.
https://developer.mozilla.org/ru/docs/Web/JavaScri...

[Tizi]

JimmDiGreez,

Код с объектом

$(document).ready(function() {
	////////
	// Объекты
	////////

	var game = new Object(),
	gameStarted = 0;

	////////
	// События 
	////////

	$('.btn_start').on('click', start_game );

	////////
	// Начало игры 
	////////

	function start_game(event) {
		
		if( $('.user_name').val() != '') {

			game.gameStarted = 1;

			$('.start_game_box').addClass('hidden');
			$('.user').removeClass('hidden');

		} else {
			$('.errors').html('<p class="errors_p">Введите имя!</p>');
		}
	}

	////////
	// Перемещения
	////////

	if (game.gameStarted = 1) {

		$(document).keydown( function(event) {

			///////
			// Определяем текущие коардинаты персонажа
			///////

			var pz_user = $('.user').offset(),
			top = pz_user['top'],
			left = pz_user['left'];

			///////
			// Вверх
			///////

			if (event.which === 38) {
				var top = top -5;
				$('.user').css('top', top + 'px')
			}

			///////
			// Вниз
			///////

			if (event.which === 40) {
				var top = top +5;
				$('.user').css('top', top + 'px')
			}

			///////
			// В лево
			///////

			if (event.which === 37) {
				var left = left -5;
				$('.user').css('left', left + 'px')
			}

			///////
			// В право
			///////

			if (event.which === 39) {
				var left = left +5;
				$('.user').css('left', left + 'px')
			}
		});
	}
});

а объект вызывать теперь можно в любой функции ?? даже если они вложенные ??

[JimmDiGreez]

Tizi, помоему у тебя неправильно получилось. Сейчас набросаю простенько, и в ответ напишу (првоерять не буду конечно же, это на тебе).

Answer 1

[JimmDiGreez]

Очень вероятны ошибки с this, давно очень на js не писал.

$(document).ready(function() {
  var Game = {
    isGameStarted: false,
    start: function(event) {
      // для этого метода нужна блокировка, но и так сойдет пока.
      if (true == this.isGameStarted) {
        return;
      }

      if ($('.user_name').val() == '') {
        $('.errors').html('<p class="errors_p">Введите имя!</p>');
        return;
      }

      this.isGameStarted = true;

      $('.start_game_box').addClass('hidden');
      $('.user').removeClass('hidden');

      $(document).keydown(this.goUser);
    },
    goUser: function(event) {
      if (false == this.isGameStarted) {
        return;
      }

      var pz_user = $('.user').offset(),
      top = pz_user['top'],
      left = pz_user['left'];

      if (event.which === 38) {
        var top = top -5;
        $('.user').css('top', top + 'px')
      }

      if (event.which === 40) {
        var top = top +5;
        $('.user').css('top', top + 'px')
      }

      if (event.which === 37) {
        var left = left -5;
        $('.user').css('left', left + 'px')
      }

      if (event.which === 39) {
        var left = left +5;
        $('.user').css('left', left + 'px')
      }
    }
  };

  $('.btn_start').on('click', Game.start);
});

Comments

[Tizi]

Ваше предложение более читаемое и правильное по логической части. но у меня проблема возникла с использованием setinterval

код

$(document).ready(function() {
	var Game = {
		////////
		// Переменные
		////////
		isGameStarted: false,
		////////
		// Начало игры
		////////
		start: function(event) {
			// для этого метода нужна блокировка, но и так сойдет пока.
			if (true == this.isGameStarted) {
				return;
			}
			//////
			// Если поле с именеп пусто то мы выводим ошибку и заканчиваем выполнение кода
			//////
			if ( $('.user_name').val() == '' ) {
				$('.errors').html('<p class="errors_p">Введите имя!</p>');
				return;
			}
			//////
			// присваиваем переменной isGameStarted значение true
			//////
			this.isGameStarted = true;
			//////
			// Скрываем блок с вводом имени и показываем персонажа
			//////
			$('.start_game_box').addClass('hidden');
			$('.user').removeClass('hidden');
			//////
			// добовляем события при нажатии на кнопки которая ображается к функции goUser
			//////
			$(document).keydown(Game.goUser);
		},
		///////
		// Добавление 
		///////
		addMobs: function() {
			setinterval(function() {
				var win_width = $('body').width() + 100,
				win_height = $('body').height(),
				height = Math.random() * win_height,
				mob = $('</div>');
				mob.attr({
					'class': 'mob'
				});
				$('body').append(mob);
			}, 1000);
		},
		///////
		// передвижение
		///////
		goUser: function(event) {
			if (false == this.isGameStarted) {
				return;
			}

			var pz_user = $('.user').offset(),
			top = pz_user['top'],
			left = pz_user['left'];

			if (event.which === 38) {
				var top = top -5;
				$('.user').css('top', top + 'px')
			}

			if (event.which === 40) {
				var top = top +5;
				$('.user').css('top', top + 'px')
			}

			if (event.which === 37) {
				var left = left -5;
				$('.user').css('left', left + 'px')
			}

			if (event.which === 39) {
				var left = left +5;
				$('.user').css('left', left + 'px')
			}
		}
	};
	////////
	// События
	////////
	$('.btn_start').on('click', Game.start);
});

взгляните на addMobs и скажите что там не так, опять проблема во вложеных функциях как я понял ??

[JimmDiGreez]

Tizi, вроде нет. а что именно не так работает?

[Tizi]

JimmDiGreez,

я разобрался вроде. работает как нужно.

только вот вопрос это же переменная ??


если да то я пробовал создавать ей подобные только с текстом или цифрами. то код вообще вырубается

[JimmDiGreez]

Tizi, покажи как пробовал создавать. переменная, поле объекта, все ок должно быть. мб синтаксически гдето ошибся.

[Tizi]

JimmDiGreez,
Магическим образом все заработало :D

Answer 2

[Айнур Валиев]

мог бы и код скинуть. где вызов goUser то происходит?

Comments

[Tizi]

Прикрепил код, я даже чет и не подумал что его проще прикрепить )))

[Айнур Валиев]

Tizi, goUser не сработает пока не будет клика на btn_start. Вынеси goUser за пределы start_game

[Tizi]

Айнур Валиев, это я знаю. вопрос собственно в этом и заключался. как вызвать ее так. подсказали про обекты js. Сижу гуглю