Question

尝试在此处重新说明/清理问题：

我正在尝试做某种条件语句来计算一个值。为了模拟我的数据，我在控制器中（临时）分配值以查看我的UI是如何出现的。我可以在视图中的功能块中执行计算，但它很长并且不属于那里。所以，我现在正在尝试在模型中进行计算（Calculations.cs）。

计算的代码正在传递一个值，除了我的条件失败并传递默认值0时它应该根据我在控制器中的模拟值传递另一个值

以下是Calculations.cs

public class Calculations
{
    PriceQuote price = new PriceQuote();
    StepFilingInformation filing = new StepFilingInformation();
    public decimal Chapter7Calculation
    {
        get
        {
            return
                price.priceChapter7
                +
                ((filing.PaymentPlanRadioButton ==
                    Models.StepFilingInformation.PaymentPlan.Yes)
                ?
                price.pricePaymentPlanChapter7
                :
                0);
        }

    }
}

我最初(filing.PaymentPlanRadioButton == Models.StepFilingInformation.PaymentPlan.Yes)检查单选按钮是否设置为“是”，但是将其更改为ReferenceEquals。这不会影响结果。

我让我的控制器将值PaymentPlanRadioButton指定为“是”，因此pricePaymentPlanChapter7应该是添加到priceChapter7的值，但事实并非如此。而是在回退到条件时添加“0”。因此即使我在控制器中分配它，PaymentPlanRadioButton也为空。

我无法弄清楚如何解决这个问题。如果我在模型中分配它并使其工作无法解决问题，因为当我删除模拟控制器并期望用户选择单选按钮时它仍然是null并且条件将失败。 / p>

这是“模拟”控制器：

public class QuoteMailerController : Controller
{
    public ActionResult EMailQuote()
    {
        Calculations calc = new Calculations();
        var total = calc.Chapter7Calculation;

        QuoteData quoteData = new QuoteData
        {
            StepFilingInformation = new Models.StepFilingInformation
            {
                //"No" is commented out, so "Yes" is assigned
                //PaymentPlanRadioButton = 
                    //Models.StepFilingInformation.PaymentPlan.No,
                PaymentPlanRadioButton = 
                    Models.StepFilingInformation.PaymentPlan.Yes,
            }
         };
    }
}

这就是我存储价格（PriceQuote.cs）的地方：

public class PriceQuote
{
    public decimal priceChapter7 { get { return 799; } }
    public decimal pricePaymentPlanChapter7 { get { return 100; } }
}

这是我的ViewModel：

public class QuoteData
{
    public PriceQuote priceQuote;
    public Calculations calculations;
    public StepFilingInformation stepFilingInformation { get; set; }
    public QuoteData()
    {
        PriceQuote = new PriceQuote();
        Calculations = new Calculations();
    }
}

因此，这应该起作用的方式是799 + 100 = 899，因为PaymentPlan.Yes被指定为控制器中单选按钮的值。但是我得到的只是799（799 + 0），因为当我调试PaymentPlanRadioButton时出现了null。

有任何想法/指导吗？

以防万一，PaymentPlanRadioButton位于StepFilingInformation.cs内（并且是我的模特之一）：

public enum PaymentPlan
{
    No,
    Yes
}
public class PaymentPlanSelectorAttribute : SelectorAttribute
{
    public override IEnumerable<SelectListItem> GetItems()
    {
        return Selector.GetItemsFromEnum<PaymentPlan>();
    }
}       
[PaymentPlanSelector(BulkSelectionThreshold = 3)]
public PaymentPlan? PaymentPlanRadioButton { get; set; }

抱歉这个长度。

对于背景这就是我想要做的事情

我最初在我的视图中的功能块中有这个计算代码。计算在那里工作正常，但显然非常冗长且不合适。

这是我的功能块看起来像（部分）

