Was wondering what might be the better way to do something in java. I
have a class of static methods that are a library and I want to use the
java. To do that I need to use the log variable I have setup already.
What is the best way to initialize and use that in the methods?
1. Create an instance variable in the static class lib for log and do a
new on the thing when start my program
_util = new JBFSAUtility(logger);
...
static Logger logger;
JBFSAUtility(Logger l)
{
logger = l;
}
.....
2. Pass the log variable to each method?
JBFSAUtility.method1(Logger l);
Thanks,
Frank
Boudewijn Dijkstra - 14 Jul 2005 16:55 GMT
> Was wondering what might be the better way to do something in java. I have a
> class of static methods that are a library and I want to use the java. To
[quoted text clipped - 14 lines]
> }
> .....
This code will overwrite static member logger everytime a new object is
created. If class JBFSAUtility should use the same logger in the entire VM,
then keep it static and don't use a constructor at all. Otherwise, don't make
logger static, and use JBFSAUtility objects.
> 2. Pass the log variable to each method?
>
> JBFSAUtility.method1(Logger l);
This code is even less sensible from a object-oriented point of view...
Hemal Pandya - 14 Jul 2005 17:23 GMT
> static Logger logger;
>
> JBFSAUtility(Logger l)
> {
> logger = l;
> }
In the interest of symmetry consider providing a static method
setLogger in JBFSAUtility. You would still have to worry about some
other method of the class getting called before the setLogger is
called, but that problem you have to tackle even with the current
approach.
So the call
_util = new JBFSAUtility(logger);
changes to
JBFSAUtility.setLogger(logger);