Возможно непреднамеренное сравнение ссылок в C #

Question

Возможно непреднамеренное сравнение ссылок в C #

1

Я получаю возможное непреднамеренное сравнение ссылок; чтобы получить сравнение значений, введите левую сторону для ввода "string" в операторах if в методе GetPrice. Он показывает его выделение на всех операциях 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;
        }

tstokes8040 24 май 2014, в 16:39

Источник

Теги:

c#

methods

variables

4 ответа

Ещё вопросы

Thomas Levesque · Answer 1 · 2014-05-24T13-01-00.000Z

Параметр size объявляется для object типа, поэтому компилятор не знает его на самом деле типа string. Таким образом, он использует равенство по умолчанию для object type, которое является эталонным сравнением.

Если вы измените тип size на string, он будет использовать перегрузку оператора равенства из класса string, который выполняет сравнение значений.

private decimal GetPrice(string size)

Быстрый ответ на вопрос. Будет ли приведение объекта в виде строки все равно выдать ошибку, или компилятор не достигнет этого уровня?
@ Погриндис, это просто предупреждение, а не ошибка. Но приведение к строке подавит предупреждение и даст правильный результат, потому что тогда вы будете сравнивать два значения, которые статически известны как строки, поэтому компилятор вызовет корректную == перегрузку.

Robert Chumley · Answer 2 · 2014-05-24T12-17-00.000Z

Попробуйте изменить "объект" в GetPrice на "string".

Это также сработало, и я думаю, что так лучше. Спасибо!

Jevgeni Geurtsen · Answer 3 · 2014-05-24T11-50-00.000Z

Предупреждение возникает, потому что вы сравниваете строку с объектом. Если вы меняете

 if (size == "Small")

в

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

предупреждение будет удалено.

Это сработало, спасибо! Я думаю, я не осознавал, что я это делал.

Candide · Answer 4 · 2014-05-24T12-29-00.000Z

Поскольку в GetPrize тип size является object, но вы сравниваете его со string с size == "Large".

Измените сравнение со "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;
}