c# - How can I clean up this ugly if statement? -

-

I have the following ugly statement that is part of a square that is drawn from the IOC container:

< Pre-> Protected Virtual Zero ExecuteSourceControlGet (IBILMMadata Build Metadata, IPCertTTTT) {If ((BuildmateDataDepository Elementalist! = Faucet) & amp; (BuildMetadata RepositoryArrayList.Count & gt; 0)} {componentTree.DeleteWorkingDirectory (); Forex (version repository element build metadata. Repository elementalist) {repositoryElement.PrepareRepository (get components, threads). Export (); }} If ((buildMetaData.ExportList! = Null) & amp; (Buildmetda.export list.account> 0)) {var initialise = true; Foreach (buildMetaData.ExportList in the above sourceControl) {log.InfoFormat ("\ nThere is a problem getting the horn {0} \ n \ n." .TowUpper (), sourceControl.Url); Get.from (sourceControl) .ExportTo (componentTree, sourceControl.ExportPath, initialize); Initialise = false; }} Log.InfoFormat ("\ \ horn is being brought {0}. \ N \ n" .upper (), buildMetaData.SourceControl.Url); Get.From (buildMetaData.SourceControl) .ExportTo (componentTree); }

If there is a subclass for every condition, then my general view of destroying it.

What's different about this example:

  1. The class in which this method has been drawn from the IOC container.
  2. If the statement is on or not, then I would like to argue between 2.

Any advice is very welcome.

about using two removal methods , and if revert To be a guard: 

  Protected Virtual Zero ExecuteSourceControlGet (IBuildMetaData buildMetaData, IPackageTree componentTree) {ExecuteRepositoryElementList (buildMetaData.RepositoryElementList, componentTree); ExecuteExportList (BuildmateData.exportalist, componentTree); Log.InfoFormat ("\ \ horn is fetching {0}. \ N \ n" .upper (), buildMetaData.SourceControl.Url); Get.From (buildMetaData.SourceControl) .ExportTo (componentTree); } Private Zero ExecuteRepositoryElementList (RepositoryElementList repositoryElements, IPackageTree componentTree) {// Guard: No Elements if (repositoryElements == null} repositoryElements.Count == 0 Return; ComponentTree.DeleteWorkingDirectory (); Forrest (different repository elements in repository elements) {repositoryElement.PrepareRepository (get component, tree). Export (); }} Private Zero ExecuteExportList (ExportList exportList, IPackageTree componentTree) {// Guard: No Elements if (exportList == null} exportList.Count == 0 Return; Var initialise = true; Foreach (various source controls in the export source) {log. Informat ("\ nhunting the string {0}. \ N \ n" .User (source), source control. Url); Get.from (sourceControl) .ExportTo (componentTree, sourceControl.ExportPath, initialize); Initialise = false; }}

BTW: Two methods must be fixed with the correct type IBuildMetaData.RepositoryElementList and IBuildMetaData.ExportList .


Comments

Post a Comment

Popular posts from this blog

c++ - Linux and clipboard -

-
After selecting the copies of the buffer in Linux, after the text, we can click and paste the button between the mouse. I think there is a special buffer for this, I want to use it. Programming Language: C ++ Your Library: Qt Thanks. There is just one more correct answer than Paul Dixon, which answers your needs: / P > QClipboard * clipboard = QApplication :: clipboard (); CastString Selected Text = Clipboard-> Text (QClipboard :: Selection);
Read more

What is expire header and how to achive them in ASP.NET and PHP? -

-
I have examined the performance statistics of my website today. I have received a warning (or error may occur) that is given below Add Terminal Headers Here are 15 static components without a 15-futures end date * (no end) Http://www.example.com/video/css/global.css * (no time limit) http://www.example.com/video/js/global.js * (no expiration ) Http: //www.example.com/video/images/main-bg.png what it means and how to achieve it both in PHP and ASP.Net. I am on a shared hosting server, so please tell me some ways to use the code, because I can not make any modifications at the end of the server. If I expire the header, then there is no chance that if I make changes to the CSS, then the user will not be able to remove them immediately because CSS and other files have a fixed time limit (1 month, week ) Are cached for. Is Is there any harm to using PHP? $ time = time () + 3 * 24 * 60 * 60; // 3 day headings ('End:'. GMDAT ('D, D MYH: i: s \ g \ m \ t',
Read more

sql server - How can I determine which of my SQL 2005 statistics are unused? -

-
After a few years, one of my biggest databases has deposited 73 figures out of its largest tables. With Indexes, I can run many types of reports and queries, how often / heavily specific indexes are used. What is an equivalent for data? We are running SQL 2005. To make statistics automatically managed and honest, I would like to "update statistics", "create statistics" and There is no information about any management except "drop statistics" orders. In theory, UPDATE should handle the addition and removal of statistical data. In addition, I've never heard about the amount of storage or storage taking in large amounts, so I'm not sure whether there is any reason for the alarm or not. If you are seeing a potential problem, then I would suggest running an update before, if it does not clean it, it is leaving and then it appears to be safe operation (because it is the only query optimizer Is for). I will not leave "left" in the
Read more