fix: 处理了一下成绩空指针

HandiSWUST · Sep 19, 2024 · 0f23d2b · 0f23d2b
1 parent 31abc1d
commit 0f23d2b
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/src/main/java/org/shirakawatyu/handixikebackend/api/impl/MatrixScoreApi.java b/src/main/java/org/shirakawatyu/handixikebackend/api/impl/MatrixScoreApi.java
@@ -3,7 +3,9 @@
 import lombok.extern.slf4j.Slf4j;
 import org.apache.hc.client5.http.cookie.CookieStore;
 import org.jsoup.Jsoup;
+import org.jsoup.nodes.Document;
 import org.shirakawatyu.handixikebackend.api.ScoreApi;
+import org.shirakawatyu.handixikebackend.exception.NotLoginException;
 import org.shirakawatyu.handixikebackend.exception.OutOfCreditException;
 import org.shirakawatyu.handixikebackend.pojo.GradePointAverage;
 import org.shirakawatyu.handixikebackend.pojo.Score;
@@ -64,10 +66,11 @@ public LinkedHashMap<String, ArrayList<Score>> getScore(CookieStore cookieStore)
                 throw new OutOfCreditException();
             }
             resp = Requests.getForString(SCORE_URL + "?event=studentProfile:courseMark", "", cookieStore);
-            scores = Jsoup.parse(resp).getElementsByClass("UItable").select("tr").eachText();
-            if (scores.isEmpty()) {
-                return null;
+            Document document = Jsoup.parse(resp);
+            if (document.getElementById("blueBar") == null) {
+                throw new NotLoginException();
             }
+            scores = document.getElementsByClass("UItable").select("tr").eachText();
             return processScore(scores);
         } catch (Exception e) {
             if (scores != null) {
@@ -83,9 +86,6 @@ private LinkedHashMap<String, ArrayList<Score>> processScore(List<String> scoreS
         ScoreUtils.requiredScoreFilter(scores, hashMap);
         ScoreUtils.optionalScoreFilter(scores, hashMap);
         ScoreUtils.cetScoreFilter(scores, hashMap);
-        if (hashMap.isEmpty()) {
-            return null;
-        }
         if (!scores.isEmpty()) {
             log.warn(scores.toString());
         }