Skip to content

Commit 09f9704

Browse files
committed
🐛 fix: avoid sending HTTP request to /contest-records/real-time-rank API when user is null
1 parent 89542b2 commit 09f9704

File tree

2 files changed

+12
-11
lines changed

2 files changed

+12
-11
lines changed

client/src/components/charts/RealTimeRankChart.jsx

-2
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
import ReactEcharts from "echarts-for-react";
22

33
const RealTimeRankChart = ({ user, rankList }) => {
4-
if (!rankList) return null;
5-
64
const realTimeRank = [["Minute", "Username", "Rank"]];
75
for (let j = 1; j <= rankList.length; j++) {
86
realTimeRank.push([j, user.username, rankList[j - 1]]);

client/src/pages/Predicted/PredictedRecords.jsx

+12-9
Original file line numberDiff line numberDiff line change
@@ -230,13 +230,15 @@ const PredictedRecords = () => {
230230
// console.log(`user=${user} ${user?.username} ${user?.data_region}`);
231231

232232
const { data: rankData } = useSWR(
233-
[
234-
`${baseUrl}/contest-records/real-time-rank`,
235-
JSON.stringify({
236-
contest_name: titleSlug,
237-
user: user,
238-
}),
239-
],
233+
user
234+
? [
235+
`${baseUrl}/contest-records/real-time-rank`,
236+
JSON.stringify({
237+
contest_name: titleSlug,
238+
user: user,
239+
}),
240+
]
241+
: null,
240242
([url, body]) =>
241243
fetch(url, {
242244
method: "POST",
@@ -246,6 +248,7 @@ const PredictedRecords = () => {
246248
{ revalidateOnFocus: false }
247249
);
248250
const rankList = rankData?.real_time_rank;
251+
// console.log(`rankData=${rankData} rankList=${rankList} ${rankList?.length} ${!rankList}`)
249252

250253
if (!predictedRecords || isLoading)
251254
return (
@@ -304,8 +307,8 @@ const PredictedRecords = () => {
304307
<label htmlFor="my-modal-4" className="modal cursor-pointer">
305308
<label className="modal-box relative" htmlFor="">
306309
<div className="container mx-auto text-center">
307-
{user && <RealTimeRankChart user={user} rankList={rankList} />}
308-
{!user && (
310+
{rankList && <RealTimeRankChart user={user} rankList={rankList} />}
311+
{!rankList && (
309312
<div>
310313
<progress className="progress w-56"></progress>
311314
<p className="text-center">Loading Real Time Rank</p>

0 commit comments

Comments
 (0)