今天进行slf4j中logger的同步封装。主要目的是为了以后方便更换日志实现系统。
遇到的问题：使用Thread.currentThread().getStackTrace()[1].getClassName()得到的是当前类而不是调用类。见以下代码：
private org.slf4j.Logger logger = null; /**
* construction method
*/
public Logger(){
// get the current class logger
logger = LoggerFactory.getLogger(Thread.currentThread().getStackTrace()[1].getClassName());
}
所以打印日志的时候并没有得到日志真正所属类的信息。
然后，我进行了一组測试：
測试A：
public class ThreadTest {
public static void TestString(){
StackTraceElement[] arr = new Exception().getStackTrace();
for(int i=0;i<=arr.length-1;i++){
System.out.println(arr[i].getClassName()+";"+arr[i].getMethodName()+";"+arr[i].getFileName());
}
}
}
public class App
{
public static void main( String[] args )
{
ThreadTest.TestString();
}
}
结果是：
test.ThreadTest;TestString;ThreadTest.java（当前方法和类）
test.App;main;App.java（调用该方法的方法和类）
測试B：
public class ThreadTest {
public static void TestString(){
StackTraceElement[] arr = Thread.currentThread().getStackTrace();
for(int i=0;i<=arr.length-1;i++){
System.out.println(arr[i].getClassName()+";"+arr[i].getMethodName()+";"+arr[i].getFileName());
}
}
}
App类同上。得到的结果是：
java.lang.Thread;getStackTrace;Thread.java（Thread的信息）
test.ThreadTest;TestString;ThreadTest.java（当前方法和类）
test.App;main;App.java（调用该方法的方法和类）
啥米情况？？？难道就是由于Thread的getStackTrace比Throwable（Exception的父类）的getStackTrace多打了一段Thread类的信息？？？
因为不确定是不是真的是这样子的，故去查看了下jdk的源代码（纯属装X，不抱幻想）。
首先是Throwable的getStackTrace方法，代码例如以下：
public StackTraceElement[] getStackTrace() {
return getOurStackTrace().clone();
}
private synchronized StackTraceElement[] getOurStackTrace() {
// Initialize stack trace field with information from
// backtrace if this is the first call to this method
if (stackTrace == UNASSIGNED_STACK ||
(stackTrace == null && backtrace != null) /* Out of protocol state */) {
int depth = getStackTraceDepth();
stackTrace = new StackTraceElement[depth];
for (int i=0; i < depth; i++)
stackTrace[i] = getStackTraceElement(i);
} else if (stackTrace == null) {
return UNASSIGNED_STACK;
}
return stackTrace;
}
貌似没有什么不正确劲的地方哈，当中
int depth = getStackTraceDepth();
stackTrace[i] = getStackTraceElement(i);
两个方法都是native的，不予深究了。这里没有找到问题。就去看了下Thread的getStackTrace方法，代码例如以下：
public StackTraceElement[] getStackTrace() {
if (this != Thread.currentThread()) {
// check for getStackTrace permission
SecurityManager security = System.getSecurityManager();
if (security != null) {
security.checkPermission(
SecurityConstants.GET_STACK_TRACE_PERMISSION);
}
// optimization so we do not call into the vm for threads that
// have not yet started or have terminated
if (!isAlive()) {
return EMPTY_STACK_TRACE;
}
StackTraceElement[][] stackTraceArray = dumpThreads(new Thread[] {this});
StackTraceElement[] stackTrace = stackTraceArray[0];
// a thread that was alive during the previous isAlive call may have
// since terminated, therefore not having a stacktrace.
if (stackTrace == null) {
stackTrace = EMPTY_STACK_TRACE;
}
return stackTrace;
} else {
// Don't need JVM help for current thread
return (new Exception()).getStackTrace();
}
}
乍一看也没啥错啊~~~不正确，突然间跟着逻辑走一下。在if语句里面由于
this != Thread.currentThread()是为true的。所以方法会运行else里面的语句，里面是啥。！！
！
return (new Exception()).getStackTrace();
All right！
就是这里，这里jvm去运行了new Exception().getStackTrace();就是这句话让使用Thread的getStackTrace方法就有可能多打印一句java.lang.Thread;getStackTrace;Thread.java出来。这也就是为什么使用
logger = LoggerFactory.getLogger(Thread.currentThread().getStackTrace()[1].getClassName());
会得不到正确的日志信息的原因了。
应该就是这样子了。为了验证我想的对不正确，写了一个測试验证的样例，代码例如以下：
public class TestException {
public static StackTraceElement[] getStackTrace() {
return new Exception().getStackTrace();
}
}
public class ThreadTest {
public static void TestString(){
StackTraceElement[] arr = TestException.getStackTrace();
for(int i=0;i<=arr.length-1;i++){
System.out.println(arr[i].getClassName()+";"+arr[i].getMethodName()+";"+arr[i].getFileName());
}
}
}
public class App
{
public static void main( String[] args )
{
ThreadTest.TestString();
}
}
运行之后的结果是：
test.TestException;getStackTrace;TestException.java
test.ThreadTest;TestString;ThreadTest.java
test.App;main;App.java
红色的一行即证明了我的想法是正确的！
全部的验证至此结束了。这个问题最后解决的方法非常easy，要么使用new Exception().getStackTrace()[1]；要么使用Thread.currentThread().getStackTrace()[2]。
尽管这个问题非常小，也非常基础。可是我还是非常兴奋。毕竟粘上了源代码，顿时逼格升高不少~~~
新手上路。高手饶命~！