Ошибка в коде Character Controller Unity 2D, что делать?

Question

[WalloutDay]

Написал код для управления персонажем на android, всё работает. Но вот проблемка(
1-при зажатии кнопка (gui imagine) работает только как один клик и что бы персонаж шёл дальше, нужно снова кликать. Как сделать или что изменить в скрипте что бы при зажимании кнопки он шёл до того момента пока я её не отпущу?
2-Анимация персонажа. Когда персонаж стоит проигрывается анимация idle (т.к. переменная "Speed" в окне аниматор стоит на "0.0"). Как только он начинает двигаться анимация меняется на walk (т.к. переменная "Speed" в окне аниматор переходит на "1.0"), НО как только кнопка перестает действовать, т.е. мы её отпускаем (но пока что она сама отпускается), то переменная не меняется. Как было 1.0 так и осталась, только иногда через пару секунд возвращается на 0.0. Помогите что делать?
Вот скрипт.

using UnityEngine;
using System.Collections;
using UnityStandardAssets.CrossPlatformInput;

public class CharacterController2 : MonoBehaviour {
	
	public float maxSpeed = 10f;
	bool facingRight = true;
	
	Animator anim;
	// Use this for initialization
	void Start () {
		anim = GetComponent<Animator> ();

	}
	
	// Update is called once per frame
	void FixedUpdate () {

	}

	public void Walk(int InputAxis)
	{
		float move = Input.GetAxisRaw ("Horizontal");

		move = InputAxis;
		
		anim.SetFloat ("Speed", Mathf.Abs (move));
		
		GetComponent<Rigidbody2D>().velocity = new Vector2 (move * maxSpeed, GetComponent<Rigidbody2D>().velocity.y);

		if (move > 0 && !facingRight) 
			Flip ();
		else if (move < 0 && facingRight)
			Flip ();

	}
	
	void Flip() {
		facingRight = !facingRight;
		Vector3 theScale = transform.localScale;
		theScale.x *= -1;
		transform.localScale = theScale;
	}
}

Answer 1

[MrDywar Pichugin]

Руководство Unity

1) Вам нужно почитать статью Unity3D Best Practices
(не запрашивайте объекты при каждом нажатии/кадре, кэшируйте их в полях класса; двигать объект с rigidbody нужно только в FixedUpdate и т.д.)

1.1) Что бы кэшировать элементы текущего объекта используйте Awaik() { ... }. Что бы обратиться к другим объектам используйте Start() { ... /* другие объекты уже будут созданы, поэтому вы их тут точно найдете */ }.

2) Что бы персонаж шел постоянно при зажатой кнопке, сделайте флаг (true/false). Нажали кнопку, через Event Trigger вызвали событие PointerDown (или любое другое в своем скрипте, все на ваше усмотрение), в нем флаг выставили в true, в FixedUpdate у нас условие if ( this._move ) { ... } персонаж пошел. Отжали кнопку PointerUp, флаг выставили в false, персонаж перестал идти. Вся суть в том что движение происходит только в FixedUpdate, вам нужно только переключать флаг включен или выключен из других методов/метода. На кнопку можно навесить любой скрипт и вызвать из него открытый метод.

3) По анимации не понял. Для состояния вкл/выкл лучше использовать bool, SetBool вроде, и это значит что значение нужно задавать в коде самому, оно не вернется в false пока вы сами этого не пожелаете (для таких целей используется триггер).

Пройдите бесплатный курс на MVA по разработке 2D и 3D игр.

Многие вещи я и сам регулярно забываю, приходится перечитывать постоянно т.к. использую редко.
Если вы серьезно решили заняться разработкой игр и вообще программированием то нужно учить English и еще один из популярных для развития мышления, https://www.busuu.com/enc/ бесплатно.
90% всех вопросов уже решены, нужно только уметь искать. Раньше тратил на это кучу времени, теперь относительно быстро решаются даже сложные вопросы.

Comments

[WalloutDay]

Извините, то есть мне придётся дописать код в FixedUpdate?

