c#中可能出现的意外引用比较

本文关键字:意外 引用 比较 | 更新日期: 2023-09-27 18:05:11

我得到一个可能意外的引用比较;要获得值比较,请在GetPrice方法的if语句中对左侧进行强制转换,键入'string'。它在所有"if (size == "Small")"语句中高亮显示。以下是我的变量:

drinkType = GetDrinkType();
size = GetDrinkSize();
price = GetPrice(size);
private string GetDrinkType()
        {
            string theDrink;
            theDrink = "None Selected";
            if (rdoCoffee.Checked)
            {
                theDrink = "Coffee";
            }
            else if (rdoCoco.Checked)
            {
                theDrink = "Hot Chocolate";
            }
            else if (rdoSmoothie.Checked)
            {
                theDrink = "Smoothie";
            }
            return theDrink;
        }
        private string GetDrinkSize()
        {
            string theSize;
            theSize = "None Selected";
            if (rdoSmall.Checked)
            {
                theSize = "Small";
            }
            else if (rdoMedium.Checked)
            {
                theSize = "Medium";
            }
            else if (rdoLarge.Checked)
            {
                theSize = "Large";
            }
            return theSize;
        }
        private decimal GetPrice(object size)
        {
            decimal thePrice;
            thePrice = 0;
            if (size == "Small")
            {
                thePrice = 1.25m;
            }
            else if (size == "Medium")
            {
                thePrice = 2.50m;
            }
            else if (size == "Large")
            {
                thePrice = 3.35m;
            }
            return thePrice;
        }

c#中可能出现的意外引用比较

参数size被声明为object类型,因此编译器不知道它实际上是string类型。因此,它对object类型使用默认的相等,这是一个引用比较。

如果将size的类型更改为string,它将使用string类的相等操作符重载,该操作符执行值比较。

private decimal GetPrice(string size)

出现警告是因为您正在将字符串与对象进行比较。如果你改变

 if (size == "Small")

 if (size.ToString() == "Small")

尝试将GetPrice中的"object"更改为"string".

由于在GetPrize中,size的类型是object,但您将其与size == "Large"string进行比较。

更改与"string text".Equals(object)的比较,如下所示

private decimal GetPrice(object size)
{
      decimal thePrice = 0m;
      if ("Small".Equals(size))
      {
            thePrice = 1.25m;
      }
      else if ("Medium".Equals(size))
      {
            thePrice = 2.50m;
      }
      else if ("Large".Equals(size))
      {
            thePrice = 3.35m;
      }
      return thePrice;
}