🎤 😬 📗 तकनीक: वस्तुओं के बीच कार्य करना (एम। फाउलर रीफैक्टरिंग) 🏴󠁧󠁢󠁷󠁬󠁳󠁿 🌱 😱

कोड को चोक से शुरू करें
तकनीक: रचना के तरीके

पुस्तक Refactoring के लिए निरंतर, रिफैक्टिंग तकनीक । मौजूदा कोड मार्टिन फाउलर में सुधार।

सिंटैक्स C # में होगा, लेकिन मुख्य बात यह है कि विचार को समझना है, और इसका उपयोग किसी अन्य प्रोग्रामिंग भाषा में किया जा सकता है।

वस्तुओं के बीच गतिमान कार्य

एक विधि ले जाना ( कक्षा में समान शरीर के साथ एक नई विधि बनाना जो इसे अधिक बार उपयोग करता है )।
विधि एक ऐसे वर्ग में होनी चाहिए जो अपने विषय क्षेत्र को बेहतर ढंग से दर्शाती है।
से

class View { private Shop shop; private List<User> FilterUsers(List<User> users, decimal koef) { List<User> activeUsers = new List<User>(); foreach(User user in users) { if(shop.IsActiveUser(user, koef)) { activeUsers.Add(user); } } return activeUsers; } } class Shop { public bool IsActiveUser(User user, decimal koef) { decimal rate = 1; if(koef > 1000) { rate = koef * 1.75; } return user.Boss || user.HasDiscount() && rate > 10; } } class User { private bool boss; public bool Boss { get { return boss; } } public bool HasDiscount() { // some condition } }

को

 class View { private Shop shop; private List<User> FilterUsers(List<User> users, decimal koef) { List<User> activeUsers = new List<User>(); foreach(User user in users) { if(user.IsActiveUser(koef)) { activeUsers.Add(user); } } return activeUsers; } } class Shop { } class User { private bool boss; public bool Boss { get { return boss; } } public bool HasDiscount() { // some condition } public bool IsActiveUser(decimal koef) { decimal rate = 1; if(koef > 1000) { rate = koef * 1,75; } return Boss || HasDiscount() && rate > 10; } }

एक फ़ील्ड चलना ( लक्ष्य वर्ग में एक नया फ़ील्ड बनाना )।
क्षेत्र एक ऐसे वर्ग में होना चाहिए जो अपने विषय क्षेत्र को बेहतर ढंग से दर्शाता है
से

 class Shop { private decimal rate; private DiscountType discountType; public Shop(DiscountType discountType, decimal rate) { this.rate = rate; this.discountType = discountType; } public decimal Rate { get{ return rate; } set{ rate = value; } } public bool HasDiscount(DateTime from, DateTime to) { // some calculation of flag return discountType.IsValid(rate) && flag; } public decimal GetDiscount(decimal amount, bool fullSum) { // some calculation of koef return discountType.GetAmount(rate, fullSum) * koef; } } class DiscountType { public bool IsValid(decimal rate) { // some calculation using rate } public decimal GetAmount(decimal rate, bool fullSum) { // some calculation using rate } }

को

 class Shop { private DiscountType discountType; public Shop(DiscountType discountType) { this.discountType = discountType; } public bool HasDiscount(DateTime from, DateTime to) { // some calculation of flag return discountType.IsValid() && flag; } public decimal GetDiscount(decimal amount, bool fullSum) { // some calculation of koef return discountType.GetAmount() * koef; } } class DiscountType { private decimal rate; public DiscountType(decimal rate) { this.rate = rate; } public decimal Rate { get{ return rate; } set{ rate = value; } } public bool IsValid() { // some calculation using rate } public decimal GetAmount(bool fullSum) { // some calculation using rate } }

एक कक्षा को हाइलाइट करना ( एक नया वर्ग बनाएं और पुराने वर्ग से नए क्षेत्र में फ़ील्ड और विधियाँ स्थानांतरित करें )।
कक्षा में अपना डेटा मॉडल और इसके साथ काम करने के तरीके होने चाहिए, अन्यथा यह एक ईश्वर वस्तु में बदल जाता है।
से

