如何用很多參數重構類的方法？

Question

大家好我在ASP.NET MVC應用程序中的遺留代碼有問題， 在這段代碼中有一個業務邏輯層的Service類。這個類有20個參數的方法，這個方法使用這20個參數創建一個對象的實例。 如何重構此代碼，因爲這是創建的對象發生更改時出現的問題，並且需要更改方法中的參數。 該服務類用於控制器類和單元測試。 幫我重構此代碼 在此先感謝。

編輯其他信息：

我可以表明，該方法的

public Qualification CreateQualification(string achievableCode, string achievableTitle, 
     string accreditationRef, bool brandingPrefix, long brand, float guidedLearningHours, 
     int creditValue, long level, long type, long gradingType, long area, int subArea, 
     DateTime accreditationStartDate, DateTime accreditationEndDate, 
     DateTime lastCertDate, string nameOnCert, 
     long organisationId) 

我覺得這是需要申請KELY和Chevex的形式給出，例如我可以提取一些類簽名

一個將來自參數：

long area, int subArea 

其他

bool brandingPrefix, long brand, 

，並提取子類，我可以使用引入參數對象我正確理解後？

Answer 1

創建一個對象來保存這20個參數並將該對象傳遞給該方法。

例如：

public void MyMethod(MyArguments args) 
{ 
    // do stuff 
} 

編輯

雖然這種模式可能是一次重構有用的，如果你發現自己使用相同的參數在多個方法，考慮Chevex's答案。這是更好的方法。

Answer 2

您可能會嘗試識別參數中的相關數據，並將它們歸入自己的自定義對象中。例如，假設你有這樣的對象：

public class Person 
{ 
    public Person(string firstName, string lastName, int age, 
     string streetAddress, string city, string state, int zipCode) 
    { 
     this.FirstName = firstName; 
     this.LastName = lastName; 
     this.Age = age; 
     this.StreetAddress = streetAddress; 
     this.City = city; 
     this.State = state; 
     this.ZipCode = zipCode; 
    } 

    public string FirstName { get; set; } 
    public string LastName { get; set; } 
    public int Age { get; set; } 
    public string StreetAddress { get; set; } 
    public string City { get; set; } 
    public string State { get; set; } 
    public int ZipCode { get; set; } 
} 

嘗試重構這是兩個類，提取相關的地址信息到自己的類，然後將對象作爲原始對象的屬性：

public class Person 
{ 
    public Person(string firstName, string lastName, int age, Address address) 
    { 
     this.FirstName = firstName; 
     this.LastName = lastName; 
     this.Age = age; 
     this.Address = address; 
    } 

    public string FirstName { get; set; } 
    public string LastName { get; set; } 
    public int Age { get; set; } 
    public Address Address { get; set; } 
} 

public class Address 
{ 
    public Address(string streetAddress, string city, string state, int zipCode) 
    { 
     this.StreetAddress = streetAddress; 
     this.City = city; 
     this.State = state; 
     this.ZipCode = zipCode; 
    } 

    public string StreetAddress { get; set; } 
    public string City { get; set; } 
    public string State { get; set; } 
    public int ZipCode { get; set; } 
} 

沒有更多的信息我會說這是你最好的方法。

Answer 3

使用Builder模式

QualificationBuilder builder = new QualificationBuilder(); 
builder.setAchievableCode(achievableCode) 
     .setAchievableTitle(achievableTitle)... 
Qualification = builder.build();