Как сделать падающие блоки лучше?

Несколько вещей, которые вы определенно можете улучшить ...

1. Когда у вас есть коллекция экземпляров, используйте структуру данных, подходящую для коллекций.

Вместо этого:

private GameObject box_0;
private GameObject box_2; //<<<That typo tho!
private GameObject box_3;
private GameObject box_4;
private GameObject box_5;
private GameObject box_6;
//...

Вы можете и должны это сделать:

private GameObject[] boxes = new GameObject[6];
private List<GameObject> clones = new List<GameObject>();

Это, соответственно, array и list; два из многих типов данных коллекции C #.

• Массивы отлично подходят для фиксированного количества элементов определенного типа объекта.
• Списки отлично подходят для динамического количества элементов.

Другие типы включают: Set<>, Dictionary<>, Queue<>, Stack<> и другие; у каждого есть свой оптимальный вариант использования.

2. Помните, что вы программируете. По возможности старайтесь, чтобы машина выполняла тяжелую работу.

Вместо этого:

box_0 = GameObject.Find("box_0");
box_2 = GameObject.Find("box_2"); //<<<That typo tho!
box_3 = GameObject.Find("box_3");
box_4 = GameObject.Find("box_4");
box_5 = GameObject.Find("box_5");
box_6 = GameObject.Find("box_6");

У вас может быть это:

for(int i=0; i<6; i++)
    boxes[i] = GameObject.Find("box_"+i);

Это называется циклом. Это способ сделать это, когда вам нужно выполнить повторяющуюся задачу несколько раз. В частности, это цикл for.

Это также:

• while (while(/*condition evaluated before loop*/){/*some code*/})
• do-while (do{/*some code*/}while(/*condition evaluated after loop*/))
• for-each (foreach(var element in someCollection){/*some code*/})
• И другие.

3. Помните о дублировании!

Переменная a в:

if (Input.GetKeyDown (KeyCode.A)) {
    a = true;
    if (a) {
        clone = Instantiate (box_0, transform.position, Quaternion.identity);
        a = false;
    }
}
if (Input.GetKeyDown (KeyCode.B) || Input.GetKeyDown (KeyCode.C)) {
    a = true;
    if (a) {
        clone1 = Instantiate (box_2, transform.position, Quaternion.identity);
        a = false;
    }
}
//...

ничего! И b даже не используется!

4. Если ваши клоны являются динамическими (иногда могут иметь значение NULL, а иногда нет), вы можете обрабатывать их динамически!

Если вы сохраните clones список как список существующих клонов, а не список возможных клонов, будет намного проще автоматизировать работу. потом!

if (Input.GetKeyDown (KeyCode.A))
    clones.Add(Instantiate(boxes[0], transform.position, Quaternion.identity));
