Question

我已经被困在这个剧本上大约几个星期了。我有这个脚本，每次新游戏开始时，列表中的项目都会随机播放。我不知道为什么，但我似乎根本无法改变项目清单。请帮忙，如果可以的话，自己测试一下。另外请向我解释可能存在的问题或我可以采取哪些措施来解决这个问题，我对此仍然很陌生。

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

public class RayCasting : MonoBehaviour
{
    public float pickupDistance;
    public List<Item> items;
    public List<Item> finalItems;

    #region Unity
    void Start ()
    {
        Screen.lockCursor = true;
        // Do a while loop until finalItems contains 5 Items
        while (finalItems.Count < 5) {
            Item newItem = items[Random.Range(0, items.Count)];
            if (!finalItems.Contains(newItem)) {
                finalItems.Add(newItem);
                }
            items.Clear();
            }
    }
    void Update ()
    {
        RaycastHit hit;
        Ray ray = new Ray(transform.position, transform.forward);
        if (Physics.Raycast(ray, out hit, pickupDistance))
        {
            foreach(Item item in finalItems)
            {

                if(Input.GetMouseButtonDown(0)) {
                    if (item.gameObject.Equals(hit.collider.gameObject))
                {
                    numItemsCollected++;
                    item.Collect();
                    break;
                        }
                }
            }
        }
    }

    void OnGUI()
    {

        GUILayout.BeginArea(new Rect(130,400,100,130));
        {
            GUILayout.BeginVertical();
            {
        if (numItemsCollected < items.Count)
        {
            foreach (Item item in finalItems)

                        GUILayout.Label(string.Format("[{0}] {1}", items.Collected ? "" + items.password: " ", items.name ));
        }
        else
        {
            GUILayout.Label("You Win!");
        }
            }
            GUILayout.EndVertical();
    }
        GUILayout.EndArea();
    }
    #endregion

    #region Private
    private int numItemsCollected;
    #endregion
}

[System.Serializable]
public class Item
{
    public string name;
    public GameObject gameObject;
    public int password;

    public bool Collected { get; private set; }



    public void Collect()
    {
        Collected = true;
        gameObject.SetActive(false);
    }

    public void passwordNumber()
    {
        password = 0;
        Collected = true;
        gameObject.SetActive(false);
    }


}

`

Answer 1

这个例子太复杂了，而且你似乎要问的问题有很多不相关的代码。作为一种基本策略，您应该尝试隔离您认为存在问题的代码，修复该问题，然后将其应用于更大的案例。

例如，您的基本逻辑至少有一个错误：Random.Range是包含，因此您可能有一个越界索引，除非您使用“index.Count-1” 。

这是一个从较大的“原始”列表中创建5个随机项列表的示例。您应该可以将其应用于您的代码：

using UnityEngine;
using System.Collections.Generic;

public class RandomList : MonoBehaviour {

  public List <int> originals = new List<int>();
  public List <int> randomized = new List<int>();
  public int desiredNumberOfRandomInts = 5;
    // Use this for initialization
    void Awake () {
    for(int i = 1; i <= 20; ++i) {
      originals.Add(i);
    }
    while(randomized.Count < desiredNumberOfRandomInts) {
      int randomSelection = originals[Random.Range(0, originals.Count-1)];
      if (!randomized.Contains(randomSelection)) {
        randomized.Add(randomSelection);
      }
    }
  }
}

Answer 2

我认为删除items.Clear()可能会有效，原因我在评论中解释过。

while (finalItems.Count < 5) {
   Item newItem = items[Random.Range(0, items.Count)];
   if (!finalItems.Contains(newItem)) {
       finalItems.Add(newItem);
   }
}

但是，我假设你已经在某个地方已经有了items的列表（并且列表必须是＆gt; 5，因为你将循环最少5次）。如果您不items.Count仍然会保持为0，并且随机变为Random.Range(0,0)。

我在这里做错了Unity 3D

2 个答案: