关于实例化的建议？

Question

这是我关于 Stack Overflow 的第一个问题。我是编码新手，所以请耐心等待我和我可怕的代码。

void ChickenInstantiate()
    {
        LocOfChkn.Add(spawnLoc);
        int i = 0;
        for (int ChickenCount = 0; ChickenCount < maxChickenCount; ChickenCount++)
        {
            while (Vector3.Distance(spawnLoc, LocOfChkn[i]) < 2)
            {
                spawnLoc = new Vector3(Random.Range(randXMin, randXMax), Random.Range(randYMin, randYMax), 1);
                spawnLoc.z = 5;
                i += 1;
                if (i >= LocOfChkn.Count)
                {
                    break;
                }
            }
            Instantiate(ChickenPrefab1, spawnLoc, Quaternion.identity);
            LocOfChkn.Add(spawnLoc);
            i = 0;
        }
    }

这是一些用于安装预制件的代码，但要确保它不会在一个区域内实例化，坦率地说，我不知道应该如何修复它——它完全坏了；鸡没有出现，我什至不确定它们是否在正确的区域产卵，等等（spawnLoc 的第一个实例是在启动时定义的，顺便说一句）

我正在使用当前最新版本的统一和视觉工作室。

Answer 1

• 您应该为您添加到locOfChkn 的第一个位置Instantiate 一只鸡。这既可以确保您的代码正在运行，又可以避免出现问题。

• 使用 VSCode 或您最喜欢的 IDE 对您的函数进行逐步评估，以准确了解正在发生的事情。除此之外，为每个步骤添加Debug.Logs。

• 检查任何生成的鸡的层次结构。也许他们被禁用了。

现在讨论其他问题：

• 请按照 C# 约定编写代码。 Chkn 和 Loc 非常不直观。

• 如果在评估距离时忽略 z 位置，请考虑使用 Vector2.Distance。

• 将字段用于仅在一个函数中使用的值是没有用的。只需将 spawnLoc 设置为 ChickenInstantiate 中的 var。

这是您的代码的清理版本

void InstantiateChicken()
{
    List<Vector3> chickenPositions = new List<Vector3>();

    for (int i = 0; i < maxChickenCount; i++)
    {
        bool doesCollide;
        Vector3 spawnLocation;
        do
        {
            spawnLocation = new Vector3(
                Random.Range(randXMin, randXMax),
                Random.Range(randYMin, randYMax),
                5);
            doesCollide = false;
            foreach (var pos in chickPositions)
            {
                if (Vector2.Distance(pos, spawnLocation) < 2f)
                    doesCollide = true;
            }
        }
        while (doesCollide);

        Instantiate(chickenPrefab1, spawnLocation, Quaternion.identity);
        chickenPositions.Add(spawnLocation);
    }
}

或者，只需手动放置所有鸡，并让它们在产卵时以随机因素偏移它们的位置。