I have a logging function that takes the calling object as a parameter. I then call getClass().getSimpleName() on it so that I can easily get the class name to add to my log en
You could add "Class" as first parameter and overload the log method:
public class SomeClass {
// Usage test:
public static void main( String [] args ) {
log( SomeClass.class, "Hola" );
log( new java.util.Date(), "Hola" );
}
// Object version would call Class method...
public static void log( Object o , String msg ) {
log( o.getClass(), msg );
}
public static void log( Class c , String message ) {
System.out.println( c.getSimpleName() + " " + message );
}
}
Output:
$ java SomeClass
SomeClass Hola
Date Hola
But it feels just bad to have the calling class passed as the first argument. Here's where object oriented model comes into play as opposite of "procedural" style.
You could get the calling class using the stacktrace, but as you metioned, there would be an overhead if you call it thousands of times.
But if you create is as class variable, then there would be only one instance for class if you happen to have 1,000 classes using this utility, you would have at most 1,000 calls.
Something like this would be better ( subtle variation of this other answer) :
public class LogUtility {
private final String loggingFrom;
public static LogUtility getLogger() {
StackTraceElement [] s = new RuntimeException().getStackTrace();
return new LogUtility( s[1].getClassName() );
}
private LogUtility( String loggingClassName ) {
this.loggingFrom = "("+loggingClassName+") ";
}
public void log( String message ) {
System.out.println( loggingFrom + message );
}
}
Usage test:
class UsageClass {
private static final LogUtility logger = LogUtility.getLogger();
public static void main( String [] args ) {
UsageClass usageClass = new UsageClass();
usageClass.methodOne();
usageClass.methodTwo();
usageClass.methodThree();
}
private void methodOne() {
logger.log("One");
}
private void methodTwo() {
logger.log("Two");
}
private void methodThree() {
logger.log("Three");
}
}
OUtput
$ java UsageClass
(UsageClass) One
(UsageClass) Two
(UsageClass) Three
Notice the declaration:
....
class UsageClass {
// This is invoked only once. When the class is first loaded.
private static final LogUtility logger = LogUtility.getLogger();
....
That way, it doesn't matter if you use the "logger" from objectA, objectB, objectC or from a class method ( like main ) they all would have one instance of the logger.