溫馨提示×
您好，登錄后才能下訂單哦！
點擊 登錄注冊 即表示同意《億速云用戶服務條款》
您好，登錄后才能下訂單哦！
基本類型偏執(Primitive Obsession)
- 使用基本類型而不是小對象來實現簡單任務（例如貨幣、范圍、電話號碼字符串等）。
- 使用常量編碼信息（例如一個用于引用管理員權限的常量
USER_ADMIN_ROLE = 1）。- 使用字符串常量作為字段名在數組中使用。
類似其他大部分壞味道，基本類型偏執誕生于類初建的時候。一開始，可能只是不多的字段，隨著表示的特性越來越多，基本數據類型字段也越來越多。
基本類型常常被用于表示模型的類型。你有一組數字或字符串用來表示某個實體。
還有一個場景：在模擬場景，大量的字符串常量被用于數組的索引。
大多數編程語言都支持基本數據類型和結構類型（類、結構體等）。結構類型允許程序員將基本數據類型組織起來，以代表某一事物的模型。
基本數據類型可以看成是機構類型的積木塊。當基本數據類型數量成規模后，將它們有組織地結合起來，可以更方便的管理這些數據。
以類取代類型碼(Replace Type Code with Class)
。
引入參數對象(Introduce Parameter Object)
或
保持對象完整(Preserve Whole Object)
。
以類取代類型碼(Replace Type Code with Class)
將它替換掉。如果你有與類型碼相關的條件表達式，可運用
以子類取代類型碼(Replace Type Code with Subclass)
或
以狀態/策略模式取代類型碼(Replace Type Code with State/Strategy)
加以處理。
以對象取代數組(Replace Array with Object)
。
問題
類之中有一個數值類型碼，但它并不影響類的行為。
解決
以一個新的類替換該數值類型碼。
問題
某些參數總是很自然地同時出現。
解決
以一個對象來取代這些參數。
問題
你從某個對象中取出若干值，將它們作為某一次函數調用時的參數。
int low = daysTempRange.getLow(); int high = daysTempRange.getHigh(); boolean withinPlan = plan.withinRange(low, high);
解決
改為傳遞整個對象。
boolean withinPlan = plan.withinRange(daysTempRange);
問題
你有一個不可變的類型碼，它會影響類的行為。
解決
以子類取代這個類型碼。
問題
你有一個類型碼，它會影響類的行為，但你無法通過繼承消除它。
解決
以狀態對象取代類型碼。
問題
你有一個數組，其中的元素各自代表不同的東西。
String[] row = new String[3]; row[0] = "Liverpool"; row[1] = "15";
解決
以對象替換數組。對于數組中的每個元素，以一個字段來表示。
Performance row = new Performance();
row.setName("Liverpool");
row.setWins("15");
數據泥團(Data Clumps)
有時，代碼的不同部分包含相同的變量組（例如用于連接到數據庫的參數）。這些綁在一起出現的數據應該擁有自己的對象。
通常，數據泥團的出現時因為糟糕的編程結構或“復制-粘貼式編程”。
有一個判斷是否是數據泥團的好辦法：刪掉眾多數據中的一項。這么做，其他數據有沒有因而失去意義？如果它們不再有意義，這就是個明確的信號：你應該為它們產生一個新的對象。
提煉類(Extract Class)
將它們提煉到一個獨立對象中。
引入參數對象(Introduce Parameter Object)
將它們組織成一個類。
保持對象完整(Preserve Whole Object)
將整個數據對象傳入到函數中。
問題
某個類做了不止一件事。
解決
建立一個新類，將相關的字段和函數從舊類搬移到新類。
問題
某些參數總是很自然地同時出現。
解決
以一個對象來取代這些參數。
問題
你從某個對象中取出若干值，將它們作為某一次函數調用時的參數。
int low = daysTempRange.getLow(); int high = daysTempRange.getHigh(); boolean withinPlan = plan.withinRange(low, high);
解決
改為傳遞整個對象。
boolean withinPlan = plan.withinRange(daysTempRange);
過大的類(Large Class)
一個類含有過多字段、函數、代碼行。
類通常一開始很小，但是隨著程序的增長而逐漸膨脹。
類似于過長函數，程序員通常覺得在一個現存類中添加新特性比創建一個新的類要容易。
設計模式中有一條重要原則：職責單一原則。一個類應該只賦予它一個職責。如果它所承擔的職責太多，就該考慮為它減減負。
提煉類(Extract Class)
。
提煉子類(Extract Subclass)
。
提煉接口(Extract Interface)
。
復制被監視數據(Duplicate Observed Data)
可以告訴你怎么做。
問題
某個類做了不止一件事。
解決
建立一個新類，將相關的字段和函數從舊類搬移到新類。
問題
一個類中有些特性僅用于特定場景。
解決
創建一個子類，并將用于特殊場景的特性置入其中。
問題
多個客戶端使用一個類部分相同的函數。另一個場景是兩個類中的部分函數相同。
解決
移動相同的部分函數到接口中。
問題
如果存儲在類中的數據是負責 GUI 的。
解決
一個比較好的方法是將負責 GUI 的數據放入一個獨立的類，以確保 GUI 數據與域類之間的連接和同步。
過長函數(Long Method)
一個函數含有太多行代碼。一般來說，任何函數超過 10 行時，你就可以考慮是不是過長了。 函數中的代碼行數原則上不要超過 100 行。
通常情況下，創建一個新函數的難度要大于添加功能到一個已存在的函數。大部分人都覺得：“我就添加這么兩行代碼，為此新建一個函數實在是小題大做了。”于是，張三加兩行，李四加兩行，王五加兩行。。。函數日益龐大，最終爛的像一鍋漿糊，再也沒人能完全看懂了。于是大家就更不敢輕易動這個函數了，只能惡性循環的往其中添加代碼。所以，如果你看到一個超過 200 行的函數，通常都是多個程序員東拼西湊出來的。
一個很好的技巧是： 尋找注釋 。添加注釋，一般有這么幾個原因：代碼邏輯較為晦澀或復雜；這段代碼功能相對獨立；特殊處理。 如果代碼前方有一行注釋，就是在提醒你：可以將這段代碼替換成一個函數，而且可以在注釋的基礎上給這個函數命名。如果函數有一個描述恰當的名字，就不需要去看內部代碼究竟是如何實現的。就算只有一行代碼，如果它需要以注釋來說明，那也值得將它提煉到獨立函數中。
提煉函數(Extract Method)
。
以查詢取代臨時變量(Replace Temp with Query)
，
引入參數對象(Introduce Parameter Object)
或
保持對象完整(Preserve Whole Object)
。
以函數對象取代函數(Replace Method with Method Object)
嘗試移動整個函數到一個獨立的對象中。
分解條件表達式(Decompose Conditional)
。至于循環，應該使用
提煉函數(Extract Method)
將循環和其內的代碼提煉到獨立函數中。
是否像許多人說的那樣，增加函數的數量會影響性能？在幾乎絕大多數情況下，這種影響是可以忽略不計，所以不用擔心。 此外，現在有了清晰和易讀的代碼，在需要的時候，你將更容易找到真正有效的函數來重組代碼和提高性能。
問題
你有一段代碼可以組織在一起。
void printOwing() {
printBanner();
//print details
System.out.println("name: " + name);
System.out.println("amount: " + getOutstanding());
}
解決
移動這段代碼到一個新的函數中，使用函數的調用來替代老代碼。
void printOwing() {
printBanner();
printDetails(getOutstanding());
}
void printDetails(double outstanding) {
System.out.println("name: " + name);
System.out.println("amount: " + outstanding);
}
問題
將表達式的結果放在局部變量中，然后在代碼中使用。
double calculateTotal() {
double basePrice = quantity * itemPrice;
if (basePrice > 1000) {
return basePrice * 0.95;
}
else {
return basePrice * 0.98;
}
}
解決
將整個表達式移動到一個獨立的函數中并返回結果。使用查詢函數來替代使用變量。如果需要，可以在其他函數中合并新函數。
double calculateTotal() {
double basePrice = quantity * itemPrice;
if (basePrice > 1000) {
return basePrice * 0.95;
}
else {
return basePrice * 0.98;
}
}
問題
某些參數總是很自然地同時出現。
解決
以一個對象來取代這些參數。
問題
你從某個對象中取出若干值，將它們作為某一次函數調用時的參數。
int low = daysTempRange.getLow(); int high = daysTempRange.getHigh(); boolean withinPlan = plan.withinRange(low, high);
解決
改為傳遞整個對象。
boolean withinPlan = plan.withinRange(daysTempRange);
問題
你有一個過長函數，它的局部變量交織在一起，以致于你無法應用提煉函數(Extract Method) 。
class Order {
//...
public double price() {
double primaryBasePrice;
double secondaryBasePrice;
double tertiaryBasePrice;
// long computation.
//...
}
}
解決
將函數移到一個獨立的類中，使得局部變量成了這個類的字段。然后，你可以將函數分割成這個類中的多個函數。
class Order {
//...
public double price() {
return new PriceCalculator(this).compute();
}
}
class PriceCalculator {
private double primaryBasePrice;
private double secondaryBasePrice;
private double tertiaryBasePrice;
public PriceCalculator(Order order) {
// copy relevant information from order object.
//...
}
public double compute() {
// long computation.
//...
}
}
問題
你有復雜的條件表達式。
if (date.before(SUMMER_START) || date.after(SUMMER_END)) {
charge = quantity * winterRate + winterServiceCharge;
}
else {
charge = quantity * summerRate;
}
解決
根據條件分支將整個條件表達式分解成幾個函數。
if (notSummer(date)) {
charge = winterCharge(quantity);
}
else {
charge = summerCharge(quantity);
}
過長參數列(Long Parameter List)
一個函數有超過 3、4 個入參。
過長參數列可能是將多個算法并到一個函數中時發生的。函數中的入參可以用來控制最終選用哪個算法去執行。
過長參數列也可能是解耦類之間依賴關系時的副產品。例如，用于創建函數中所需的特定對象的代碼已從函數移動到調用函數的代碼處，但創建的對象是作為參數傳遞到函數中。因此，原始類不再知道對象之間的關系，并且依賴性也已經減少。但是如果創建的這些對象，每一個都將需要它自己的參數，這意味著過長參數列。
太長的參數列難以理解，太多參數會造成前后不一致、不易使用，而且一旦需要更多數據，就不得不修改它。
以函數取代參數(Replace Parameter with Methods)
。在這里，，“已有的對象”可能是函數所屬類里的一個字段，也可能是另一個參數。
保持對象完整(Preserve Whole Object)
將來自同一對象的一堆數據收集起來，并以該對象替換它們。
引入參數對象(Introduce Parameter Object)
為它們制造出一個“參數對象”。
問題
對象調用某個函數，并將所得結果作為參數，傳遞給另一個函數。而接受該參數的函數本身也能夠調用前一個函數。
int basePrice = quantity * itemPrice; double seasonDiscount = this.getSeasonalDiscount(); double fees = this.getFees(); double finalPrice = discountedPrice(basePrice, seasonDiscount, fees);
解決
讓參數接受者去除該項參數，并直接調用前一個函數。
int basePrice = quantity * itemPrice; double finalPrice = discountedPrice(basePrice);
問題
你從某個對象中取出若干值，將它們作為某一次函數調用時的參數。
int low = daysTempRange.getLow(); int high = daysTempRange.getHigh(); boolean withinPlan = plan.withinRange(low, high);
解決
改為傳遞整個對象。
boolean withinPlan = plan.withinRange(daysTempRange);
問題
某些參數總是很自然地同時出現。
解決
以一個對象來取代這些參數。
免責聲明：本站發布的內容（圖片、視頻和文字）以原創、轉載和分享為主，文章觀點不代表本網站立場，如果涉及侵權請聯系站長郵箱：is@yisu.com進行舉報，并提供相關證據，一經查實，將立刻刪除涉嫌侵權內容。
