code-smell

    9熱度

    1回答

    作爲一般規則,我喜歡使用基於構造函數的依賴注入,但最近我正在研究一個依賴於其他4個類的類。因爲很長的參數列表很難閱讀,所以我用4個setter替換了4個參數的構造函數。 當我向一位同事提到這件事時,他認爲這本身就是一種代碼味道。他建議我「分手」這堂課。 這個班本身已經比較小了,它恰好使用多個協作者來完成大部分工作。它由一個簡短的(~12行,包括空白)方法組成,可以調用4個協作者。你同意還是不同意這

    0熱度

    2回答

    這似乎是一個非常有效的方法做錯誤處理,但我想知道怎麼做是正確的,如果這是臭: Class Widget ... Public Function IsValid() As Boolean If (some condition isnt met) Then Throw New ApplicationException("Error message")

    1熱度

    3回答

    啊,我收到了一個警告,說我的問題'顯得主觀,很可能會被關閉'。但我認爲這是關於建築設計的一個相關且重要的問題。 我的所有編程年度都被告知,在UI代碼中直接創建並執行的SQL語句(例如ASP.NET頁面代碼隱藏)是一種巨大的代碼異味,數據訪問和表示問題應該在相反的端點應用程序的圖層。 現在我有一個閃亮的數據訪問層,使用NHibernate實體通過Repository模式。我發現自己直接在我的UI代碼

    3熱度

    2回答

    我正在分析大量的多線程代碼,我看到了這麼多的鎖。一些方法將有兩個鎖在這樣的一行: ClassA::foo() { lockA.lock(); lockB.lock(); ...//do some stuff lockB.unlock(); lockA.unlock(); } 我的問題是很一般(我不能提供實際的代碼)。這是代碼味道嗎?這通

    2熱度

    2回答

    構造函數中的邏輯是代碼異味。然而,它是一個構造函數中代碼的數量是壞的還是代碼的類型(是否有某些代碼可以在構造函數中使用?)? 感謝

    4熱度

    7回答

    我已經開始注意到ASP.NET開發中的反模式。它讓我感到困擾,因爲它感覺像保持良好設計的做法是正確的,但同時它味道錯了。 問題是這樣的:我們有一個多層應用程序,底層是一個處理調用服務的類,它提供給我們數據。在上面是可能轉換,操作和檢查數據的一層類。以上是ASP.NET頁面。 在許多情況下,從服務層的方法不前,在視圖上會需要的任何變化,所以模型只是一條直線穿過,如: public List<IDat

    4熱度

    2回答

    在我們的2007年德爾福應用程序,我們使用了大量以下構造 FdmBasic:=TdmBasicData(FindOwnerClass(AOwner,TdmBasicData)); 的FindOwnerClass行進的電流分量的業主層次向上查找特定類(在本例中TdmBasicData)。結果對象存儲在Field變量FdmBasic中。我們主要使用它來傳遞數據模塊。 示例: 生成報告時,結果數據被

    1熱度

    2回答

    我有if ($this->_userexists()) return false;兩個功能。他們變成了 function exitifuserexists() { if ($this->_userexists()) return false; } 它退出本身,但必須退出調用者。如何使兩個「如果這樣」與兩個「函數調用」?

    1熱度

    1回答

    我有一個名爲MapBuilder<T>的課程,它在內部使用Dictionary<PropertyInfo,string> 該類用於快速構建將代理哪些屬性的映射。 類看起來像這樣:: public class MapBuilder<T>{ private Dictionary<PropertyInfo, string> m_Map = new Dictionary<PropertyInfo

    3熱度

    2回答

    我不打算進入哪一個被調用的時間和原因。 (已經有很多了) 既然我們不能依靠viewDidUnload在dealloc之前被調用,我發現自己在這些方法之間有很多重複的代碼。 - (void)viewDidUnload { [super viewDidUnload]; self.foo = nil; self.bar = nil; } - (void)deal