Логирование в L2J

[ALF.]

Немного загрустив о L2 решил посмотреть как там использовалось логирование.
После чего был сильно опечален.


Давайте рассмотрим кусочек кода

public class FileLogFormatter extends Formatter
{
	private static final String _ = "\t";
	private final SimpleDateFormat dateFmt = new SimpleDateFormat("yyyy.MM.dd HH:mm:ss,SSS");
	
	@Override
	public String format(LogRecord record)
	{
		return StringUtil.concat(dateFmt.format(new Date(record.getMillis())), _, record.getLevel().getName(), _, String.valueOf(record.getThreadID()), _, record.getLoggerName(), _, record.getMessage(), Config.EOL);
	}
}

А конкретнее эту строчку

return StringUtil.concat(dateFmt.format(new Date(record.getMillis())), _, record.getLevel().getName(), _, String.valueOf(record.getThreadID()), _, record.getLoggerName(), _, record.getMessage(), Config.EOL);

А еще конкретнее эту:

new Date(record.getMillis()))

Я конечно не знаю, что они там курили но это же полный пздц!!!

Рекомендую переписать немного код:
Сделать поле класса

private final Date date = new Date();

После

public String format(LogRecord record)
	{

дописать

date.setTime(record.getMillis());

строку

dateFmt.format(new Date(record.getMillis()))

заменить на

dateFmt.format(date)

Результат будет намного лучше.
п.с. сори, писал на быструю руку без IDE и по памяти. Если где то ошибка - сори.

Так же советую посмотреть исходники http://docs.oracle.com/javase/7/docs...Formatter.html этого класса для лучшего понимания как устроено логирование в Logger-е.

[VISTALL]

в SimpleDateFormat - метод format(Object), принимается аргумент, двох типов, Date + Long.

В итоге все заменится на

return StringUtil.concat(dateFmt.format(record.getMillis()), _, record.getLevel().getName(), _, String.valueOf(record.getThreadID()), _, record.getLoggerName(), _, record.getMessage(), Config.EOL);

[ALF.]

Сообщение от VISTALL

в SimpleDateFormat - метод format(Object), принимается аргумент, двох типов, Date + Long.

В итоге все заменится на

return StringUtil.concat(dateFmt.format(record.getMillis()), _, record.getLevel().getName(), _, String.valueOf(record.getThreadID()), _, record.getLoggerName(), _, record.getMessage(), Config.EOL);

норм.
Еще можно было типа через

String.format("%1$tH:%1$tM:%1$tS %2s %3s %4s %5s%n" и т д.

Но блин, ну создавать new Date() на каждую запись лога это же п*здец((

[Asmodiel]

Главное, чтоб работало.

[ALF.]

Сообщение от Asmodiel

Главное, чтоб работало.

Windows Vista тоже работает.
Если Вы конечно понимаете, о чем я.

[VISTALL]

Сообщение от ALF.

Windows Vista тоже работает.
Если Вы конечно понимаете, о чем я.

смирись. Есть люди которые её юзают, после СП1 она стала по лучше.

Как и большинство юзеров - юзают Клипсу, а она такое *овно

Другие юзеры - юзают IntelliJ IDEA, она не сильно отошла от Клипсы

[bloodshed]

dateFmt.format(record.getMillis())
Code: Java

внутри смотрите, так же будет создаваться новый Date

	public final StringBuffer format(Object obj, StringBuffer toAppendTo, FieldPosition fieldPosition)
	{
		if(obj instanceof Date)
			return format((Date) obj, toAppendTo, fieldPosition);
		else if(obj instanceof Number)
			return format(new Date(((Number) obj).longValue()), toAppendTo, fieldPosition);
		else
			throw new IllegalArgumentException("Cannot format given Object as a Date");
	}
Code: Java

String.format("%1$tH:%1$tM:%1$tS %2s %3s %4s %5s%n" и т д.

так же будет создавать новые объекты.

Проще переделать на другой логгер, чем тут непонятные костыли ставить.

[ALF.]

ой сори, не то увидел. Всё ок.
п.с. затераю свой преыдущий текст Кто успел прочитать - тот поймет с чем я затупил)

[VISTALL]

Сообщение от ALF.

if(obj instanceof Date)
			return format((Date) obj, toAppendTo, fieldPosition);
Code: Java

А приведение типа это же не создание объекта.

он писал ко мне

[ALF.]

Сообщение от VISTALL

он писал ко мне

Я уже понял)))
Сори)