From 539a1c44ce42c407edc8acf494d130ae1e6507bb Mon Sep 17 00:00:00 2001
From: undefined <i@undefined.moe>
Date: Thu, 5 May 2022 21:51:12 +0800
Subject: [PATCH] judge: use case.score & normalizeSubtask API

---
 packages/hydrojudge/src/cases.ts              |  89 ++++++--------
 packages/hydrojudge/src/judge/default.ts      |   2 +-
 packages/hydrojudge/src/judge/interactive.ts  |   2 +-
 packages/hydrojudge/src/judge/interface.ts    |   1 +
 .../hydrojudge/src/judge/submit_answer.ts     |   2 +-
 packages/hydrooj/src/interface.ts             |   8 +-
 packages/hydrooj/src/lib/testdataConfig.ts    |  31 ++---
 .../problemconfig/ProblemConfigEditor.tsx     |   6 +-
 .../components/problemconfig/SubtaskTable.tsx |  52 ++++++--
 .../problemconfig/reducer/config.ts           |  22 ++--
 .../ui-default/pages/problem_config.page.styl |   8 +-
 .../ui-default/pages/problem_config.page.tsx  |  26 ++--
 packages/utils/lib/cases.ts                   |  68 ++---------
 packages/utils/lib/common.ts                  | 115 ++++++++++++++----
 14 files changed, 231 insertions(+), 201 deletions(-)

diff --git a/packages/hydrojudge/src/cases.ts b/packages/hydrojudge/src/cases.ts
index d81ab2e17..ca824114b 100644
--- a/packages/hydrojudge/src/cases.ts
+++ b/packages/hydrojudge/src/cases.ts
@@ -3,40 +3,12 @@ import fs from 'fs-extra';
 import yaml from 'js-yaml';
 import { max, sum } from 'lodash';
 import readYamlCases, { convertIniConfig } from '@hydrooj/utils/lib/cases';
-import { readSubtasksFromFiles } from '@hydrooj/utils/lib/common';
+import { normalizeSubtasks, readSubtasksFromFiles } from '@hydrooj/utils/lib/common';
 import { changeErrorType } from '@hydrooj/utils/lib/utils';
 import { getConfig } from './config';
 import { FormatError, SystemError } from './error';
 import { ensureFile, parseMemoryMB } from './utils';
 