 class User { private string name; private string street; private string city; public string Name { get{ return name; } set{ name = value; } } public string Street { get{ return street; } set{ city = value; } } public string City { get{ return city; } set{ city = value; } } public string GetAddressInfo() { return string.Format("{0} \ {1}", city, street); } }

को

 class User { private string name; private Address address; public string Name { get{ return name; } set{ name = value; } } public string GetAddressInfo() { return address.GetFullAddress(); } } class Address { private string city; private string street; public string Street { get{ return street; } set{ city = value; } } public string City { get{ return city; } set{ city = value; } } public string GetFullAddress() { return string.Format("Adress: {0} \ {1}", city, street); } }

एक वर्ग को सम्मिलित करना ( सभी कार्यों को दूसरे वर्ग में ले जाना और मूल को हटाना )।
एक वर्ग के पास बहुत कम कार्य हैं।
से

 class User { private string name; private Address address; public string Name { get{ return name; } set{ name = value; } } public Address GetAddress() { return address } } class Address { private string areaCode; private string country; public string AreaCode { get{ return areaCode; } set{ areaCode = value; } } public string Country { get{ return country; } set{ country = value; } } }

को

 class User { private string name; private string areaCode; private string country; public string AreaCode { get{ return areaCode; } set{ areaCode = value; } } public string Country { get{ return country; } set{ country = value; } } public string Name { get{ return name; } set{ name = value; } } }

प्रतिनिधिमंडल को छिपाना ( प्रतिनिधि को छिपाने के तरीके बनाना )।
एनकैप्सुलेशन, सिस्टम के कुछ हिस्से।
से

 class View { private void Init() { // some code User manager = currentUser.Office.Manager; } } class User { private Department department; public Department Office { get{ return department; } set{ department = value; } } } class Department { private User manager; public Department(User manager) { this.manager = manager; } public User Manager { get{ return manager; } } }

को

 class View { private void Init() { // some code User manager = currentUser.Manager; } } class User { private Department department; public User Manager { get{ return department.Manager; } } } class Department { private User manager; public Department(User manager) { this.manager = manager; } public User Manager { get{ return manager; } } }

ब्रोकर को हटाना ( ग्राहक को प्रतिनिधि से सीधे संपर्क करें )।
साधारण प्रतिनिधिमंडल के साथ वर्ग बहुत व्यस्त है।
से

 class View { private void Init() { // some code User manager = currentUser.Manager; decimal rate = currentUser.Rate; } } class User { private Department department; private UserType userType; public User Manager { get{ return department.Manager; } } public decimal Rate { get{ return userType.Rate; } } } class Department { private User manager; public User Manager { get{ return manager; } } } class UserType { private decimal rate; public decimal Rate { get{ return rate; } } }

को

