Question

在我的mvc应用程序中，我们希望在一年中的某些时间显示两个链接中的一个。基本上，当我接到管理层的电话时，我必须切换链接。因此，我认为不必重新编译应用程序，而是将自定义应用程序设置添加到web.config文件中。然后我创建了一个包装器，以便强类型化。现在，我的问题是我不知道在哪里执行逻辑。应该在我的视图模型中添加属性并根据配置设置值在控制器中设置它？或者我应该直接在我的视图中阅读它并在两个链接之间切换？我很确定它只属于视图或控制器，而不属于服务层，因为它专门用于UI内容。

Details.cshtml //当前代码

 @if(Search.App.ParcelDetailDisplayMode == Search.App.DisplayMode.Tax ){
     <a id="tax-link" href="@taxlink" title="View Tax Bill on Tax Collectors Website">Tax Bill</a>
 }
 else if(Search.App.ParcelDetailDisplayMode == Search.App.DisplayMode.Trim ){
        <a id="trim-link" href="@trimlink" title="View your TRIM notice online">Trim Notice</a>
 }

的web.config

<add key="ParcelDetailDisplayMode" value="Tax"/>

配置包装

namespace Search
{
    /// <summary>
    /// The app.
    /// </summary>
    public static class App
    {
        /// <summary>
        /// Gets the tax bill link.
        /// </summary>
        public static string TaxBillLink
        {
            get
            {
                return ConfigurationManager.AppSettings["TaxBillLink"];
            }
        }

        /// <summary>
        /// Gets the trim notice link.
        /// </summary>
        public static string TrimNoticeLink
        {
            get
            {
                return ConfigurationManager.AppSettings["TrimLink"];
            }
        }

        /// <summary>
        /// Gets the map link.
        /// </summary>
        public static string MapLink
        {
            get
            {
                return ConfigurationManager.AppSettings["MapLink"];
            }
        }

        /// <summary>
        /// Gets the update address link.
        /// </summary>
        public static string UpdateAddressLink
        {
            get
            {
                return ConfigurationManager.AppSettings["UpdateAddressLink"];
            }
        }

        /// <summary>
        /// Gets the release name.
        /// </summary>
        public static string ReleaseName
        {
            get
            {
                return ConfigurationManager.AppSettings["ReleaseName"];
            }
        }

        /// <summary>
        /// Gets the parcel detail display mode.
        /// </summary>
        public static DisplayMode ParcelDetailDisplayMode
        {
            get
            {
                var r = DisplayMode.Tax;
                DisplayMode.TryParse(ConfigurationManager.AppSettings["ParcelDetailDisplayMode"], out r);
                return r;
            }
        }

        /// <summary>
        /// The display mode.
        /// </summary>
        public enum DisplayMode
        {
            /// <summary>
            /// The trim.
            /// </summary>
            Trim, 

            /// <summary>
            /// The tax.
            /// </summary>
            Tax
        }
    }
}

Answer 1

我想说这并不重要。将其添加为模型的属性可能会让您更加分离。

但重要的是你的包装器是静态的。这将使得为单元测试（或任何其他目的）模拟它非常困难

Answer 2

控制器中应该没有逻辑。请阅读此内容，例如：Where should I put my controller business logic in MVC3

或者这个：https://softwareengineering.stackexchange.com/questions/165444/where-to-put-business-logic-in-mvc-design

我知道这很有诱惑力，但是你把它放在那里的逻辑越少，你将来会发现自己最好的。

我认为答案是：

您应该在控制器的业务层中读取您的属性，并将其一直传递到模型对象中的视图。

Answer 3

我同意Maurizio一般认为所有业务逻辑都应该在某个服务/业务逻辑层中。但是在这种情况下，因为您只是从web.config中获取值，无论是在控制器操作中，还是执行以下操作：

var someValue = App.TaxBillLink;

或者你这样做：

var someValue = _linkService.GetTodaysLink();

除非有某种逻辑需要进行单元测试，否则

确实无关紧要。

控制器应该从web.config读取自定义应用程序设置吗？

3 个答案:

我认为答案是：