refactor: refactoring logger

Author: OEOTYAN

src/legacy/api/LoggerAPI.cpp

@@ -52,25 +52,7 @@ string& StrReplace(string& str, const string& to_replaced, const string& new_str
 void inline LogDataHelper(LogLevel level, const Arguments& args) {
     std::string res;
     for (int i = 0; i < args.size(); ++i) res += ValueToString(args[i]);
-    switch (level) {
-    case LogLevel::Fatal:
-        ENGINE_OWN_DATA()->getModInstance()->getLogger().fatal(res);
-        break;
-    case LogLevel::Error:
-        ENGINE_OWN_DATA()->getModInstance()->getLogger().error(res);
-        break;
-    case LogLevel::Warn:
-        ENGINE_OWN_DATA()->getModInstance()->getLogger().warn(res);
-        break;
-    case LogLevel::Info:
-        ENGINE_OWN_DATA()->getModInstance()->getLogger().info(res);
-        break;
-    case LogLevel::Debug:
-        ENGINE_OWN_DATA()->getModInstance()->getLogger().debug(res);
-        break;
-    default:
-        break;
-    }
+    ENGINE_OWN_DATA()->getModInstance()->getLogger().log(level, res);
 }
 
 Local<Value> LoggerClass::log(const Arguments& args) {
@@ -206,7 +188,7 @@ Local<Value> LoggerClass::setLogLevel(const Arguments& args) {
 
     try {
         auto conf         = ENGINE_OWN_DATA();
-        conf->getModInstance()->getLogger().setFlushLevel(static_cast<LogLevel>(args[0].toInt() + 1));
+        conf->getModInstance()->getLogger().setLevel(static_cast<LogLevel>(args[0].toInt() + 1));
         return Boolean::newBoolean(true);
     }
     CATCH("Fail in SetLogLevel!")

src/lse/api/PlayerSink.cpp

@@ -2,14 +2,14 @@
 
 #include "ll/api/io/FileUtils.h"
 #include "ll/api/io/PatternFormatter.h"
-#include "mc/world/actor/player/Player.h"
 #include "ll/api/service/Bedrock.h"
+#include "mc/world/actor/player/Player.h"
 #include "mc/world/level/Level.h"
 
 namespace lse::io {
 
 PlayerSink::PlayerSink(mce::UUID const& uuid)
-: Sink(ll::makePolymorphic<ll::io::PatternFormatter>("{3:.3%T.} {2} {1} {0}", ll::io::Formatter::supportColorLog(), 0b0010)),
+: Sink(ll::makePolymorphic<ll::io::PatternFormatter>("<{tit}|{lvl}> [{tm:%T}] {msg}", false)),
   playerUuid(uuid) {}
 
 PlayerSink::~PlayerSink() = default;
@@ -22,11 +22,10 @@ void PlayerSink::append(ll::io::LogMessageView const& view) {
     std::lock_guard lock(mutex);
     std::string     buffer;
     formatter->format(view, buffer);
-    ll::service::getLevel()->getPlayer(playerUuid)->sendMessage(buffer);
-    buffer.clear();
+    if (auto* player = ll::service::getLevel()->getPlayer(playerUuid)) {
+        player->sendMessage(buffer);
+    }
 }
 
-void PlayerSink::setUUID(mce::UUID const& uuid) {
-    playerUuid = uuid;
-}
-} // namespace ll::io
+void PlayerSink::setUUID(mce::UUID const& uuid) { playerUuid = uuid; }
+} // namespace lse::io