Procedural Mesh problem from Editor Window

Question

Here is the method to generate cube collected over internet:

     static void CreateCube()
     {
         Vector3[] vertices = {
             new Vector3 (0, 0, 0),
             new Vector3 (1, 0, 0),
             new Vector3 (1, 1, 0),
             new Vector3 (0, 1, 0),
             new Vector3 (0, 1, 1),
             new Vector3 (1, 1, 1),
             new Vector3 (1, 0, 1),
             new Vector3 (0, 0, 1),
         };
 
         int[] triangles = {
             0, 2, 1, //face front
             0, 3, 2,
             2, 3, 4, //face top
             2, 4, 5,
             1, 2, 5, //face right
             1, 5, 6,
             0, 7, 4, //face left
             0, 4, 3,
             5, 4, 7, //face back
             5, 7, 6,
             0, 6, 7, //face bottom
             0, 1, 6
         };
 
         GameObject tem = new GameObject("MUA-OSM Custom Mesh");
         tem.AddComponent<MeshFilter>();
 
 
 
         Mesh mesh = tem.GetComponent<MeshFilter>().mesh;
         mesh.Clear();
         mesh.vertices = vertices;
         mesh.triangles = triangles;
         mesh.Optimize();
         mesh.RecalculateNormals();
     }

When I call this method, it gives me error :

Instantiating mesh due to calling MeshFilter.mesh during edit mode. This will leak meshes. Please use MeshFilter.sharedMesh instead. UnityEngine.MeshFilter:get_mesh()

And when finally I change Mesh mesh = tem.GetComponent().mesh; to Mesh mesh = tem.GetComponent().sharedMesh;

It gives me another error, saying :

NullReferenceException: Object reference not set to an instance of an object

I clicked on the Console and it referred me to the same line.

Answer 1

Answer by Bunny83 · Jun 22, 2019 at 12:14 PM

Because you did not create any mesh yet. You should do:

 Mesh mesh = new Mesh();
 mesh.vertices = vertices;
 mesh.triangles = triangles;
 mesh.Optimize();
 mesh.RecalculateNormals();
 mesh.RecalculateBounds();
 
 GameObject tem = new GameObject("MUA-OSM Custom Mesh");
 MeshFilter mf = tem.AddComponent<MeshFilter>();
 mf.sharedMesh = mesh;

You forget the important RecalculateBounds as well. Though you should realise that the mesh you're going to create here is not a cube mesh. A single vertex can only have one vertex normal. So your "cube" will look more like a sphere when it comes to shading the faces. In order to have a cube mesh to actually look like a cube you need 24 vertices (4 vertices per face, so 6*4 == 24). This is exactly what the default Unity cube mesh looks like. Apart from the vertex normals if you want to have a texture mapped onto the faces you need to specify UV coordinates as well. Again those have to be specified per vertex. So another reason why a single face need to have it's own vertices.

0 Show 1 · Share

Bunny83 · Jun 22, 2019 at 12:36 PM 0

Share

I've posted a code example how to create a mesh that resembles the camera view frustum. With a few modifications it would create a normal cube mesh. The only difference would be the initial 8 corners. The code actually splits up the 8 corners into 24 vertices using the m_VertOrder map. All that needs to be changed is like 30 and 31 you would set them to 0 and 1 instead of using the near and far planes and als remove the for loop in line 33 and 34. This should give you a cube mesh. However still without UV coordinates, so no texture mapping.

Procedural Mesh problem from Editor Window

1 Reply

Your answer

Update about the future of Unity Answers

Follow this Question

Related Questions