Ndtyjky

I've been working on this project for some days now and I've encountered a bug that seems impossible to solve because not only no error messages appear but it also 'skips' my debug messages and crashes the editor itself.

The following script is a dialog displayer, it's apparently what's causing the issue (forgive the messed code, i messed it around while trying to solve the problem):

using System.Collections;

using System.Collections.Generic;

using UnityEngine;

using UnityEngine.UI;

using TMPro;



public class DialogDisplayer : MonoBehaviour

{

    [SerializeField] Dialog dialogFiles;

    TextMeshPro outputTxt;



    bool next, finished;

    char comma = (char)44;

    char period = (char)46;



    // Use this for initialization

    void Start()

    {

        outputTxt = GetComponent<TextMeshPro>();

        StartCoroutine(type());

    }



    IEnumerator type()

    {

        int dialogIndex = 0;



        do

        {

            foreach (char c in dialogFiles[dialogIndex].dialogText)

            {

                if (Input.GetKeyDown(KeyCode.Z))

                {

                    outputTxt.text = dialogFiles[dialogIndex].dialogText;

                    Debug.Log("z pressed in the foreach");

                    break;

                }



                outputTxt.text += c;

                if (c == ' ')

                    continue;



                if (dialogFiles[dialogIndex].delayforPunctuations)

                {

                    if (c == comma)

                        yield return new WaitForSeconds(dialogFiles[dialogIndex].delayBetweenLetters + 0.1f);

                    else if (c == period)

                        yield return new WaitForSeconds(dialogFiles[dialogIndex].delayBetweenLetters + 0.2f);

                    else

                        yield return new WaitForSeconds(dialogFiles[dialogIndex].delayBetweenLetters);

                }

                else

                    yield return new WaitForSeconds(dialogFiles[dialogIndex].delayBetweenLetters);

            }

            Debug.Log("Either finished or broken out of the loop");



            while (!finished)

            {

                Debug.LogWarning("Entering while loop");

                if (Input.GetKeyDown(KeyCode.Z))

                {

                    Debug.Log("entered if");

                    finished = true;

                    dialogIndex++;

                }

                Debug.Log("got out");

            }



        } while (dialogIndex != dialogFiles.Length - 1);

    }

}

You do not use yield return anywhere in your Coroutine. This will cause your coroutine to run all iterations in the same frame.

From the Scripting documentation:

// every 2 seconds perform the print()

private IEnumerator WaitAndPrint(float waitTime)

{

    while (true)

    {

        yield return new WaitForSeconds(waitTime);

        print("WaitAndPrint " + Time.time);

    }

}

Or you can use yield return null to pass the coroutine onto the next frame:

private IEnumerator YourDialogCoroutine()

{

    bool finished = false;



    while (!finished)

    {

        If(Input.GetKeyDown("Z"))

        {

            Debug.Log("Ending loop!");

            finished = true;

        }



        yield return null;

    }

}

From what I see....

In English: As soon as dialogIndex is not equal to dialogFiles.length minus one your script crashes.

Once this:

while (dialogIndex != dialogFiles.Length - 1);

is true, it enters into a endless loop. There are no directions of what to do once dialogFiles.Length -1 no longer is equal to dialogIndex.

Not sure what you are trying to accomplish with the final line of code there, but neither is the computer and it just waits for instructions once those conditions are met....forever.

It needs to be

while (dialogIndex != dialogFiles.Length - 1); {

  Do Something;

  provide Action To Eventually escape this loop (make dialogIndex = dialogFiles.Length - 1)

}

As for as the comment goes, you can do StartCoroutine(type()); or StartCoroutine("type"); That's fine.