List Removes Element For NO REASON

Scripting
1

This has just boggled my mind entirely. I’m working on a simple autofill function for my dev console. I have a list that is populated by command names that start with the current user input. Those get checked and are added to the autoFillQueue list of strings that gets iterated over when the tab key is pressed. This was working fine until I discovered this bug. When I type in a few letters, it populates my list then REMOVES EVERYTHING BUT ONE STRING. Please take a look at the comments I made in the code below so you understand exactly what is happening.

    void Update()
    {
        if (keyPressed && autoFillQueue.Count > 0) //keyPressed is calculated fine
        {
            print("Resetting Queue");

            autoFillQueue.Clear(); //I only reset the queue when a letter is changed and the queue is not empty. This is not the problem.

            keyPressed = false;
            autoFillIndex = 0; //Just a variable to keep track of the index. Works fine.
        }

        if (inputPlayer.GetButtonDown("AutoFill")) //Finds the button fine and all goes well
        {
            if (CommandList.GetCommandList() != null)
            {
                var input = userConsole.text;
                var commandListArray = CommandList.GetCommandList().ToArray();

                if (autoFillQueue.Count == 0) //Works fine here
                {
                    foreach (var command in commandListArray)
                    {
                        if (String.IsNullOrEmpty(input) || input.Trim() == "") //This check actually has no errors. If I do not input something, it all works fine.
                        {
                            autoFillQueue.Add(command.GetName());

                            continue;
                        }

                        if (command.GetName().StartsWith(input)) //THIS IS WHERE IT SEEMS TO SCREW UP
                        {
                            print(string.Format("Command Name: {0}", command.GetName())); //This logs that the only two commands I have all have passed and have been registered

                            autoFillQueue.Add(command.GetName());

                            print("Length: " + autoFillQueue.Count); //This logs the count AS 2.
                        }
                    }
                }

                //NOW AS IF IT WAS MAGIC, THE COUNT HAS SUDDENLY CHANGED TO 1. As logged with this print statement

                print(string.Format("AutoFillIndex: {0}, Length of List: {1}", autoFillIndex, autoFillQueue.Count));

                var commandName = autoFillQueue[autoFillIndex];

                UpdateTextBox(commandName);

                if (autoFillIndex == autoFillQueue.Count - 1)
                {
                    autoFillIndex = 0;
                }
                else
                {
                    autoFillIndex++;
                }
            }
        }
    }

If anyone can tell me what the hell is up with this, I would appreciate it.

2

Hmm, looked over that code… I’m not 100% convinced that the two print lines are from the same call to Update()…

There is no code between those two print statement that could remove an item from that list. As you say, it’s like magic…

In my experience, every time “magic” is happening the results I think I’m seeing are not the real results and there’s something else at play (or it’s multi-threaded and someone’s been silly, but I expect this is not multi-threaded).

Could you add a print statement after every line like:

void Update()
    {
        if (keyPressed && autoFillQueue.Count > 0) //keyPressed is calculated fine
        {
            print("Resetting Queue");
            print("** 1 **");
            autoFillQueue.Clear(); //I only reset the queue when a letter is changed and the queue is not empty. This is not the problem.
            print("** 2 **");
            keyPressed = false;
            print("** 3 **");
            autoFillIndex = 0; //Just a variable to keep track of the index. Works fine.
            print("** 4 **");
        }

        if (inputPlayer.GetButtonDown("AutoFill")) //Finds the button fine and all goes well
        {         
            print("** 5 **");

            if (CommandList.GetCommandList() != null)
            {
                print("** 6 **");
                var input = userConsole.text;
                print("** 7 **");
                var commandListArray = CommandList.GetCommandList().ToArray();
                print("** 8 **");
                if (autoFillQueue.Count == 0) //Works fine here
                {
                    print("** 9 **");
                    foreach (var command in commandListArray)
                    {
                        print("** 10 **");
                        if (String.IsNullOrEmpty(input) || input.Trim() == "") //This check actually has no errors. If I do not input something, it all works fine.
                        {
                            print("** 10.1 **");
                            autoFillQueue.Add(command.GetName());
                            print("** 10.2 **");
                            continue;
                        }

                        print("** 10.3 **");
                        if (command.GetName().StartsWith(input)) //THIS IS WHERE IT SEEMS TO SCREW UP
                        {
                            print("** 10.4 **");
                            print(string.Format("Command Name: {0}", command.GetName())); //This logs that the only two commands I have all have passed and have been registered
                            print("** 10.5 **");
                            autoFillQueue.Add(command.GetName());
                            print("** 10.6 **");
                            print("Length: " + autoFillQueue.Count); //This logs the count AS 2.
                            print("** 10.7 **");
                        }
                    }
                }
                print("** 11 **");
                //NOW AS IF IT WAS MAGIC, THE COUNT HAS SUDDENLY CHANGED TO 1. As logged with this print statement

                print(string.Format("AutoFillIndex: {0}, Length of List: {1}", autoFillIndex, autoFillQueue.Count));

                var commandName = autoFillQueue[autoFillIndex];
                print("** 12 **");
                UpdateTextBox(commandName);
                print("** 13 **");
                if (autoFillIndex == autoFillQueue.Count - 1)
                {
                    print("** 14 **");
                    autoFillIndex = 0;
                }
                else
                {
                    print("** 15 **");
                    autoFillIndex++;
                }
            }
        }
    }

and paste the output? I suspect we’re not going to see direct progression from 10.7 to 11 in this error case.

3

I did just as you said. And in doing so, I realized that I was setting keyPressed to try when I was entering the characters before attempting to autofill. This caused the variable to be true when the list was populated (not what I wanted) which resulted in it resetting itself. Little things like this are what keep me stuck on something for hours. Thanks though :stuck_out_tongue:

1 Like