Блог

У меня есть список кнопок пользовательского интерфейса и Vector2 для позиций. Я использую random.range для случайного размещения кнопок при каждой их загрузке. Но он никогда не показывает все 5 кнопок в разных положениях. вместо этого перекрывает некоторые из них. Может кто-нибудь, пожалуйста, помочь мне с этим?

 [SerializeField] List<Button> answersButtons = new List<Button>();
[SerializeField] List<Vector2> positions = new List<Vector2>();

void ShuffleAnswersList()
{             

    for (int i = 0; i < answersButtons.Count; i  )
    {              
       Vector2 tempPosition = 
       answersButtons[i].GetComponent<RectTransform>().position;

       int randomIndex = Random.Range(0, positions.Count);

       answersButtons[i].transform.position = positions[randomIndex];
       answersButtons[randomIndex].GetComponent<RectTransform>().position 
       = tempPosition;

       Debug.Log(randomIndex);          
    }      

}

// shuffle positions
public void Shuffle()
{
    for (int i = 0; i < positions.Count; i  )
    {
        int rnd = Random.Range(0, positions.Count);
        Vector2 tempGO = positions[rnd];
        positions[rnd] = positions[i];
        positions[i] = tempGO;
    }
}
 

Попробуйте этот код, не уверен, где ваш Random.Функция Range() происходит из, но есть встроенный Random, System который вы можете использовать для получения случайных целых чисел. В этом примере random объявляется заранее, а функция Next() используется для получения случайного числа:

 void Shuffle()
{
    System.Random random = new System.Random();
    for (int i = 0; i < positions.Count; i  )
    {
        int rnd = random.Next(0, positions.Count);
        Vector2 tempGO = positions[rnd];
        positions[rnd] = positions[i];
        positions[i] = tempGO;
    }
}
 

Когда вы делаете это :

 int randomIndex = Random.Range(0, positions.Count);

answersButtons[i].transform.position = positions[randomIndex];
 

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

Вы хотите либо :

• Перетасуйте список позиций (что вы сделали) и сохраните тот же порядок для списка ответов
• Перетасуйте список ответов и сохраните тот же порядок для позиции

Попробуйте это вместо :

 void ShuffleAnswersList()
{             
    //  Notice this v--------------------v to avoid out of range exception
    for (int i = 0; i < positions.Count amp;amp; i < answersButtons.Count; i  )
    {
// always use i ------V---------------------------------V
       answersButtons[i].transform.position = positions[i];
    }
}
 

Конечно, сначала убедитесь, что вы вызываете свой метод Shuffle . Этот метод необходимо исправить, Random.Range(min, max) он может возвращать max (и предоставлять исключение вне диапазона)

 // shuffle positions
public void Shuffle()
{
    for (int i = 0; i < positions.Count; i  )
    {
// notice this -------------------------------------V
        int rnd = Random.Range(0, positions.Count - 1);
        Vector2 tempGO = positions[rnd];
        positions[rnd] = positions[i];
        positions[i] = tempGO;
    }
}