While Do не работает?

Question

[WasTabon]

private void SpawnCluesAndBomb()
    {
        Vector2 firstCluePos;
        Vector2 secondCluePos;
        Vector2 thirdCluePos;

        Vector2 bombPos;

        bool isGood = false;
        
        firstCluePos = new Vector2(RandomInt(0, 256), RandomInt(0, 256));
        do
        {
            secondCluePos = new Vector2(RandomInt(0, 256), RandomInt(0, 256));
        } while (Vector2.Distance(firstCluePos, secondCluePos) < 200f);

        do
        {
            thirdCluePos = new Vector2(RandomInt(0, 256), RandomInt(0, 256));
        } while (Vector2.Distance(firstCluePos, thirdCluePos) < 200f && Vector2.Distance(secondCluePos, thirdCluePos) < 200f);

        do
        {
            bombPos = new Vector2(RandomInt(0, 256), RandomInt(0, 256));
        } while (Vector2.Distance(firstCluePos, bombPos) < 300f && Vector2.Distance(secondCluePos, bombPos) < 300f && Vector2.Distance(thirdCluePos, bombPos) < 300f);

        Instantiate(_clue, firstCluePos, Quaternion.identity);
        Instantiate(_clue, secondCluePos, Quaternion.identity);
        Instantiate(_clue, thirdCluePos, Quaternion.identity);
        
        Instantiate(_bomb, bombPos, Quaternion.identity);
        Debug.Log($"Distance 1: {Vector2.Distance(firstCluePos, bombPos)}, Distance 2: {Vector2.Distance(secondCluePos, bombPos)} Distance 3: {Vector2.Distance(thirdCluePos, bombPos)}");
    }

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


Это я что-то не так сделал, или выводит правильно и что-то не понимаю

Answer 1

[GavriKos]

Ну так первая дистанция больше 300. false && true && true даст false. Цикл и прервется.

P.S. за такие циклы в целом надо руки отрубать. Их время выполнения неизвестно от слова совсем. Могут вообще никогда не завершитсья

Comments

[WasTabon]

а как можно пофиксить?

[WasTabon]

while же должен работать пока все не будет true, разве нет?

[GavriKos]

WasTabon, переписать условие цикла. Как оно по русски формулируется? "генерировать до тех пор, пока все три точки не станут на расстоянии >300"?
while (!(dst1>300 && dst2>300 && dst3>300))
"все три" - обеспечивает &&
"НЕ станут" - отрицание в начале

[GavriKos]

WasTabon, вайл работает пока в нем true, у вас первое условие стало false и все выражение стало false потому что &&

[WasTabon]

GavriKos, окей, я так сделал и юнити зависать начал при старте

[WasTabon]

GavriKos, а не, все работает. Спасибо

[GavriKos]

WasTabon, зависать начал ровно потому, что этот алгоритм плох в принципе. Рандом может очень долго не выдавать нужный результат. Нужно отказываться от генериации невалидных точек и генерировать сразу валидные

[WasTabon]

GavriKos, а как можно сделать это? кроме как сразу готовые засунуть

[GavriKos]

Брать такие рандомы диапазона, чтобы точка была на нужном расстоянии сразу. Геометрия

[freeExec]

GavriKos, Можно взять любые три рандомные, а потом раздвинуть их на нужные расстояния. Но да согласен, чистая геометрия.

[GavriKos]

freeExec, можно так. Я думал для одной точки - брать рандомное расстояние в допустимых диапазонах (от 300 до 1000), брать рандомный угол, и через полярные получить декартовы. В общем тут задача интересная, особенно с 3 точками, да

[WasTabon]

GavriKos, есть ссылка на статью/гайд/алгоритмы какую нибудь может?

[GavriKos]

WasTabon, нет, и не будет. Это частная задача. Не все покрывается гайдами.

Answer 2

[WasTabon]

using UnityEngine;

public class ObjectSpawner : MonoBehaviour
{
    public GameObject firstObjectPrefab;
    public GameObject secondObjectPrefab;
    public GameObject thirdObjectPrefab;
    public GameObject bombObjectPrefab;

    private Vector2 firstPos;
    private Vector2 secondPos;
    private Vector2 thirdPos;
    private Vector2 bombPos;

    private void Start()
    {
        SpawnObjects();
    }

    private void SpawnObjects()
    {
        bool validPositions = false;
        while (!validPositions)
        {
            // Generate random positions for all objects
            float angle = Random.Range(0f, Mathf.PI * 2f);
            float distanceFromCenter = Random.Range(0f, 200f);
            firstPos = new Vector2(Mathf.Cos(angle) * distanceFromCenter, Mathf.Sin(angle) * distanceFromCenter);

            angle = Random.Range(0f, Mathf.PI * 2f);
            distanceFromCenter = Random.Range(0f, 200f);
            secondPos = new Vector2(Mathf.Cos(angle) * distanceFromCenter, Mathf.Sin(angle) * distanceFromCenter);

            angle = Random.Range(0f, Mathf.PI * 2f);
            distanceFromCenter = Random.Range(0f, 200f);
            thirdPos = new Vector2(Mathf.Cos(angle) * distanceFromCenter, Mathf.Sin(angle) * distanceFromCenter);

            angle = Random.Range(0f, Mathf.PI * 2f);
            distanceFromCenter = Random.Range(0f, 256f);
            bombPos = new Vector2(Mathf.Cos(angle) * distanceFromCenter, Mathf.Sin(angle) * distanceFromCenter);

            // Check if all positions are valid
            if (Vector2.Distance(firstPos, secondPos) >= 200f &&
                Vector2.Distance(secondPos, thirdPos) >= 200f &&
                Vector2.Distance(thirdPos, firstPos) >= 200f &&
                Vector2.Distance(firstPos, bombPos) >= 256f &&
                Vector2.Distance(secondPos, bombPos) >= 256f &&
                Vector2.Distance(thirdPos, bombPos) >= 256f)
            {
                validPositions = true;
            }
        }

        // Spawn objects at the generated positions
        Instantiate(firstObjectPrefab, firstPos, Quaternion.identity);
        Instantiate(secondObjectPrefab, secondPos, Quaternion.identity);
        Instantiate(thirdObjectPrefab, thirdPos, Quaternion.identity);
        Instantiate(bombObjectPrefab, bombPos, Quaternion.identity);
    }
}

ChatGPT порешал вопрос