@{ Model.PriceQuote.calculationChapter7
    =
    Model.PriceQuote.priceChapter7
    +
    ((Model.StepFilingInformation.PaymentPlanRadioButton == 
        StepFilingInformation.PaymentPlan.No)
    ?
    Model.PriceQuote.priceNoPaymentPlan
    :
    Model.PriceQuote.pricePaymentPlanChapter7)
    +
    ...//more of the same
    ;
}

所以我一直在努力将其变成.cs文件。

Answer 1

你真的得到了这段代码：

public decimal calculatingTest(MyModel MyModel)
{ ... }

而不是这个？

public decimal calculatingTest(MyModel myModel) // note the difference in case.
{ ... }

只有这样才能让生活变得复杂。

Answer 2

您是否尝试过调试以确保MyModel.MyModelProperty在运行时确实等于“否”？

如果是，也许您需要覆盖相等运算符（即可能“==”测试失败） - 请参阅MSDN文章以获取教程：

http://msdn.microsoft.com/en-us/library/ms173147(v=vs.80).aspx

Answer 3

在您的计算类中，您将获得'100'作为返回值，而不是因为

MyModel.MyProperty == MyModel.MyPropertyEnum.Yes

但是因为不等于MyModel.MyPropertyEnum.No。所以你的if块只是属于默认情况。在这段代码剪辑中，MyModel.MyProperty没有分配值，所以它只取默认值，我可以告诉它;

(MyPropertyEnum?)null

“并非所有代码路径都返回一个值”异常是由于您将MyProperty设为可以为空的类型而未提供结果的事实;

MyModel.MyProperty == null

同样，您使用类型名称作为变量名称是一个糟糕的选择。您在Calculations.cs中有一个名为“MyModel”的类实例变量，在calculateTest中有一个名为MyModel的方法参数。这很令人困惑。

而最后，你的StackOverflowException的原因是你用相同的参数递归调用calculateTest，所以它进入一个无限循环。

这里有很多问题你可以清理，你可能会找到问题的原因。

Answer 4

[添加第二个答案，因为添加的代码自原始帖子以来发生了重大变化]。

我认为这里的根本问题是你实现Calculations类的方式。 Chapter7Calculation属性总是将返回799 + 0，因为它使用私有本地类变量来确定要返回的值;

public class Calculations
{
    PriceQuote price = new PriceQuote();

    // private local variable - will ALWAYS have PaymentPlanRadioButton = null
    StepFilingInformation filing = new StepFilingInformation();

    public decimal Chapter7Calculation
    {
        get {
            return
                price.priceChapter7
                +
                (filing.PaymentPlanRadioButton == Models.StepFilingInformation.PaymentPlan.Yes)
                ? price.pricePaymentPlanChapter7
                : 0);
        }
    }
}

你有一个'控制器'修改了一些其他的StepFilingInformation实例，你的计算类没有意识到这一点。同样，您的PriceQuote类只返回常量或静态值，因此不需要实例化它。像这样修改那个类;

public static class PriceQuote
{
    public static decimal PriceChapter7 { get { return 799; } }
    public static decimal PricePaymentPlanChapter7 { get { return 100; } }
}

将您的计算更改为类似的方法;

public decimal CalculatePrice(QuoteData quoteData)
{
    return PriceQuote.PriceChapter7 + 
        (quoteData.StepFilingInformation.PaymentPlanRadioButton == Models.StepFilingInformation.PaymentPlan.Yes) 
        ? PriceQuote.PricePaymentPlanChapter7 : 0);
}

现在你的控制器可以传入它创建的QuoteData实例，你应该看到更好的结果。模拟控制器的示例代码;

public class QuoteMailerController : Controller 
{
    public ActionResult EMailQuote()
    {
        Calculations calc = new Calculations();

        QuoteData quoteData = new QuoteData
        {
            StepFilingInformation = new Models.StepFilingInformation
            {
                PaymentPlanRadioButton = Models.StepFilingInformation.PaymentPlan.Yes,
            }
         };

         var total = calc.CalculatePrice(quoteData);
    }
}

如果/ Else OR“？：”（有条件）不起作用

4 个答案: