依赖注入 - 我是否正确地重构了代码？

我在准备测试时正在尝试这个问题。根据我的理解，这是我的最佳答案，但我觉得我可能错过了一些重要的东西。我认为我已经改变了导航器的责任，但我看不到更好的方式。依赖注入 - 我是否正确地重构了代码？

的问题是：

public class Navigator 
{ 
    private Route theRoute; 

    public Navigator(UserInterface ui) { 
     String destination = ui.getDestEntry().getText(); 
     theRoute = new Route(GPS.getLocation(), destination); 
     theRoute.calculateRoute(); 
    } 

    public void display() { 
     MapView theMap = new MapView(); 
     theMap.plot(theRoute); 
    } 
} 

public class GPS { 
     public static String getLocation() { ... } 
    } 

“{ ... }” stands for an algorithm that we do not need to examine, for our purposes.

重构导航和GPS类，以符合依赖注入模式。不要改变他们的基本职责。

（a）您重构导航和GPS类：（你会对真正的考验更多的空间）

（二）喷油器的代码（就像语句序列）

我答案：

的（a）

public class Navigator { 
    private Route theRoute; 
    private MapView theMap; 

    public Navigator (Route inRoute) { 
     theRoute = inRoute; 
     theRoute.calculateRoute(); 
    } 

    public void display(MapView inMap) { 
     theMap = inMap; 
     theMap.plot(theRoute); 
    } 
} 

public class GPS { 
    public GPS(); //constructor 

    public String getLocation(){...} 
}

（b）

喷油器代码：

UserInterface ui = new UserInterface; 
String destination = new String(ui.getDestEntry().getText()); 
GPS gps = new GPS; 
Route theRoute = new Route (GPS.getLocation(), destination); 
new Navigator(theRoute);

来源

2013-10-27 Sarah

可能会更好。

public class Navigator { 

    private final Route theRoute; 
    private final MapView theMap; 

    public Navigator(Route inRoute, MapView theMap) { 
    theRoute = inRoute; 
    this.theMap = theMap; 
    } 

    public void setup() { 
    theRoute.calculateRoute(); 
    } 

    public void display() { 
    theMap.plot(theRoute); 
    } 

}

B）你的喷油器的代码是不完整的或错误的

来源

2013-10-27 08:38:17 MariuszS

谢谢，我忘了打电话给注册导航员r代码，那你的意思是不完整的？ – Sarah

嗯，是:) Navigator nav = new Navigator（theRoute）; nav.display（地图）; – MariuszS

注射器代码在这里只有两行：RouteRoute = new Route（GPS.getLocation（），destination）; 新导航（theRoute）; 其他代码与喷油器无关 – MariuszS

的Navigator具有依赖性GPS，所以你需要添加属性导航

public class Navigator 
{ 
    private GPS gps; 
    private UserInterface ui; 

    public Navigator(UserInterface ui, GPS gps) { 
     this.ui = ui; 
     this.gps = gps; 
    } 

    public void display() { 
     String destination = ui.getDestEntry().getText(); 
     Route theRoute = new Route(gps.getLocation(), destination); 
     theRoute.calculateRoute(); 
     MapView theMap = new MapView(); 
     theMap.plot(theRoute); 
    } 
} 

public class GPS { 
    public String getLocation() { ... } 
}

来源

2013-10-27 09:02:03

没有构造函数调用（Route theRoute = new Route（gps.getLocation（），destination）违反了依赖注入原则吗？ – Sarah

这取决于它是否具有依赖性，在你的问题是你声明了对GPS的依赖关系，如果它的依赖性较小，代码就更清楚了。 – 2013-10-27 09:21:46

我的C＃重构变异的样子：

public class ClientCode 
{ 
    void DoNavigations(IDestinationEntry ui, IGPS gps) 
    { 
     String destination = ui.getDestEntry().getText(); 
     IRoute theRoute = new Route(gps.getLocation(), destination); 
     INavigator nv = new Navigator(theRoute); 
     nv.display(); 
    } 
} 

public class Navigator : INavigator 
{ 
    private IRoute _theRoute; 

    public Navigator(IRoute theRoute) 
    { 
     _theRoute = theRoute; 
     _theRoute.calculateRoute(); 
    } 

    public void display() 
    { 
     MapView theMap = new MapView(); 
     theMap.plot(_theRoute); 
    } 
} 

public interface IGPS 
{ 
    string getLocation(); 
} 

public interface INavigator 
{ 
    void display(); 
} 

public interface IDestinationEntry 
{ 
    DestinationEntry getDestEntry(); 
} 

public interface IRoute 
{ 
    void calculateRoute(); 
}

来源

2013-10-27 14:24:16 zzfima

依赖注入 - 我是否正确地重构了代码？

回答

相关问题