code-smell

    1熱度

    1回答

    我正在寫一個函數,在兩個不同的服務上調用兩個不同的方法。我調用了一個對service1執行操作的方法,並且在調用成功時(沒有拋出異常),我調用調用service2的方法。 我的代碼如下: def action1(param1: String, param2: String): Option[String] = { try{ //Check if service1 contain

    4熱度

    3回答

    我想使用switch語句,但是我無法在沒有重複代碼或使用隨附的if語句的情況下構造它。有沒有解決的辦法? 我有5個案例,除了其中一個之外,我想執行某些操作。所以用switch聲明,我可以這樣做: switch(x) { case A: foo(); break; case B: case C: case D: case E: bar(); brea

    0熱度

    3回答

    我創建了一個類,它是一個tuple包裝器,元組不支持項目突變。 我應該離開__setitem__和__delitem__的實現或實現像例如下面(因此屬於Refused Bequest代碼的氣味)?哪種方法更pythonic?在這種情況下,定製異常是否更好? def __setitem__(self, key, value): """ :raise: Always. :

    4熱度

    5回答

    鑑於這些代碼示例: 樣品1個 public class SomeClass { private static final int onlyUsedByMethodFoo = 1; // many lines of code public static void foo() { final String value = items[onlyUsedByMet

    12熱度

    5回答

    如果我有一個類以某種方式表示映射到我的數據庫中的特定表。 此類包含約30個屬性。 我創建了CRUD Methods。 並發現自己需要另一個(UPDATE)方法,它應該只更新兩個字段。 我應該用一個簡單的例子以良好的方式做什麼? 用我的存在的方法,填補了整個對象,並更新所有包括我打算兩個字段的字段?(無用功)與另一名 - 創建靜態方法(但我要保持我的方法名的Cuz它的表現力)!!並且需要兩個參數?

    2熱度

    4回答

    我有一類設置像這樣: class ClassToCheck(otherClass class2, nextClass class3) class otherClass(thisClass class4, otherThisClass class5, popularClass class6) class nextClass() (這僅僅是使用類的構造函數簽名爲例) 它是一個代碼味道有有許

    -1熱度

    1回答

    我試圖表明有人在這裏我怎麼好找聲納 工具... 後來我寫了一個小的Java項目,並定義了許多故意臭方法,那些 2完全相同(複製粘貼+)DO1和DO2 surprisenly,運行後Sonnar鏡頭,沒有重複的錯誤,也沒有警告...... public void do1() { for (int i = 0; i < 10; i++) { if (i != 0) {

    0熱度

    2回答

    說我寫的包裝某些第三方的API和我有這樣的方法: public function fetchSomeData() { $url = $this->makeUrl($someArgs); $response = $this->call($url); $this->save($response); return $response; } 這是一個代碼氣

    2熱度

    6回答

    所以我想知道什麼是最合適的方式來定義長度。現在,我使用的是這樣的 Coordinates(4.00 * Length.Millimeter, 5 * Length.Millimeter) 我想知道我是否應該提取到像尋找一種方法來做到這一點太這個 getMillimeter(double value) return value; Coordinates(getMillimeter(4.00),

    0熱度

    1回答

    在我加入的一個項目,最近我們有一個類,看起來像這樣: class MethodReturn { public int status = C_OK public string message = "" public string str_value = "" public int int_value = 0 public double dbl_val