首页 > 解决方案 > [Unity]如何用switch case触发随机Icon?

问题描述

我目前正在尝试制作一个随机图标弹出机制,例如客户随机订购比萨饼。

我正在尝试制作一个随机显示机制,每个客户都有一个带有开关盒的随机订单,它会像从开关盒中选择的订单一样显示。

我将订单图标作为一个大小为 3 的游戏对象数组。所以当他们停止订购时,它将激活他们头顶的游戏对象。

以下是代码的运行方式:

private void OnCollisionStay2D(Collision2D collision)
{
    if (collision.gameObject.CompareTag("OrderingPoint")) 
    {
        int rand = Random.Range(0, 3);            
        switch (rand)
        {
            case 0:

                OrderPicture[0].SetActive(true);
                if (rand == 0 && collision.gameObject.name == "Rendang")
                {
                    OrderPicture[0].SetActive(false);
                    GetRendang();
                }               
                //RecipeObject.Artwork.SetActive(false);
                break;
            case 1:

                //CuisineObject.Artwork.SetActive(true);
                OrderPicture[1].SetActive(true);
                if (rand == 1 && collision.gameObject.name == "Gado Gado")
                {
                    OrderPicture[1].SetActive(false);
                    GetGadoGado();
                }

                //RecipeObject.Artwork.SetActive(false);
                break;
            case 2:

                OrderPicture[2].SetActive(true);
                if (rand == 2 && collision.gameObject.name == "Soto")
                {
                    OrderPicture[2].SetActive(false);
                    GetSoto();
                }

                //CuisineObject.Artwork.SetActive(false);
                break;
        }
    }      
}

我预计它将启用图像游戏对象,但它不会。所以我错过了什么?

标签: c#visual-studiounity3dtriggersswitch-statement

解决方案


首先每帧OnCollisionStay调用一次,因此目前您每帧生成并执行一个新的随机索引。

您可能更应该使用OnCollisionEnter2D仅在发生碰撞的帧中调用的。


switch-case由于您的检查if(rand == XY)是多余的,而且硬编码的索引OrderPicture[XY].SetActive(true);已经被值覆盖,因此有很多事情可以在根本不使用的情况下被简化rand

private void OnCollisionEnter2D(Collision2D collision)
{
    if(!collision.gameObject.CompareTag("OrderingPoint")) return;

    int rand = Random.Range(0, 3);

    // First deactivate all pictures because as I understand you want to show only one
    foreach(var icon in OrderPicture)
    {
        icon.SetActive(false);
    }

    // Show the icon according to the random index
    OrderPicture[rand].SetActive(true);

    switch (rand)
    {
        case 0:
            if (collision.gameObject.name == "Rendang")
            {
                GetRendang();
            }               
            //RecipeObject.Artwork.SetActive(false);
            break;

        case 1:
            //CuisineObject.Artwork.SetActive(true);
            if (collision.gameObject.name == "Gado Gado")
            {
                GetGadoGado();
            }

            //RecipeObject.Artwork.SetActive(false);
            break;

        case 2:
            if (collision.gameObject.name == "Soto")
            {
                GetSoto();
            }

            //CuisineObject.Artwork.SetActive(false);
            break;
    }
}

推荐阅读