if (Input.GetKeyDown (KeyCode.B) || Input.GetKeyDown (KeyCode.C))
    clones.Add(Instantiate(boxes[1], transform.position, Quaternion.identity);
//...

Позже вместо этого:

if (clone != null) {
    clone.transform.Translate (Vector3.down * fallspeed * Time.deltaTime, Space.World);
} else if (clone1 != null) {
    clone1.transform.Translate (Vector3.down * fallspeed * Time.deltaTime, Space.World);
} else if (clone2 != null) {
    clone2.transform.Translate (Vector3.down * fallspeed * Time.deltaTime, Space.World);
} else if (clone3 != null) {
    clone3.transform.Translate (Vector3.down * fallspeed * Time.deltaTime, Space.World);
} else if (clone4 != null) {
    clone4.transform.Translate (Vector3.down * fallspeed * Time.deltaTime, Space.World);
} else {
    if (clone5 != null) {
        clone5.transform.Translate (Vector3.down * fallspeed * Time.deltaTime, Space.World);
    }
}

Ты можешь сделать это:

foreach(var clone in clones)
    clone.transform.Translate (Vector3.down * fallspeed * Time.deltaTime, Space.World);

5. Вы должны быть последовательны.

private GameObject box_0;
private GameObject box_2; //<<<NOPE! should be box_1!
//And others should follow the sequence from 1!
private GameObject box_3;
private GameObject box_4;
private GameObject box_5;
private GameObject box_6;

private GameObject clone; //<<<NOPE! Should be 'clone_0', 'clone_1' etc...
private GameObject clone1;
private GameObject clone2;
private GameObject clone3;
private GameObject clone4;
private GameObject clone5;

Во-первых, это слишком много переменных. Когда вы обнаруживаете, что копируете и вставляете свой код, вы, вероятно, делаете что-то не так. Просто поместите всех этих плохих парней в List<GameObject>, и вы достигнете того же результата.

Я бы порекомендовал то, что ответил Микко Койвисто, а именно создание другого сценария для обработки фактического эффекта падения ящиков. Но чтобы сохранить все в одном сценарии, я поступил иначе.

Я немного изменил ваш код, чтобы он стал намного чище и читабельнее. Комментарии в основном рассказывают обо всех изменениях.

Насчет того, почему застряли старые коробки. Вы продолжали переопределять переменные клона при создании нового клона. Итак, в обновлении ваши старые клоны больше не будут обновляться, потому что они больше не определены в переменной, следовательно, они перестали падать.

Примечание: для того, чтобы это работало, вам нужно будет добавить поля в свой скрипт в редакторе, вы также можете сделать это в коде, но это намного сложнее. Просто щелкните сценарий в редакторе, откройте список ящиков, установите размер на шесть и перетащите их туда.

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class Falling : MonoBehaviour
{

    public float fallspeed = 8.0f;
    //removed bool a, b because they never get used anymore

    //"boxes" is public so you can add the boxes in the editor instead of having to do it in your start function
    public List<GameObject> boxes;  //Instead of 6 different variables you could just throw them all in a list
    private List<GameObject> clones; //Same here

    // Update is called once per frame
    void Update()
    {

        // makes box_0 fall if a key is pressed
        //Not sure why you were checking if(a) constantly when you set it true 1 line ahead, how could it be false? I took the liberty to remove that
        if (Input.GetKeyDown(KeyCode.A))
        {
            clones.Add(Instantiate(boxes[0], transform.position, Quaternion.identity)); //No need for different variables, just throw them all in one list
        }
        if (Input.GetKeyDown(KeyCode.B) || Input.GetKeyDown(KeyCode.C))
        {
             clones.Add(Instantiate(boxes[1], transform.position, Quaternion.identity));
        }
        if (Input.GetKeyDown(KeyCode.Q) && Input.GetKeyDown(KeyCode.E))
        {
            clones.Add(Instantiate(boxes[2], transform.position, Quaternion.identity));
        }
        if (Input.GetKeyDown(KeyCode.R) || (Input.GetKey(KeyCode.LeftShift) && Input.GetKeyDown(KeyCode.S)))
        {
            clones.Add(Instantiate(boxes[3], transform.position, Quaternion.identity));
        }
        if (Input.GetKeyDown(KeyCode.U))
        {
            clones.Add(Instantiate(boxes[4], transform.position, Quaternion.identity));
        }
        if (Input.GetKeyDown(KeyCode.I))
        {
            clones.Add(Instantiate(boxes[5], transform.position, Quaternion.identity));
        }


        //Instead of checking if any of them exist you can just loop through the list with clones and make them all fall
        foreach (GameObject clone in clones)
        {
            clone.transform.Translate(Vector3.down * fallspeed * Time.deltaTime, Space.World);
        } 
    }
}

«во-первых, если я создам более одного клона одного и того же клона, они застрянут, и я не думаю, что обработал нулевой игровой объект очень хорошо».

Прикрепите это к каждому ящику и удалите часть clone.transform.Translate из сценария падения:

public class BoxScript : MonoBehaviour {

    public float fallspeed = 8.0f;
    public bool fall = false;
    void Update () {
        if (fall) {
            transform.Translate (Vector3.down * fallspeed * Time.deltaTime, Space.World);
        }
    }
}

... а затем при создании новых ящиков:

clone = Instantiate (box_0, transform.position, Quaternion.identity);
clone.fall = true;

Я думаю, вы можете вместо этого составить список блоков и клонов, чтобы вы могли факторизовать свой код (я не знаком с единством, поэтому могут быть опечатки, но общая идея будет такой:

        var clones = new List<GameObject>();
        GameObject box = null;
        if (Input.GetKeyDown(KeyCode.A))
        {
            box = box_0
        }
        // else if ... box = box_1 ...
        // end of your series of else if
        if (box != null)
        {
            clone = Instantiate(box, transform.position, Quaternion.identity);
            clones.Add(clone);
        }
        box = null;
        //   
        foreach (GameObject clone in clones)
        {
            clone.transform.Translate(Vector3.down * fallspeed * Time.deltaTime, Space.World)
        }