仅在 unity3d、c# 中发生的错误（使用图形）

Question

So, here is the problem:

I have a graph class (and vertex and edge) to set up a graph. However, in C# (in unity3d), whenever I am adding a new vertex to my vertexList, the previous vertexes that were added get rewritten with the new one.

Now, I was so confused, I even asked a university student and my computer science teacher, they were stomped, they said it themselves.

Then, I tried to see if I wrote something wrong, so I wrote the whole thing (just the parts that help make the vertexList) in Python. And it worked as it should have in c# too.

Below you can see the C# code (link):

Graph class - https://hastebin.com/usizigepaf.cs

Vertex class - https://hastebin.com/ofiyozodun.cs

Edge class - https://hastebin.com/riwazawaki.cpp (this should say cs for c#, I don't know why it says cpp, it's fine anyway)

And here is the python code:

All - https://hastebin.com/ukokujesal.rb

---

You might find more code in the c# codes, but that is because I want to set up edges too, however, I need to set up vertexes first... that is the main problem here that I want to focus on and need help with.

So, how do I solve the problem? What is going wrong?

---

EDIT:

The problem starts when I add to the vertexList, basically on line 17 of the graph class. That line calls the method which starts at line 45, therefore, the problem can only be, in my opinion, within the method (addVertex(data)).

Answer 1

The problem clearly lies in

public void addVertex(Transform data)
{
    Debug.Log("data: " + data);
    //Vertex vertex = new Vertex(data);

    Vertex vertex = GetComponent<Vertex>();
    vertex.setVertex(data);
    Debug.Log("vertex: " + vertex);

    vertexList[graphIndex] = vertex;
   // Debug.Log("Type of item at graphIndex " + graphIndex + ": " + vertexList[graphIndex]);
   // Debug.Log("Data within the item at graphIndex " + graphIndex + ": " + vertexList[graphIndex].getVertex());

   graphIndex++;
}

where GetComponent<Vertex>() always returns the same instance of Vertex (the first encountered) attached to this GameObject.

As you figured new is forbidden for MonoBehaviour classes.

---

Actually there is no reason why Vertex should be a MonoBehaviour at all. Simply make it

[System.Serializable] // <- makes this class visible in the Inspector
                      //    including its serialized fields
public class Vertex
{
    ...

    public Vertex(Transform dataP)
    {
        data = dataP;
    }

    ...
}

so you can use new

public class Graph : MonoBehaviour
{
    [SerializedField] private Vertex[] vertexList;

    ...

    public void addVertex(Transform data)
    {
        Debug.Log("data: " + data);

        var vertex = new Vertex(data);

        Debug.Log("vertex: " + vertex);

        vertexList[graphIndex] = vertex;
       // Debug.Log("Type of item at graphIndex " + graphIndex + ": " + vertexList[graphIndex]);
       // Debug.Log("Data within the item at graphIndex " + graphIndex + ": " + vertexList[graphIndex].getVertex());

        graphIndex++;
    }

    ...
}

---

You might also ant to consider to use a List<Vertex> instead of the array and index if you are going to add vertices dynamically.