using UnityEngine;
using System.Collections;
using UnityStandardAssets.CrossPlatformInput;

public class CharacterController2 : MonoBehaviour {
	
	public float maxSpeed = 10f;
	public bool facingRight = true;
	public float move;
	public int InputAxis;

	Animator anim;
	// Use this for initialization
	void Start () {
		anim = GetComponent<Animator> ();

	}
	
	// Update is called once per frame
	void FixedUpdate () {


		move = InputAxis;
		
		GetComponent<Rigidbody2D>().velocity = new Vector2 (move * maxSpeed, GetComponent<Rigidbody2D>().velocity.y);
		
		if (move > 0 && !facingRight) 
			Flip ();
		else if (move < 0 && facingRight)
			Flip ();
	}

	void Flip() {
		facingRight = !facingRight;
		Vector3 theScale = transform.localScale;
		theScale.x *= -1;
		transform.localScale = theScale;
	}
}

а именно указать что, если персонаж начинает двигаться, то bool=true, и как только он остановился bool=false. В аниматоре я уже выставил значения bool и в ручном режиме всё работает. И ещё прописать код для клавиши или можно использовать getbutton?

[MrDywar Pichugin]

WalloutDay: Кнопка может вызывать этот скрипт (что выше указан), и например вызывать ваш открытый метод (который вы напишите), он будет переводить флаг в _canMove = true когда кнопка нажата, и в _canMove = false когда кнопка была отжата (другой метод). Тема самым вы имитируете состояние зажатой кнопки, хотя на самом деле это не так.

В FixedUpdate нужно только
if ( this._canMove )
{
move = InputAxis;
this._rigigBody2d.velocity = new Vector2 (move * this.maxSpeed, this._rigigBody2d.velocity.y);
// логика повотора
}

_rigigBody2d - ваш GetComponent() который нужно получить в методе

Awaike()
{
this._rigigBody2d = GetComponent();
}

private bool _canMove; - это закрытое поле класса.
private Rigidbody2D _rigigBody2d;

Пример по ссылке forum.unity3d.com/threads/touch-and-hold-a-button-...
Что то вменяемое написал пользователь под ником prefix.

[WalloutDay]

MrDywar Pichugin: То есть прописать вот это

public void Walk(int InputAxis)
{
BoolSpeed = true;
}

public void Idle(int InputAxis)
{
BoolSpeed = false;
}

и вставить под event trigger (pointer down и pointer exit) что бы при нажатии "true" и при отпускании "false"?
Если так, ещё вопрос, куда мне закинуть вот эти строки?

move = InputAxis;

GetComponent().velocity = new Vector2 (move * maxSpeed, GetComponent().velocity.y);

В принципе мне всё понятно, но если можете, объясните пожалуйста.

[WalloutDay]

MrDywar Pichugin: cпасибо вам! Всё получилось!!! При зажатии кнопки (она же gui imagine) анимация проигрывается и переключается только тогда когда я утпускаю палец. Но двигается он все равно только при нажатии, а потом останавливается, даже когда я держу( Помогите, это последнее препядствие перед полным управлением персонажем!)

[MrDywar Pichugin]

WalloutDay:
void FixedUpdate
{
if ( BoolSpeed )
{
move = InputAxis;
this._rigigBody2d.velocity = new Vector2 (move * this.maxSpeed, this._rigigBody2d.velocity.y);
// логика поворота Flip()
}
}

Суть в том что BoolSpeed = true только когда вы нажали на кнопку, как только срабатывает событие "кнопка была отпущена" BoolSpeed становится равен false. Следовательно условие if ( BoolSpeed ) не выполнится и персонаж перестанет двигаться. Но он будет двигаться все время пока кнопка была нажата, но еще не поступило событие что ее отпустили. Очень важно вызывать ваши методы Walk/Idle на правильных событиях. Вы меняете значение переключателя, а то за что он отвечает срабатывает автоматически, вы лишь включаете и выключаете его (что уже готово).

[WalloutDay]