-async function readAutoCases(folder: string, { next }, cfg, rst) {
-    const config = {
-        checker_type: 'default',
-        count: 0,
-        subtasks: [],
-        judge_extra_files: [],
-        user_extra_files: [],
-    };
-    const checkFile = ensureFile(folder);
-    try {
-        const files = await fs.readdir(folder);
-        if (await fs.pathExists(path.resolve(folder, 'input'))) {
-            const inputs = await fs.readdir(path.resolve(folder, 'input'));
-            files.push(...inputs.map((i) => `input/${i}`));
-        }
-        if (await fs.pathExists(path.resolve(folder, 'output'))) {
-            const outputs = await fs.readdir(path.resolve(folder, 'output'));
-            files.push(...outputs.map((i) => `output/${i}`));
-        }
-        const result = readSubtasksFromFiles(files, checkFile, cfg, rst);
-        Object.assign(config, result);
-        if (cfg.isSelfSubmission) next({ message: { message: 'Found {0} testcases.', params: [config.count] } });
-    } catch (e) {
-        throw new SystemError('Cannot parse testdata.', [e.message, ...e.params]);
-    }
-    return config;
-}
-
 function isValidConfig(config) {
     if (config.count > (getConfig('testcases_max') || 100)) {
         throw new FormatError('Too many testcases. Cancelled.');
@@ -52,35 +24,54 @@ function isValidConfig(config) {
     }
 }
 
+async function collectFiles(folder: string) {
+    const files = await fs.readdir(folder);
+    await Promise.all(['input', 'output'].map(async (t) => {
+        if (await fs.pathExists(path.resolve(folder, t))) {
+            const f = await fs.readdir(path.resolve(folder, t));
+            files.push(...f.map((i) => `${t}/${i}`));
+        }
+    }));
+    return files;
+}
+
 export default async function readCases(folder: string, cfg: Record<string, any> = {}, args) {
     const iniConfig = path.resolve(folder, 'config.ini');
     const yamlConfig = path.resolve(folder, 'config.yaml');
     const ymlConfig = path.resolve(folder, 'config.yml');
-    const config = { ...cfg };
-    if (fs.existsSync(yamlConfig)) {
-        Object.assign(config, yaml.load(fs.readFileSync(yamlConfig).toString()));
-    } else if (fs.existsSync(ymlConfig)) {
-        Object.assign(config, yaml.load(fs.readFileSync(ymlConfig).toString()));
-    } else if (fs.existsSync(iniConfig)) {
-        try {
-            Object.assign(config, convertIniConfig(fs.readFileSync(iniConfig).toString()));
-        } catch (e) {
-            throw changeErrorType(e, FormatError);
-        }
-    }
-    let result;
+    const config: Record<string, any> = {
+        checker_type: 'default',
+        count: 0,
+        subtasks: [],
+        judge_extra_files: [],
+        user_extra_files: [],
+        ...cfg,
+    };
     try {
-        result = await readYamlCases(config, ensureFile(folder));
+        if (fs.existsSync(yamlConfig)) {
+            Object.assign(config, yaml.load(fs.readFileSync(yamlConfig).toString()));
+        } else if (fs.existsSync(ymlConfig)) {
+            Object.assign(config, yaml.load(fs.readFileSync(ymlConfig).toString()));
+        } else if (fs.existsSync(iniConfig)) {
+            Object.assign(config, convertIniConfig(fs.readFileSync(iniConfig).toString()));
+        }
     } catch (e) {
         throw changeErrorType(e, FormatError);
     }
-    let cases = result.outputs?.length || 0;
-    cases += Math.sum((result.subtasks || []).map((subtask) => subtask.cases.length));
-    if (!cases) {
-        const c = await readAutoCases(folder, args, config, result);
-        result.subtasks = c.subtasks;
-        result.count = c.count;
+    const checkFile = ensureFile(folder);
+    const result = await readYamlCases(config, checkFile)
+        .catch((e) => { throw changeErrorType(e, FormatError); });
+    result.count = result.outputs?.length || Math.sum((result.subtasks || []).map((s) => s.cases.length));
+    if (!result.count) {
+        try {
+            result.subtasks = readSubtasksFromFiles(await collectFiles(folder), checkFile, cfg);
+            result.count = Math.sum(result.subtasks.map((i) => i.cases.length));
+            if (cfg.isSelfSubmission) args.next?.({ message: { message: 'Found {0} testcases.', params: [result.count] } });
+        } catch (e) {
+            throw new SystemError('Cannot parse testdata.', [e.message, ...e.params]);
+        }
     }
+    result.subtasks = normalizeSubtasks(result.subtasks || [], checkFile, config.time, config.memory);
     if (result.key && args.key !== result.key) throw new FormatError('Incorrect secret key');
     if (!result.key) isValidConfig(result);
     return result;
diff --git a/packages/hydrojudge/src/judge/default.ts b/packages/hydrojudge/src/judge/default.ts
index 14a589783..976cb5867 100644
--- a/packages/hydrojudge/src/judge/default.ts
+++ b/packages/hydrojudge/src/judge/default.ts
@@ -76,7 +76,7 @@ function judgeCase(c: Case, sid: string) {
                     user_stdout: stdout,
                     user_stderr: stderr,
                     checker_type: ctx.config.checker_type,
-                    score: ctxSubtask.subtask.score,
+                    score: c.score,
                     detail: ctx.config.detail ?? true,
                     env: { ...ctx.env, HYDRO_TESTCASE: c.id.toString() },
                 });
diff --git a/packages/hydrojudge/src/judge/interactive.ts b/packages/hydrojudge/src/judge/interactive.ts
index e99b33713..cc7d492a4 100644
--- a/packages/hydrojudge/src/judge/interactive.ts
+++ b/packages/hydrojudge/src/judge/interactive.ts
@@ -50,7 +50,7 @@ function judgeCase(c: Case) {
             if (code < 32) message = signals[code];
             else message = { message: 'Your program returned {0}.', params: [code] };
         } else {
-            const result = parse(resInteractor.stderr, ctxSubtask.subtask.score);
+            const result = parse(resInteractor.stderr, c.score);
             status = result.status;
             score = result.score;
             message = result.message;
diff --git a/packages/hydrojudge/src/judge/interface.ts b/packages/hydrojudge/src/judge/interface.ts
index b5f5f45d5..448ae7cef 100644
--- a/packages/hydrojudge/src/judge/interface.ts
+++ b/packages/hydrojudge/src/judge/interface.ts
@@ -105,4 +105,5 @@ export interface Case {
     id: number;
     input: string;
     output: string;
+    score: number;
 }
diff --git a/packages/hydrojudge/src/judge/submit_answer.ts b/packages/hydrojudge/src/judge/submit_answer.ts
index 67096be5c..d0623a795 100644
--- a/packages/hydrojudge/src/judge/submit_answer.ts
+++ b/packages/hydrojudge/src/judge/submit_answer.ts
@@ -74,7 +74,7 @@ function judgeCase(c: Case, sid: string) {
                 user_stdout: file,
                 user_stderr: { content: '' },
                 checker_type: ctx.config.checker_type,
-                score: ctxSubtask.subtask.score,
+                score: c.score,
                 detail: ctx.config.detail ?? true,
                 env: { ...ctx.env, HYDRO_TESTCASE: c.id.toString() },
             });
diff --git a/packages/hydrooj/src/interface.ts b/packages/hydrooj/src/interface.ts
index f2772bf57..dc33df041 100644
--- a/packages/hydrooj/src/interface.ts
+++ b/packages/hydrooj/src/interface.ts
@@ -129,8 +129,8 @@ export interface FileInfo {
 export interface TestCaseConfig {
     input: string;
     output: string;
-    time?: number;
-    memory?: number;
+    time?: string;
+    memory?: string;
     score?: number;
 }
 
@@ -148,8 +148,8 @@ export enum SubtaskType {
 }
 
 export interface SubtaskConfig {
-    time?: number;
-    memory?: number;
+    time?: string;
+    memory?: string;
     score?: number;
     if?: number[];
     id?: number;
diff --git a/packages/hydrooj/src/lib/testdataConfig.ts b/packages/hydrooj/src/lib/testdataConfig.ts
index 3607b5d06..c6a1fde45 100644
--- a/packages/hydrooj/src/lib/testdataConfig.ts
+++ b/packages/hydrooj/src/lib/testdataConfig.ts
@@ -1,13 +1,12 @@
 import { load } from 'js-yaml';
 import readYamlCases from '@hydrooj/utils/lib/cases';
 import type { ProblemConfig, ProblemConfigFile } from '../interface';
+import { normalizeSubtasks, parseMemoryMB, parseTimeMS } from '../utils';
 
 export async function parseConfig(config: string | ProblemConfigFile = {}) {
-    let cfg: ProblemConfigFile = {};
-    if (typeof config === 'string') {
-        // TODO should validate here?
-        cfg = await readYamlCases(load(config) as Record<string, any>);
-    } else if (typeof config === 'object') cfg = await readYamlCases(config);
+    const cfg: ProblemConfigFile = typeof config === 'string'
+        ? await readYamlCases(load(config) as Record<string, any>)
+        : await readYamlCases(config);
     const result: ProblemConfig = {
         count: 0,
         memoryMin: Number.MAX_SAFE_INTEGER,
@@ -18,23 +17,19 @@ export async function parseConfig(config: string | ProblemConfigFile = {}) {
     };
     if (cfg.subType) result.subType = cfg.subType;
     if (cfg.target) result.target = cfg.target;
-    if (cfg.subtasks.length) {
-        for (const subtask of cfg.subtasks) {
-            result.memoryMax = Math.max(result.memoryMax, subtask.memory);
-            result.memoryMin = Math.min(result.memoryMin, subtask.memory);
-            result.timeMax = Math.max(result.timeMax, subtask.time);
-            result.timeMin = Math.min(result.timeMin, subtask.time);
+    if (cfg.subtasks?.length) {
+        for (const subtask of normalizeSubtasks(cfg.subtasks as any || [], (i) => i, cfg.time, cfg.memory)) {
+            result.memoryMax = Math.max(result.memoryMax, ...subtask.cases.map((i) => parseMemoryMB(i.memory)));
+            result.memoryMin = Math.min(result.memoryMin, ...subtask.cases.map((i) => parseMemoryMB(i.memory)));
+            result.timeMax = Math.max(result.timeMax, ...subtask.cases.map((i) => parseTimeMS(i.time)));
+            result.timeMin = Math.min(result.timeMin, ...subtask.cases.map((i) => parseTimeMS(i.time)));
         }
-    } else if (cfg.time || cfg.memory) {
+    } else {
         if (cfg.time) result.timeMax = result.timeMin = cfg.time as unknown as number;
         if (cfg.memory) result.memoryMax = result.memoryMin = cfg.memory as unknown as number;
     }
-    if (result.memoryMax < result.memoryMin) {
-        result.memoryMax = result.memoryMin = 256;
-    }
-    if (result.timeMax < result.timeMin) {
-        result.timeMax = result.timeMin = 1000;
-    }
+    if (result.memoryMax < result.memoryMin) result.memoryMax = result.memoryMin = 256;
+    if (result.timeMax < result.timeMin) result.timeMax = result.timeMin = 1000;
     if (cfg.langs) result.langs = cfg.langs;
     if (cfg.redirect) result.redirect = cfg.redirect.split('/', 2) as any;
     if (cfg.filename && result.type === 'default') {
diff --git a/packages/ui-default/components/problemconfig/ProblemConfigEditor.tsx b/packages/ui-default/components/problemconfig/ProblemConfigEditor.tsx
index 68774fc7c..6da3b1baf 100644
--- a/packages/ui-default/components/problemconfig/ProblemConfigEditor.tsx
+++ b/packages/ui-default/components/problemconfig/ProblemConfigEditor.tsx
@@ -38,14 +38,14 @@ const subtasksKey = [
 
 const casesKey = ['time', 'memory', 'input', 'output'];
 
-function configYamlFormat(config: ProblemConfigFile) {
+export function configYamlFormat(config: ProblemConfigFile) {
   const formatConfig: ProblemConfigFile = {};
   configKey.forEach((key) => {
     if (config[key] !== undefined) {
       if (key === 'subType' && ['single', 'multi'].includes(config[key]) && config.type !== 'submit_answer') return;
       if (key === 'checker_type' && config.type !== 'default') return;
       if (key === 'checker'
-        && (['default', 'strict'].includes(formatConfig.checker_type) || formatConfig.checker_type === undefined)) return;
+        && (['default', 'strict'].includes(formatConfig.checker_type) || !formatConfig.checker_type)) return;
       if (key === 'interactor' && config.type !== 'interactive') return;
       if (key === 'subtasks') {
         formatConfig[key] = [];
@@ -62,7 +62,7 @@ function configYamlFormat(config: ProblemConfigFile) {
         formatConfig[key] = [];
         config[key].forEach((caseItem) => {
           const formatCase: TestCaseConfig = {
-            time: 1000, memory: 256, input: '', output: '',
+            time: '1000ms', memory: '256MB', input: '', output: '',
           };
           casesKey.forEach((caseKey) => {
             if (caseItem[caseKey] !== undefined) formatCase[caseKey] = caseItem[caseKey];
diff --git a/packages/ui-default/components/problemconfig/SubtaskTable.tsx b/packages/ui-default/components/problemconfig/SubtaskTable.tsx
index 0f3e68148..145b23e7f 100644
--- a/packages/ui-default/components/problemconfig/SubtaskTable.tsx
+++ b/packages/ui-default/components/problemconfig/SubtaskTable.tsx
@@ -5,7 +5,7 @@ import { useSelector, useDispatch } from 'react-redux';
 import { SubtaskConfig } from 'hydrooj/src/interface';
 import { parseTimeMS, parseMemoryMB } from '@hydrooj/utils/lib/common';
 import CustomSelectAutoComplete from '../autocomplete/components/CustomSelectAutoComplete';
-import { FormItem, ManagedInput } from './BasicForm';
+import { FormItem } from './BasicForm';
 import { RootState } from './reducer';
 import { CasesTable } from './TestCasesTable';
 
@@ -71,7 +71,7 @@ export function SubtasksTable({ index }) {
             {['id', 'score'].map((key) => (
               <td key={key}>
                 <input
-                  value={subtask[key] || ''}
+                  value={subtask[key] ?? ''}
                   onChange={dispatcher(key)}
                   type="number"
                   className="textbox"
@@ -80,7 +80,7 @@ export function SubtasksTable({ index }) {
             ))}
             <td>
               <select
-                value={subtask.type || ''}
+                value={subtask.type ?? 'min'}
                 onChange={dispatcher('type')}
                 className="select"
               >
@@ -116,20 +116,48 @@ export function SubtasksTable({ index }) {
   );
 }
 
+function GlobalTaskConfig() {
+  const time = useSelector((state: RootState) => state.config.time);
+  const memory = useSelector((state: RootState) => state.config.memory);
+  const dispatch = useDispatch();
+  const dispatcher = (key: string, suffix = '') => (ev: React.ChangeEvent<HTMLInputElement | HTMLSelectElement> | number) => {
+    let value = typeof ev !== 'object' ? ev : ev.currentTarget?.value;
+    if (value === 0) value = '';
+    if (value && suffix) value += suffix;
+    dispatch({ type: 'CONFIG_FORM_UPDATE', key, value });
+  };
+  return (
+    <>
+      <FormItem columns={6} label="Time">
+        <NumericInput
+          rightElement={<Tag minimal>ms</Tag>}
+          value={time ? parseTimeMS(time).toString() : ''}
+          placeholder={parseTimeMS('1000ms').toString()}
+          onValueChange={dispatcher('time', 'ms')}
+          buttonPosition="none"
+          fill
+        />
+      </FormItem>
+      <FormItem columns={6} label="Memory">
+        <NumericInput
+          rightElement={<Tag minimal>MB</Tag>}
+          value={memory ? parseMemoryMB(memory).toString() : ''}
+          placeholder={parseMemoryMB('256m').toString()}
+          onValueChange={dispatcher('memory', 'MB')}
+          buttonPosition="none"
+          fill
+        />
+      </FormItem>
+    </>
+  );
+}
+
 export function TaskConfig() {
   const len = useSelector((state: RootState) => state.config.subtasks?.length);
   return (
     <FormItem columns={12} label="Task Settings">
       <div className="row">
-        <FormItem columns={4} label="Time">
-          <ManagedInput placeholder="Time" formKey="time" />
-        </FormItem>
-        <FormItem columns={4} label="Memory">
-          <ManagedInput placeholder="Memory" formKey="memory" />
-        </FormItem>
-        <FormItem columns={4} label="Score">
-          <ManagedInput placeholder="Score" formKey="score" />
-        </FormItem>
+        <GlobalTaskConfig />
         <FormItem columns={12} label="TestCases" disableLabel>
           {len && [...Array(len).keys()].map((i) => <SubtasksTable index={i} key={i} />)}
         </FormItem>
diff --git a/packages/ui-default/components/problemconfig/reducer/config.ts b/packages/ui-default/components/problemconfig/reducer/config.ts
index eec223d6a..5c1aa47f8 100644
--- a/packages/ui-default/components/problemconfig/reducer/config.ts
+++ b/packages/ui-default/components/problemconfig/reducer/config.ts
@@ -1,6 +1,7 @@
 import yaml from 'js-yaml';
 import type { ProblemConfigFile } from 'hydrooj/src/interface';
 import { cloneDeep } from 'lodash';
+import { parseMemoryMB, parseTimeMS } from '@hydrooj/utils/lib/common';
 
 type State = ProblemConfigFile & { __loaded: boolean };
 
@@ -25,23 +26,20 @@ export default function reducer(state = { type: 'default', __loaded: false } as
     }
     case 'CONFIG_AUTOCASES_UPDATE': {
       const next = { ...state };
-      const autocases = action.value;
-      if (autocases.subtasks.length === 0) next.subtasks = [];
+      const { subtasks } = action;
+      for (const subtask of subtasks) {
+        if (subtask.time === parseTimeMS(state.time || '1s')) delete subtask.time;
+        if (subtask.memory === parseMemoryMB(state.memory || '256m')) delete subtask.memory;
+        if (subtask.time) subtask.time += 'ms';
+        if (subtask.memory) subtask.memory += 'MB';
+      }
+      if (subtasks.length === 0) next.subtasks = [];
       else {
-        next.subtasks = autocases.subtasks.map((subtask) => (
+        next.subtasks = subtasks.map((subtask) => (
           { ...subtask, ...{ cases: subtask.cases.map((i) => ({ input: i.input, output: i.output })) } }));
       }
       return next;
     }
-    case 'CONFIG_SUBTASKS_SWITCH': {
-      const next = { ...state };
-      next.subtasks = next.cases.map((k, v) => ({
-        id: v, time: k.time, memory: k.memory, cases: [{ input: k.input, output: k.output }],
-      }));
-      if (!next.subtasks.length) next.subtasks.push({ id: 0 });
-      delete next.cases;
-      return next;
-    }
     case 'CONFIG_SUBTASK_UPDATE': {
       const subtasks = cloneDeep(state.subtasks);
       const subsubtasks = cloneDeep(state.subtasks[action.id]);
diff --git a/packages/ui-default/pages/problem_config.page.styl b/packages/ui-default/pages/problem_config.page.styl
index 0166572f8..8073be520 100644
--- a/packages/ui-default/pages/problem_config.page.styl
+++ b/packages/ui-default/pages/problem_config.page.styl
@@ -52,8 +52,7 @@
         transition: outline-color 0.2s, border-color 0.2s
         transition-timing-function: cubic-bezier(0.215, 0.61, 0.355, 1)
         box-shadow: none
-      input::placeholder
-        color: inherit
+      
 
     .data-table
       tr
@@ -68,4 +67,7 @@
         background: transparent
 
   .hover-row.status-bar
-    display: none
\ No newline at end of file
+    display: none
+  
+  input::placeholder
+    color: rgba(95,107,124,.6)!important
\ No newline at end of file
diff --git a/packages/ui-default/pages/problem_config.page.tsx b/packages/ui-default/pages/problem_config.page.tsx
index 06eeeef47..6fc7a1320 100644
--- a/packages/ui-default/pages/problem_config.page.tsx
+++ b/packages/ui-default/pages/problem_config.page.tsx
@@ -7,10 +7,10 @@ import yaml from 'js-yaml';
 import Notification from 'vj/components/notification';
 import Dialog, { ConfirmDialog } from 'vj/components/dialog/index';
 import download from 'vj/components/zipDownloader';
-import {
-  size, readSubtasksFromFiles, parseTimeMS, parseMemoryMB,
-} from '@hydrooj/utils/lib/common';
+import { size, readSubtasksFromFiles } from '@hydrooj/utils/lib/common';
 import tpl from 'vj/utils/tpl';
+import { SubtaskType } from 'hydrooj/src/interface';
+import { configYamlFormat } from 'vj/components/problemconfig/ProblemConfigEditor';
 
 async function handleSection(ev: JQuery.ClickEvent<Document, undefined, any, any>, type: string) {
   const $section = $(ev.currentTarget).closest('.section--problem-sidebar-testdata');
@@ -146,7 +146,7 @@ const page = new NamedPage('problem_config', () => {
   }
 
   async function uploadConfig(config: object) {
-    const configYaml = yaml.dump(config);
+    const configYaml = yaml.dump(configYamlFormat(config));
     Notification.info(i18n('Saving file...'));
     const data = new FormData();
     data.append('filename', 'config.yaml');
@@ -179,20 +179,20 @@ const page = new NamedPage('problem_config', () => {
       // TODO set yaml schema
       const state = store.getState();
       if (!state.config.__loaded) return;
-      if (state.config.subtasks || state.config.cases) return;
+      if (state.config.cases) {
+        const score = state.config.score * state.config.cases.length;
+        state.config.subtasks = [{ type: 'sum' as SubtaskType, score: score && score < 100 ? score : 100, cases: state.config.cases }];
+        delete state.config.cases;
+        delete state.config.score;
+      }
+      if (state.config.subtasks) return;
       const testdata = (state.testdata || []).map((i) => i.name);
       const checkFile = (file: string) => (testdata.includes(file) ? file : null);
       unsubscribe();
-      const value = readSubtasksFromFiles(testdata, checkFile, state.config, {});
-      for (const subtask of value.subtasks) {
-        if (subtask.time === parseTimeMS(state.config.time || '1s')) delete subtask.time;
-        if (subtask.memory === parseMemoryMB(state.config.memory || '256m')) delete subtask.memory;
-        if (subtask.time) subtask.time += 'ms';
-        if (subtask.memory) subtask.memory += 'MB';
-      }
+      const subtasks = readSubtasksFromFiles(testdata, checkFile, state.config);
       store.dispatch({
         type: 'CONFIG_AUTOCASES_UPDATE',
-        value,
+        subtasks,
       });
     });
     render(
diff --git a/packages/utils/lib/cases.ts b/packages/utils/lib/cases.ts
index a9e841bda..faa5a0bea 100644
--- a/packages/utils/lib/cases.ts
+++ b/packages/utils/lib/cases.ts
@@ -3,13 +3,10 @@ import { findFileSync, parseMemoryMB, parseTimeMS } from './utils';
 // eslint-disable-next-line @typescript-eslint/no-unused-vars
 export default async function readYamlCases(cfg: Record<string, any> = {}, checkFile = (s: string, errMsg: string) => s) {
     const config: any = {
-        checker_type: 'default',
-        count: 0,
-        subtasks: [],
+        checker_type: cfg.checker_type || 'default',
         judge_extra_files: [],
         user_extra_files: [],
     };
-    config.checker_type = cfg.checker_type || 'default';
     if (cfg.checker) {
         if (!cfg.checker.includes('.')) {
             config.checker = findFileSync(`@hydrooj/hydrojudge/vendor/testlib/checkers/${cfg.checker}.cpp`, false);
@@ -17,62 +14,19 @@ export default async function readYamlCases(cfg: Record<string, any> = {}, check
         if (!config.checker) config.checker = checkFile(cfg.checker, 'Cannot find checker {0}.');
     }
     if (cfg.interactor) config.interactor = checkFile(cfg.interactor, 'Cannot find interactor {0}.');
-    if (cfg.judge_extra_files) {
-        if (typeof cfg.judge_extra_files === 'string') {
-            config.judge_extra_files = [checkFile(cfg.judge_extra_files, 'Cannot find judge extra file {0}.')];
-        } else if (cfg.judge_extra_files instanceof Array) {
-            for (const file of cfg.judge_extra_files) {
-                config.judge_extra_files.push(checkFile(file, 'Cannot find judge extra file {0}.'));
-            }
-        } else throw new Error('Invalid judge_extra_files config.');
-    }
-    if (cfg.user_extra_files) {
-        if (typeof cfg.user_extra_files === 'string') {
-            config.user_extra_files = [checkFile(cfg.user_extra_files, 'Cannot find user extra file {0}.')];
-        } else if (cfg.user_extra_files instanceof Array) {
-            for (const file of cfg.user_extra_files) {
-                config.user_extra_files.push(checkFile(file, 'Cannot find user extra file {0}.'));
-            }
-        } else throw new Error('Invalid user_extra_files config.');
-    }
-    if (cfg.outputs) {
-        config.type = cfg.type || 'objective';
-    } else if (cfg.cases?.length) {
+    ['judge', 'user'].forEach((n) => {
+        const conf = cfg[`${n}_extra_files`];
+        if (!conf) return;
+        if (conf instanceof Array) {
+            config[`${n}_extra_files`] = conf.map((file) => checkFile(file, `Cannot find ${n} extra file {0}.`));
+        } else throw new Error(`Invalid ${n}_extra_files config.`);
+    });
+    if (cfg.outputs) config.type = cfg.type || 'objective';
+    if (cfg.cases?.length) {
         config.subtasks = [{
-            score: +cfg.score || Math.floor(100 / cfg.cases.length),
-            time: parseTimeMS(cfg.time || '1s'),
-            memory: parseMemoryMB(cfg.memory || '256m'),
-            cases: [],
+            cases: cfg.cases,
             type: 'sum',
         }];
-        for (const c of cfg.cases) {
-            config.count++;
-            config.subtasks[0].cases.push({
-                input: c.input ? checkFile(c.input, 'Cannot find input file {0}.') : '/dev/null',
-                output: c.output ? checkFile(c.output, 'Cannot find output file {0}.') : '/dev/null',
-                id: config.count,
-            });
-        }
-    } else if (cfg.subtasks?.length) {
-        for (const subtask of cfg.subtasks) {
-            const cases = [];
-            for (const c of subtask.cases || []) {
-                config.count++;
-                cases.push({
-                    input: c.input ? checkFile(c.input, 'Cannot find input file {0}.') : '/dev/null',
-                    output: c.output ? checkFile(c.output, 'Cannot find output file {0}.') : '/dev/null',
-                    id: config.count,
-                });
-            }
-            config.subtasks.push({
-                score: Number.isSafeInteger(+subtask.score) ? +subtask.score : 100,
-                if: subtask.if || [],
-                cases,
-                type: subtask.type || 'min',
-                time: parseTimeMS(subtask.time || cfg.time || '1s'),
-                memory: parseMemoryMB(subtask.memory || cfg.memory || '256m'),
-            });
-        }
     }
     if (cfg.time) config.time = parseTimeMS(cfg.time);
     if (cfg.memory) config.memory = parseMemoryMB(cfg.memory);
diff --git a/packages/utils/lib/common.ts b/packages/utils/lib/common.ts
index 1edd95a88..829419522 100644
--- a/packages/utils/lib/common.ts
+++ b/packages/utils/lib/common.ts
@@ -201,10 +201,45 @@ const SubtaskMatcher: MatchRule[] = [
     },
 ];
 
-export function readSubtasksFromFiles(files: string[], checkFile, cfg, rst) {
-    const subtask = {};
-    for (const s of rst.subtasks || []) if (s.id) subtask[s.id] = s;
-    const subtasks = [];
+function* getScore(totalScore: number, count: number) {
+    const base = Math.floor(totalScore / count);
+    const extra = count - (100 % count);
+    for (let i = 0; i < count; i++) {
+        if (i >= extra) yield base + 1;
+        else yield base;
+    }
+}
+
+interface CaseInput {
+    id?: number;
+    time?: number | string;
+    memory?: number | string;
+    score?: number;
+    input?: string;
+    output?: string;
+}
+interface SubtaskInput {
+    cases: CaseInput[];
+    type: 'min' | 'max' | 'sum';
+    time?: number | string;
+    memory?: number | string;
+    score?: number;
+    id?: number;
+    if?: number[];
+}
+interface ParsedCase extends CaseInput {
+    time?: number;
+    memory?: number;
+}
+interface ParsedSubtask extends SubtaskInput {
+    cases: ParsedCase[];
+    time?: number;
+    memory?: number;
+}
+
+export function readSubtasksFromFiles(files: string[], checkFile, config) {
+    const subtask: Record<number, ParsedSubtask> = {};
+    for (const s of config.subtasks || []) if (s.id) subtask[s.id] = s;
     for (const file of files) {
         for (const rule of SubtaskMatcher) {
             const data = rule.regex.exec(file);
@@ -212,13 +247,13 @@ export function readSubtasksFromFiles(files: string[], checkFile, cfg, rst) {
             const sid = rule.subtask(data);
             const c = { input: file, output: '', id: rule.id(data) };
             for (const func of rule.output) {
-                if (cfg.noOutputFile) c.output = '/dev/null';
+                if (config.noOutputFile) c.output = '/dev/null';
                 else c.output = func(data);
                 if (c.output === '/dev/null' || checkFile(c.output)) {
                     if (!subtask[sid]) {
                         subtask[sid] = {
-                            time: parseTimeMS(cfg.time || '1s'),
-                            memory: parseMemoryMB(cfg.memory || '256m'),
+                            time: parseTimeMS(config.time || '1s'),
+                            memory: parseMemoryMB(config.memory || '256m'),
                             type: rule.preferredScorerType,
                             cases: [c],
                         };
@@ -229,24 +264,50 @@ export function readSubtasksFromFiles(files: string[], checkFile, cfg, rst) {
             }
         }
     }
-    for (const i in subtask) {
-        subtask[i].cases.sort((a, b) => (a.id - b.id));
-        subtasks.push(subtask[i]);
-    }
-    subtasks.sort((a, b) => (a.id - b.id));
-    const base = Math.floor(100 / subtasks.length);
-    const extra = subtasks.length - (100 % subtasks.length);
-    const config = { count: 0, subtasks };
-    const keys = Object.keys(subtask);
-    for (let i = 0; i < keys.length; i++) {
-        if (i >= extra) subtask[keys[i]].score = base + 1;
-        else subtask[keys[i]].score = base;
-        for (const j of subtask[keys[i]].cases) {
-            config.count++;
-            j.input = checkFile(j.input);
-            j.output = checkFile(j.output);
-            j.id = config.count;
-        }
-    }
-    return config;
+    const subtaskScore = getScore(100, Object.keys(subtask).length);
+    return Object.keys(subtask).map((i) =>
+        ({ ...subtask[i], score: subtaskScore.next().value as number }),
+    );
+}
+
+type NormalizedCase = Required<ParsedCase>;
+interface NormalizedSubtask extends Required<ParsedSubtask> {
+    cases: NormalizedCase[];
+}
+
+export function normalizeSubtasks(
+    subtasks: SubtaskInput[], checkFile: (name: string, errMsg: string) => string,
+    time: number | string, memory: number | string,
+): NormalizedSubtask[] {
+    subtasks.sort((a, b) => (a.id - b.id));
+    const subtaskScore = getScore(100, subtasks.length);
+    let id = 0;
+    let count = 0;
+    return subtasks.map((s) => {
+        id++;
+        s.cases.sort((a, b) => (a.id - b.id));
+        const score = subtaskScore.next().value as number;
+        const caseScore = getScore(score, s.cases.length);
+        return {
+            id,
+            score,
+            type: 'min',
+            if: [],
+            ...s,
+            time: parseTimeMS(s.time || time),
+            memory: parseMemoryMB(s.memory || memory),
+            cases: s.cases.map((c) => {
+                count++;
+                return {
+                    id: count,
+                    score: s.type === 'sum' ? caseScore.next().value as number : score,
+                    ...c,
+                    time: parseTimeMS(c.time || s.time || time),
+                    memory: parseMemoryMB(c.memory || s.memory || memory),
+                    input: c.input ? checkFile(c.input, 'Cannot find input file {0}.') : '/dev/null',
+                    output: c.output ? checkFile(c.output, 'Cannot find output file {0}.') : '/dev/null',
+                };
+            }) as NormalizedCase[],
+        };
+    });
 }