refactoring

    0熱度

    1回答

    這是我的代碼。我需要幫助的所有的if else朝着稱爲validation()另一種方法,所以我可以優化它,而不是一遍又一遍使用相同的代碼。我該怎麼辦?我可以在doGet()方法中使用這種新方法嗎?還是我必須將它放在/之下? private static final long serialVersionUID = 1L; public Currency() { super(); }

    0熱度

    2回答

    我最近經歷了一次代碼審查,並堅決建議我將兩種方法合併爲一種。兩種方法都是相同的,除了每個方法的調用外,其中一個方法不需要參數。 方法#1 private void updateCache(List<CategoryObject> objectList) { ServiceApi serviceApi = getService(); if (serviceApi != null)

    1熱度

    3回答

    我使用Eclipse Oxygen 4.7.0.I20170612-0950和PyDev 5.8.0。 當我嘗試使用重構我收到以下錯誤消息: Could not initialize class org.apache.lucene.codesc.Codec$Holder ,當我嘗試使用搜索功能(STRG-H) An internal error occurred during: "PyDev In

    0熱度

    1回答

    我正在處理遺留代碼,其中有很多if/else。我可以使用條件分解和/或守衛語句等來清除它。可以使用的另一種方法是從分支中創建函數,並將這些函數作爲鏈並以第一個響應退出。我想知道如果使用功能鏈是一種矯枉過正?任何意見? 我的鏈看起來像下面。這是一個簡單的例子。 public class RequestStateHandler { private final List<Function<Re

    0熱度

    2回答

    我有兩個非常相似的方法。唯一的區別是在BindingList中使用的類型,但我無法弄清楚如何爲這兩種類型創建通用方法。我想我必須使用泛型類型才能使其工作,但我無法弄清楚。這兩種類型都有一個Title屬性。 private int GetObjectIndexFromTitle(string str, BindingList<MyClass1> list) { foreach (var

    0熱度

    1回答

    我現在正在學習React Native和React Navigator,我遇到了一些正在開發中的事情,我覺得我想實施DRY(不要重複自己)技術。 這裏是我的部分代碼: export default class App extends React.Component { render() { const MainNavigator = TabNavigator({ w

    0熱度

    2回答

    我在將以下代碼塊重構爲單獨的函數時遇到了問題。 socketHandler = (io) => { io.on('connection', (socket) => { socket.on('doLogin', data => { userService.getUserByName(data.uname) .then((doc) =>{ if

    0熱度

    1回答

    文件:login.component.spec.ts 茉莉花,噶及角4 代碼: // To set up import { By } from '@angular/platform-browser'; import { DebugElement } from '@angular/core'; import { TestBed, ComponentFixture, async } from '

    0熱度

    2回答

    我正在使用Visual Studio 2017 Professional v。15.0.0-RTW。 當按下ctrl + .上顯示任何未知類型的快速行動和重構菜單也不再顯示using [Namespace]像以前一樣.. 有沒有辦法解決這個問題?

    0熱度

    1回答

    我只是想知道什麼是最好的方法來重構這個聲明使用較少的condidtions?林reallt strucggling清理這一說法,而不會具有相同的功能,如果有人爲可以點我在正確的方向,我會很gratefull try { var errorProviders = new List<ErrorProvider>() { epEmail, epAlternative, epMobile, e