fix: fix #39 #40

Author: ShrBox

docs/apis/DataAPI/PlayerData.md

@@ -108,7 +108,8 @@ When a player enters the server for the first time, his name and XUID are automa
   - If the return value is `Null`, it means the query failed.
 
 
-
+!!! tip
+    getAllPlayerInfo has been deprecated, it will only return online players's data
 #### Get all player information
 
 `data.getAllPlayerInfo()`

docs/apis/DataAPI/PlayerData.zh.md

@@ -108,7 +108,8 @@ XUID数据库让你可以即使在玩家离线的时候，也可以查询玩家
   - 如果返回值为`Null`，则代表查询失败
 
 
-
+!!!tip
+    getAllPlayerInfo已被弃用，它现在只会返回在线玩家的信息
 #### 获取所有的玩家信息
 
 `data.getAllPlayerInfo()`

src/legacy/api/DataAPI.cpp

@@ -743,15 +743,18 @@ Local<Value> DataClass::getAllPlayerInfo(const Arguments& args) {
     CHECK_ARGS_COUNT(args, 0);
 
     try {
-        auto arr = Array::newArray();
-        ll::service::getLevel()->forEachPlayer([&arr](Player& player) {
-            auto obj = Object::newObject();
-            obj.set("name", String::newString(player.getRealName()));
-            obj.set("xuid", String::newString(player.getXuid()));
-            obj.set("uuid", String::newString(player.getUuid().asString()));
-            arr.add(obj);
-            return true;
-        });
+        auto arr   = Array::newArray();
+        auto level = ll::service::getLevel();
+        if (level.has_value()) {
+            level->forEachPlayer([&arr](Player& player) {
+                auto obj = Object::newObject();
+                obj.set("name", String::newString(player.getRealName()));
+                obj.set("xuid", String::newString(player.getXuid()));
+                obj.set("uuid", String::newString(player.getUuid().asString()));
+                arr.add(obj);
+                return true;
+            });
+        }
 
         return arr;
     }

src/legacy/api/PlayerAPI.cpp

@@ -579,11 +579,14 @@ Local<Value> McClass::getPlayer(const Arguments& args) {
 
 Local<Value> McClass::getOnlinePlayers(const Arguments& args) {
     try {
-        Local<Array> list = Array::newArray();
-        ll::service::getLevel()->forEachPlayer([&](Player& player) {
-            list.add(PlayerClass::newPlayer(&player));
-            return true;
-        });
+        Local<Array> list  = Array::newArray();
+        auto         level = ll::service::getLevel();
+        if (level.has_value()) {
+            level->forEachPlayer([&](Player& player) {
+                list.add(PlayerClass::newPlayer(&player));
+                return true;
+            });
+        }
         return list;
     }
     CATCH("Fail in GetOnlinePlayers!")