如何做「適當」的MVC？

Question

我是一個較新的Web開發人員（約1年的經驗），他剛剛加入了一家新公司，該公司使用Umbraco CMS開發了許多我們開發的網站。在我上一份工作中，我只接觸了WebForms，所以在開始這項工作之前我做了一點關於MVC的研究，並且做了一個初學者的教程，但我不確定當涉及到的時候我是否理解社區的「最佳實踐」這個東西，所以希望你們可以幫助我。在Umbraco CMS中，有一些名爲「Macro-Scripts」的組件，它基本上是一個Razor View（.CSHTML），它允許您在頁面中編寫一堆C＃& HTML，並在其上顯示此「Macro-Script」您選擇的任何CMS頁面。在我的教程中（我在2個月前完成了這個教程），我想我記得讀過一些關於從不將條件邏輯/ C＃與HTML混合在一起的東西，並且最好使用Models &控制器？我有點困惑，因爲我們並不真正爲我們的網站建立控制器，因爲它們是CMS頁面，如果它們需要除Rich-Text以外的任何內容，那麼我們只需創建一個「Macro-Script」並放置它在需要擴展功能的頁面上。

我在這裏工作的第一個月裏發現的是，如果我能夠讓我的代碼在整個網站上可重複使用，那麼我可以節省大量時間，但我並不確定該怎麼做以「MVC友好」的方式？下面是一個「宏腳本」示例，它顯示了在CMS中創建的表單，然後嘗試獲取公司「聯繫人」信息並將其顯示在表單下。我附加的例子非常簡單，但是我所見過的一些宏腳本變成了超過500行的巨大混亂，每個循環，開關和代碼塊都不可讀。

是否有可能將這種類型的代碼拆分成模型以便在此視圖中使用？分割代碼有沒有什麼好處，而不是把它們放在同一個視圖中？ （如果有足夠的好處，我想向我的同事介紹這一點，並更改編碼標準）。有人能給我一個例子，說明如果將它分解成適當的「MVC」，這將如何看待？如果這不是發佈此類問題的正確位置，請將我指向更好的StackExchange站點。謝謝。

宏腳本搶CMS表&公司信息

@inherits umbraco.MacroEngines.DynamicNodeContext 

@* 
    Model = The current page the macro is executed on 
      @Model.bodyText 

    Parameter = collection of parameter values passed from the macro 
      @Paramter.myParam 

    Library = utillity library with common methods 
      @Library.NodeById(1233) 
*@ 

@* The fun starts here *@ 

@{ 
    string formClass = "form-content"; 
    var configNode = Model.AncestorOrSelf().Descendants("ConfigurationContainer").FirstOrDefault(); 
    if (configNode != null && configNode.Id > 0) 
    { 
     formClass = "form-content show-address"; 
    } 
    if (custom.Library.NodeHasPropertyAndValue(Model, "bodyText")) 
    { 
     <hr /> 
    } 

    <div class="form-page contact-us"> 
     <div> 
      <div class="@formClass"> 
       @Html.Raw(umbraco.library.RenderMacroContent("<?UMBRACO_MACRO macroAlias=\"umbracoContour.RazorRenderForm\" formGuid=\""+Model.selectedForm+"\" ></?UMBRACO_MACRO>", Model.Id)) 
      </div> 
     @if (configNode != null && configNode.Id > 0) 
     { 
      <address class="address-content"> 
       @if (custom.Library.NodeHasPropertyAndValue(configNode, "officeAddress")) 
       { 
        var addressDisplay = (custom.Library.NodeHasPropertyAndValue(configNode, "officeAddress2")) ? string.Format("{0}, {1}", configNode.officeAddress, configNode.officeAddress2) : configNode.officeAddress; 
        string locationString = string.Empty; 
        string cityDisplay = (custom.Library.NodeHasPropertyAndValue(configNode, "officeCity")) ? configNode.officeCity : string.Empty, 
        stateDisplay = (custom.Library.NodeHasPropertyAndValue(configNode, "officeState")) ? configNode.officeState : string.Empty, 
        zipDisplay = (custom.Library.NodeHasPropertyAndValue(configNode, "officeZip")) ? configNode.officeZip : string.Empty; 
        if (!string.IsNullOrEmpty(cityDisplay) && !string.IsNullOrEmpty(stateDisplay)) 
        { 
         locationString = string.Format("<br />{0}, {1} {2}", cityDisplay, stateDisplay, zipDisplay); 
        } 

        @addressDisplay@Html.Raw(locationString) 

       } 
       @if (custom.Library.NodeHasPropertyAndValue(configNode, "officePhone")) 
       { 
        <br /><i class="fa fa-phone"></i> @configNode.officePhone 
       } 
       @if (custom.Library.NodeHasPropertyAndValue(configNode, "officeFax")) 
       { 
        <br /><i class="fa fa-fax"></i> @configNode.officeFax 
       } 
       @if (custom.Library.NodeHasPropertyAndValue(configNode, "officeEmail")) 
       { 
        <br /><i class="fa fa-envelope"></i> <a href="mailto:@configNode.officeEmail">@configNode.officeEmail</a> 
       } 
      </address> 
     } 
     </div> 
    </div> 

    if (custom.Library.NodeHasPropertyAndValue(Model, "lowerText")) 
    { 
     @Html.Raw(Model.lowerText) 
    } 
} 

Answer 1

我要在這裏用一把umbraco術語乍一看似乎並不像Asp.Net MVC：

你需要問問自己 - 你需要使用宏嗎？如果將宏直接嵌入到模板中，那麼您可以使用PartialView（與標準MVC一樣在/ Views/Partials中找到），也可以直接將該佈局合併到Template中（位於/ Views中）。

你應該做的第一件事是查看一把umbraco文檔（取一把umbraco的版本註釋每個主題目標 - 還是有一把umbraco 4.10+具體內容有）：

https://our.umbraco.org/documentation/Reference/Templating/

關於拆分出來的邏輯：

代碼首款車型： 在Umbraco Ditto看一看 - 這是使用帶有一把umbraco IPublishedContent模型的方法;這裏有相當多的文檔，所以實現起來應該相當簡單。

控制器： 您可以使用ASP.Net MVC控制器來備份您的部分並提供模型;你需要從SurfaceControllers繼承，但：

https://our.umbraco.org/documentation/Reference/Routing/surface-controllers

渲染宏：不要使用@Html.Raw(umbraco.library.RenderMacroContent(..)) - 這幾乎是不推薦使用。現在執行此操作的標準方法是使用@Umbraco.RenderMacro()。所以這個：

@Html.Raw(umbraco.library.RenderMacroContent("<?UMBRACO_MACRO macroAlias=\"umbracoContour.RazorRenderForm\" formGuid=\""+Model.selectedForm+"\" ></?UMBRACO_MACRO>", Model.Id)) 

成爲

@Umbraco.RenderMacro("FormsRenderForm", new { FormGuid = Model.selectedForm }) 

最後的思考：

就個人而言，我不會用宏來生成內容，除非你需要允許後臺用戶嵌入功能添加到頁面並提供參數。如果情況並非如此，則最好直接在模板或部分視圖中實施佈局。