MrDywar Pichugin:
Написал код. Но он как не ходил, так и не ходит( + перестал поворачиваться. вот код. boolspeed заключал и в скобки, но тоже самое.

using UnityEngine;
using System.Collections;
using UnityStandardAssets.CrossPlatformInput;

public class CharacterController2 : MonoBehaviour {
	
	public float maxSpeed = 10f;
	public bool facingRight = true;
	public float move;
	public int InputAxis;
	bool BoolSpeed;

	Animator anim;
	// Use this for initialization
	void Start () {
		anim = GetComponent<Animator> ();

	}
	
	// Update is called once per frame
	void FixedUpdate () {

		if (BoolSpeed){

		   move = InputAxis;
		   this.GetComponent<Rigidbody2D>().velocity = new Vector2 (move * maxSpeed, GetComponent<Rigidbody2D>().velocity.y);


		if (move > 0 && !facingRight) 
			Flip ();
		else if (move < 0 && facingRight) 
			Flip ();
		}
	}

    public void Walk(int InputAxis){

		anim.SetBool ("BoolSpeed", true);

		}

	public void Idle(int InputAxis){

		anim.SetBool ("BoolSpeed", false);

	}

	void Flip() {
		facingRight = !facingRight;
		Vector3 theScale = transform.localScale;
		theScale.x *= -1;
		transform.localScale = theScale;
	}
}

[WalloutDay]

MrDywar Pichugin: А можно ли сделать так? Но при этом пишет error CS1501: No overload for method `Move' takes `0' arguments. Просто в первом случае мне просто не к какой переменной я не могу прицепить к кнопке (типо public void Move(int InputAxis)) как во втором случае.

using UnityEngine;
using System.Collections;
using UnityStandardAssets.CrossPlatformInput;

public class CharacterController2 : MonoBehaviour {
	
	public float maxSpeed = 10f;
	public bool facingRight = true;
	public float move;
	public int InputAxis;
	bool BoolSpeed;

	Animator anim;
	// Use this for initialization
	void Start () {
		anim = GetComponent<Animator> ();

	}
	
	// Update is called once per frame
	void FixedUpdate () {

		if (BoolSpeed){

			Move();
			 
		if (move > 0 && !facingRight) 
			Flip ();
		else if (move < 0 && facingRight) 
			Flip ();
		}
	}

	public void Move(int InputAxis){

		move = InputAxis;
		this.GetComponent<Rigidbody2D>().velocity = new Vector2 (move * maxSpeed, GetComponent<Rigidbody2D>().velocity.y);

	}

    public void Walk(int InputAxis){

		anim.SetBool ("BoolSpeed", true);

		}

	public void Idle(int InputAxis){

		anim.SetBool ("BoolSpeed", false);

	}

	void Flip() {
		facingRight = !facingRight;
		Vector3 theScale = transform.localScale;
		theScale.x *= -1;
		transform.localScale = theScale;
	}
}

[MrDywar Pichugin]

WalloutDay:
bool BoolSpeed; - это поле класса, оно не имеет никакого отношения к переменной в Аниматоре (там своя переменная, возможно имя совпадает).

Все что вам осталось, это назначать BoolSpeed = true и false прямо в коде, сейчас у вас этого не вижу. У вас BoolSpeed всегда равен false, и никогда не изменяет своего значения.

public void ButtonPressed()
{
BoolSpeed = true;
}

public void ButtonRelease()
{
BoolSpeed = false;
}

И вызывать эти методы из кнопки, когда ее нажали и когда ее отпустили. И все сразу заработает.

[WalloutDay]

Если я вас правильно понял, то у меня ошибка( Я понимаю что скорее всего вас задолбал) Я написал BoolSpeed в void Walk и idle (true и false соответственно)
и указал в fixedupdate путь на move.

using UnityEngine;
using System.Collections;
using UnityStandardAssets.CrossPlatformInput;

public class CharacterController2 : MonoBehaviour {
	
	public float maxSpeed = 10f;
	public bool facingRight = true;
	public float move;
	public int InputAxis;
	bool BoolSpeed;

	Animator anim;
	// Use this for initialization
	void Start () {
		anim = GetComponent<Animator> ();

	}
	
	// Update is called once per frame
	void FixedUpdate () {

		if (BoolSpeed) {

			Move();

		}

		if (move > 0 && !facingRight) 
			Flip ();
		else if (move < 0 && facingRight) 
			Flip ();

	}

	public void Move(int InputAxis){

		move = InputAxis;
		this.GetComponent<Rigidbody2D>().velocity = new Vector2 (move * maxSpeed, GetComponent<Rigidbody2D>().velocity.y);

	}

    public void Walk(int InputAxis){

		anim.SetBool ("BoolSpeed", true);   
		//Включает анимацию ходьбы
		BoolSpeed = true; 
		}

	public void Idle(int InputAxis){

		anim.SetBool ("BoolSpeed", false);
		//Включает анимацию пакоя
		BoolSpeed = false;
	}

	void Flip() {
		facingRight = !facingRight;
		Vector3 theScale = transform.localScale;
		theScale.x *= -1;
		transform.localScale = theScale;
	}
}

[WalloutDay]

MrDywar Pichugin: или даже так.

using UnityEngine;
using System.Collections;
using UnityStandardAssets.CrossPlatformInput;

public class CharacterController2 : MonoBehaviour {

public float maxSpeed = 10f;
public bool facingRight = true;
public float move;
public int InputAxis;
bool BoolSpeed;

Animator anim;
// Use this for initialization
void Start () {
anim = GetComponent ();

}

// Update is called once per frame
void FixedUpdate () {

if (BoolSpeed) {
move = InputAxis;
this.GetComponent ().velocity = new Vector2 (move * maxSpeed, GetComponent ().velocity.y);
}

if (move > 0 && !facingRight)
Flip ();
else if (move < 0 && facingRight)
Flip ();

}

public void Walk(int InputAxis){

anim.SetBool ("BoolSpeed", true);
//Включает анимацию ходьбы
BoolSpeed = true;

}

public void Idle(int InputAxis){

anim.SetBool ("BoolSpeed", false);
//Включает анимацию пакоя
BoolSpeed = false;
}

void Flip() {
facingRight = !facingRight;
Vector3 theScale = transform.localScale;
theScale.x *= -1;
transform.localScale = theScale;
}
}

[MrDywar Pichugin]

WalloutDay: Да, теперь должно работать если вы в правильное время вызываете методы Walk/Idle.

Еще, как у вас anim = GetComponent(...) инициализируется в void Start(), так и RigidBody2D должен там быть. Сделайте по аналогии с anim, один раз запросите его в void Start() и все, потом уже используйте его как anim, не запрашивая каждый раз. Сейчас у вас 50 раз в секунду вы два раза запрашиваете у объекта один и тот же RigidBody2D, а это гораздо медленнее работает чем обращение к уже запрошенному 1 раз RigidBody2D.

public float maxSpeed = 10f;
public bool facingRight = true;
public float move;
public int InputAxis;
bool BoolSpeed;
RigidBody2D rigidbody; <- вот он, будет всегда доступен, быстро.

void Start () {
anim = GetComponent ();
rigidbody = GetComponent(); <- один раз получили, как анимацию.
}

[MrDywar Pichugin]

Если не работает, стучитесь в Skype покажу как лучше и где почитать/посмотреть.

[WalloutDay]

MrDywar Pichugin: хорошо, спасибо, буду разбираться.

[WalloutDay]

MrDywar Pichugin: В общем как не старался не получается( Т.к. персонаж у меня двигается только тогда когда я пишу это не в Fixed Update. А придаю им какую то переменную или что-то в этом роде. Что бы потом это действие вызвать в Event trigger. Вот картинка.

public void Move(int InputAxis){
	    
		directionInput = InputAxis;

		rigidbody.velocity = new Vector2(playerSpeed * directionInput, rigidbody.velocity.y);
	}

using UnityEngine;
using System.Collections;
using UnityStandardAssets.CrossPlatformInput;

public class CharacterController2 : MonoBehaviour {
	
	public float playerSpeed;
	public bool facingRight = true;
	public int directionInput;
	public int InputAxis;
	public Rigidbody2D rigidbody;
	bool BoolSpeed;

	Animator anim;
	// Use this for initialization
	void Start () {
		anim = GetComponent ();
		rigidbody = GetComponent();
	}
	
	// Update is called once per frame
	void FixedUpdate () {

		
		if (directionInput > 0 && !facingRight) 
			Flip ();
		else if (directionInput < 0 && facingRight) 
			Flip ();

	}


    public void Move(int InputAxis){
	    
		directionInput = InputAxis;

		rigidbody.velocity = new Vector2(playerSpeed * directionInput, rigidbody.velocity.y);
	}


    public void Walk(int InputAxis){

		anim.SetBool ("BoolSpeed", true);   
		//Включает анимацию ходьбы
		BoolSpeed = true; 

	}

	public void Idle(int InputAxis){

		anim.SetBool ("BoolSpeed", false);
		//Включает анимацию пакоя
		BoolSpeed = false;
	}

	void Flip() {
		facingRight = !facingRight;
		Vector3 theScale = transform.localScale;
		theScale.x *= -1;
		transform.localScale = theScale;
	}
}

Но если переместить содержимое из Move в Fixed Upate, то перестаёт работать передвижение. Т.к. нету переменной которую можно указать для кнопки. Что бы персонаж двигался.

Думал поможет способ вызывать Move из Fixed Update, но пишет вот что "Assets/Scripts/CharacterController2.cs(26,25): error CS1501: No overload for method `Move' takes `0' arguments"

using UnityEngine;
using System.Collections;
using UnityStandardAssets.CrossPlatformInput;

public class CharacterController2 : MonoBehaviour {
	
	public float playerSpeed;
	public bool facingRight = true;
	public int directionInput;
	public int InputAxis;
	public Rigidbody2D rigidbody;
	bool BoolSpeed;

	Animator anim;
	// Use this for initialization
	void Start () {
		anim = GetComponent<Animator> ();
		rigidbody = GetComponent<Rigidbody2D>();
	}
	
	// Update is called once per frame
	void FixedUpdate () {

		if (directionInput != 0) {
			Move();
		}

		if (directionInput > 0 && !facingRight) 
			Flip ();
		else if (directionInput < 0 && facingRight) 
			Flip ();

	}


    public void Move(int InputAxis){
	    
		directionInput = InputAxis;

		rigidbody.velocity = new Vector2(playerSpeed * directionInput, rigidbody.velocity.y);
	}


    public void Walk(int InputAxis){

		anim.SetBool ("BoolSpeed", true);   
		//Включает анимацию ходьбы
		BoolSpeed = true; 

	}

	public void Idle(int InputAxis){

		anim.SetBool ("BoolSpeed", false);
		//Включает анимацию пакоя
		BoolSpeed = false;
	}

	void Flip() {
		facingRight = !facingRight;
		Vector3 theScale = transform.localScale;
		theScale.x *= -1;
		transform.localScale = theScale;
	}
}

Что мне делать? Или вам по Scype будет удобнее мне объяснить?

[MrDywar Pichugin]

WalloutDay: Однозначно Skype быстрее, вечером.

[WalloutDay]

MrDywar Pichugin: хорошо

Answer 2

[Владимир Лимарченко]

Input.GetAxisRaw ("Horizontal");
Это надо делать в Update или FixedUpdate.

public void Walk(int InputAxis)
  {
    float move = Input.GetAxisRaw ("Horizontal");

    move = InputAxis;

Тут получается Вы Axis значение получили. а потом перекрыли его аргументом из метода InputAxis. Так Ваша переменная move будет всегда равна InputAxis.

Еще - в примере кода не видно где Вы вызываете метод Walk()
Без собственно вызова этого метода ничего не будет работать.

Я бы попробовал так:

void FixedUpdate () {
           Walk(Input.GetAxis("Horizontal"));
  }

А строку
float move = Input.GetAxisRaw ("Horizontal");
из метода Walk можно смело удалять, все равно вы потом move присваиваете значение InputAxis.