Decoupled target_analysis_strategy and aspect_attack

Detail: target_analysis_strategy used to save a pointer to
aspect_attack which initialize it. Now the aspect_attack is set to a
parameter passed when the strategy is called
attack_analysis_strategy is done. ready to get hold of attack_analysis class!
attack_analysis is allowed to vary implementation of both "rating" and "analyze" methods

Detail:
1. default strategy is attack_analysis_strategy1 which matches current
implementation
2. fixed a few glitches in previous commit
3. next will allow to switch the implementation in constructor