Sprite ändert sich nicht, wenn die Taste gedrückt wird - unity 2D

Question

Momentan versuche ich nur, die Sprites-Kerze von unbeleuchtet zu beleuchten, wenn der Spieler sowohl die Kerze als auch die Streichhölzer "abgeholt" hat und die Kerze erlischt. nach einer bestimmten Zeit. Wenn jedoch die Leertaste gedrückt wird, tritt der Übergang von unbeleuchtet zu erleuchtet nicht auf, obwohl das Debug-Protokoll True zurückgibt, wenn es sollte. Ich poste hier, um eine Anleitung zu bekommen, da ich den größten Teil des Tages damit verbracht habe, online zu gucken und buchstäblich keine Ahnung zu haben, wie ich vorgehen soll. Grundsätzlich sind die Bilder, zwischen denen ich zu wechseln versuche, zwei verschiedene Bilder, die sich im Sprites-Ordner unter Assets befinden.

Das ist, was ich bisher habe.

//the two sprites transition 
public Sprite unlitCandle; 
public Sprite litCandle; 

private SpriteRenderer spriteRenderer; 

bool pickUpMatches = false; 
bool pickUpCandle = false; 
float timeRemaining =5; 
bool candleLit = false; 

// Use this for initialization 
void Start() { 
    spriteRenderer = GetComponent<SpriteRenderer>(); 
    if (spriteRenderer.sprite == null) 
     spriteRenderer.sprite = unlitCandle; 

} 

// Update is called once per frame 

private void OnTriggerEnter2D(Collider2D collision) 
{ 
    if(collision.gameObject.CompareTag("Matches")) 
    { 
     collision.gameObject.SetActive(false); 
     pickUpMatches = true; 
    } 

    if (collision.gameObject.CompareTag("UnlitCandle")) 
    { 
     collision.gameObject.SetActive(true); 
     pickUpCandle = true; 
    } 

} 

public void CandleTimer() 
{ 
    if (candleLit == true) 
    { 
     timeRemaining = 5; 
     timeRemaining -= Time.deltaTime; 
     if (timeRemaining <= 0) 
     { 
      candleLit = false; 
      spriteRenderer.sprite = unlitCandle; 
     } 
    } 
} 

public void ChangeSprite() 
{ 
    if (spriteRenderer.sprite == unlitCandle) 
    { 
     spriteRenderer.sprite = litCandle; 
    } 
} 

void Update() { 


    if (pickUpCandle == true && pickUpMatches == true) 
    { 
     //Debug.Log(candleLit); 
     if (Input.GetKey(KeyCode.Space) && !candleLit) 
     { 
      CandleTimer(); 
      ChangeSprite(); 
      Debug.Log(timeRemaining); 
      candleLit = true; 
      //Debug.Log(candleLit); 
     } 
    } 
} 

}

Answer 1

Versuchen Sie es mit einem Verfahren wie equals Vergleich() anstelle von == in spriteRenderer.sprite == unlitCandle

Denn jetzt sind Sie nur zu vergleichen Referenzen und nicht die Objekte.

Zumindest denke ich, dass das das Problem ist.

Answer 2

Es gibt einige mögliche Probleme mit Ihrem Code. Zuerst rufen Sie changeSprite an der Spitze von Update, was bedeutet, dass es bedingungslos jeden Frame aufgerufen wird. Wenn ein einzelner Rahmen Ihrer Kerze nicht leuchtet, ändert sich das Sprite sofort in litCandle.

Ich nehme an, dass der Grund, dass Sie changeSprite jeden Rahmen aufrufen, ist, um den Timer zu verarbeiten, wenn Sie bereits eine brennende Kerze haben. Wirklich, Sie sollten den Code verschieben, um den Timer (Ihre ganze Sekunde if Anweisung in changeSprite) zu einer separaten Funktion zu verarbeiten und nennen Sie es so etwas wie processCandleTimer. Rufen Sie , dass an der Spitze der Update und speichern Sie die changeSprite Methode, nur auf den Tastendruck aufgerufen werden.

Schließlich, das Problem, das ich vermute, gibt Ihnen das größte Problem ist, dass Sie nicht Ihren Timer zurücksetzen, timeRemaining. Das erste Mal, wenn Sie die Kerze anzünden, wird der Timer nach 5 Sekunden auf 0 sinken. Jedes Mal, changeSprite danach ausgeführt wird, werden Sie das Sprit zu litCandle in der ersten if Anweisung ändern und dann es sofort Änderung unlitCandle zurück, weil der Timer 0 im zweiten ist. Um dies zu beheben, müssen Sie eine Zeile wie timeRemaining = 5.0f; hinzufügen, wenn der Schlüssel gedrückt wird.