 class View { private void Init() { // some code User manager = currentUser.Office.Manager; decimal rate = currentUser.Type.Rate; } } class User { private Department department; private UserType userType; public Department Office { get{ return department; } } public UserType Type { get{ return userType; } } } class Department { private User manager; public User Manager { get{ return manager; } } } class UserType { private decimal rate; public decimal Rate { get{ return rate; } } }

एक बाहरी विधि का परिचय ( क्लाइंट में, एक विधि है जिसमें सर्वर वर्ग को 1 पैरामीटर के रूप में पारित किया जाता है )।
एक विधि शुरू करने की आवश्यकता है, लेकिन कक्षा को संशोधित करने की कोई संभावना नहीं है।
से
```
 class View { private string GetUserName() { // some code return "\"" + userName + "\""; } private string GetCompanyName() { // some code return "\"" + companyName + "\""; } } 
```
को
```
 class View { private string GetUserName() { // some code return userName.InDoubleQuote(); } private string GetCompanyName() { // some code return companyName.InDoubleQuote(); } } static class StringExtension { public static string InDoubleQuote(this string str) { return "\"" + str + "\""; } } 
```
एक स्थानीय विस्तार का परिचय ( वर्ग के लिए एक आवरण बनाएं (या विरासत का उपयोग करके) और आवश्यक तरीकों से इसे पूरक करें )।
विधियों को पेश किया जाना चाहिए, लेकिन कक्षा को संशोधित करने का कोई तरीका नहीं है।
से
```
 sealed class Account { public decimal CalculateSum() { // calculation summ return summ; } } 
```
को
```
 class ImportantAccount { private Account account; public decimal CalculateMaxSum() { // calculation rate return account.CalculateSum() * rate; } } 
```

हितों का टकराव (समीक्षा के भाग के रूप में)।

एक नियम के रूप में, समीक्षक और कॉमिट-इरेट के बीच उत्पन्न होने वाले सभी विवाद / असहमति को निम्नलिखित में विभाजित किया जा सकता है:

सहकर्मी उपयुक्त समाधान के विकल्प पर चर्चा करते हैं कार्यान्वयन पर असहमति पैदा हुई। सही संचार के ढांचे के भीतर, वे समझते हैं कि वे एक सामान्य बात कर रहे हैं और आपको केवल एक समाधान (10%) चुनने की आवश्यकता है।
सहकर्मियों को चैट करने की आवश्यकता है: शुक्रवार, एक कठिन सप्ताह या बस पर्याप्त संचार नहीं। ऐसे लोगों की एक श्रेणी है, जिन्हें सिर्फ दरार की जरूरत है। सिद्धांत रूप में, यह सामान्य ज्ञान के बिना नहीं है, क्योंकि संचार और चर्चा महान है (10%)।
विवाद जो नए नियमों को जन्म देते हैं। उदाहरण के लिए, जब संस्करण कीवर्ड .NET में दिखाई दिया, तो सहकर्मियों में से एक ने हर जगह इसका उपयोग करना शुरू कर दिया, किसी ने दूसरे तरीके से। इसलिए, समीक्षा के दौरान, विवाद उत्पन्न हुए, जो बेहतर है। नतीजतन, कुछ चर्चाओं के बाद, एक निर्णय किया गया (10%)।
```
 //     ,    ,    var var users = new List<User>(); var departments = user.Departmens.Cast<Department>(); //  var users = departments.Select(d => d.User).Where(u => !u.Deleted); var departments = service.GetNotDeletedDepartments(); 
```
किसके निर्णय पर गर्म बहस बेहतर है। और अगर प्रोग्रामर समान स्तर पर हैं, तो जुनून भड़क सकता है। इसके अलावा, इन सभी विवादों, केवल समय की बर्बादी और खुद को शांत साबित करने की इच्छा (70%)। यहां मैं सलाह देना चाहता हूं, अगर आप इस तरह के विवाद में फंस गए हैं, तो कभी भी एक व्यक्ति के ऊपर न जाएं। और वाक्यांश "आपका निर्णय फिट नहीं है," इसे बदलने के लिए बेहतर है "निर्णय पूरी तरह से सही नहीं है और अपनी बात समझाएं । " यानी जोर निर्णय पर चर्चा पर होना चाहिए, न कि किसी सहकर्मी के कौशल पर।

मजेदार मामले:
1) समीक्षा-एर - "आपको कोड ठीक करने की आवश्यकता है"
कमेटी - "आपको पसंद नहीं है और आप शासन करते हैं" / "आपने आपको और सही पाया"

2) या कैसे सही ढंग से
```
 void Method() { ... if(flag) koef = GetMaxRate(); ... } 
```
या
```
 void Method() { ... if(flag) { koef = GetMaxRate(); } ... } 
```

मैं आगे "तकनीक: डेटा संगठन" जारी रखने की उम्मीद करता हूं।

तकनीक: वस्तुओं के बीच कार्य करना (एम। फाउलर रीफैक्टरिंग)

वस्तुओं के बीच गतिमान कार्य

हितों का टकराव (समीक्षा के भाग के रूप में)।

More articles: