Menu

#50 Clean up java code so that starters could be inherited.

v3.0.5
open
nobody
API (9)
5
2012-12-11
2004-04-11
No

I have a need to inherit WrapperStartStopApp in order
to add some functionality there. I found it just
impossible. It has private constructor, and uses private
methods. Then I thought to copy it completely to my
project and add whatever I need, but failed again -
because this class uses methods from other classes with
restricted visibility. It looks to me as those starter
classes has never been designed for extencibility and
here are the signs of it:
- The class is invoked by using main method;
- There a lot functionality in main method;
- Private constructor;
- There are no utility classes to facilitate common
operations within wrapper;
- Use of static and private methods;
- Use of methods from other classes with package
visibility.

It would be really nice if those classes could be
redisighed little bit so that they are more open for
enhancements.

Regards.

Alexei
alexei_novakov@yahoo.com

Discussion

  • Leif Mortenson

    Leif Mortenson - 2004-04-11

    Logged In: YES
    user_id=228081

    I'll go back and take a look at that for the future. But
    they are not really meant to be extended at this time. If
    you need some custom functionality then integration Method 3
    is available for that purpuse. I am sure that I can make
    these helper classes more extensible hower and will review that.

    What exactly are you wanting to do? I may have some
    suggestions.

    Cheers,
    Leif

     
  • Alexei Novakov

    Alexei Novakov - 2004-04-11

    Logged In: YES
    user_id=618032

    I was happy with the current functionality of
    WrapperStartStopApp, but wanted to add configurable
    actions to perform before and after service start and stop.
    Another application would be to create a chain of dependent
    services so that change in status of one service would affect
    another one in some way. I thought it would be relatively
    easy but didn't find the way.

    Cheers.
    Alexei.

     

Log in to post a comment.

MongoDB Logo MongoDB