Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Duplicated StrategyManager in ExampleLSPReplanning? #40

Open
kt86 opened this issue Jul 28, 2022 · 0 comments
Open

Duplicated StrategyManager in ExampleLSPReplanning? #40

kt86 opened this issue Jul 28, 2022 · 0 comments
Labels
question Further information is requested

Comments

@kt86
Copy link
Contributor

kt86 commented Jul 28, 2022

@kainagel : Isn't it somehow duplicated / done twice: L203-210 vs 214-219?

@Override public void install(){
bind( LSPStrategyManager.class ).toProvider( new Provider<LSPStrategyManager>(){
@Override public LSPStrategyManager get(){
LSPStrategyManager strategyManager = new LSPStrategyManagerImpl();
{
ShipmentAssigner maybeTodayAssigner = new MaybeTodayAssigner();
maybeTodayAssigner.setLSP( lsp );
strategyManager.addStrategy( new TomorrowShipmentAssignerStrategyFactory( maybeTodayAssigner ).createStrategy(), null, 1 );
}
return strategyManager;
}
} );
}
} );
GenericStrategyManager<LSPPlan, LSP> strategyManager = new GenericStrategyManagerImpl<>();
ShipmentAssigner maybeTodayAssigner = new MaybeTodayAssigner();
maybeTodayAssigner.setLSP(lsp);
strategyManager.addStrategy( new TomorrowShipmentAssignerStrategyFactory(maybeTodayAssigner).createStrategy(), null, 1);

Originally posted by @kt86 in #39 (comment)

@kt86 kt86 added the question Further information is requested label Jul 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant