refactor: 优化字典和系统参数页面的表单处理和抽屉关闭逻辑

- 在 `dictionaries/index.tsx` 中添加 `closeTypeDrawer` 和 `closeItemDrawer` 函数，统一处理抽屉关闭和表单重置 - 在 `sys-params/index.tsx` 中添加 `closeDrawer` 函数，统一处理抽屉关闭和表单重置 - 更新 `dictionaries/index.tsx` 和 `sys-params/index.tsx` 中的相关方法，使用新的关闭函数 - 移除 `PageContainer` 组件，改用 `div` 和 `PageHeader` 组件进行页面布局
2026-06-22 09:41:03 +08:00 · 2026-06-22 09:41:03 +08:00 · bdc91f4f8b
parent 194a05cbe0
commit bdc91f4f8b
2 changed files with 68 additions and 24 deletions
--- a/frontend/src/pages/system/dictionaries/index.tsx
+++ b/frontend/src/pages/system/dictionaries/index.tsx
@ -6,7 +6,6 @@ import { createDictItem, createDictType, deleteDictItem, deleteDictType, fetchDi
 import { useDict } from "@/hooks/useDict";
 import { usePermission } from "@/hooks/usePermission";
 import PageHeader from "@/components/shared/PageHeader";
-import PageContainer from "@/components/shared/PageContainer";
 import { getStandardPagination } from "@/utils/pagination";
 import type { SysDictItem, SysDictType } from "@/types";
 import "./index.less";
@ -79,10 +78,17 @@ export default function Dictionaries() {

  const handleEditType = (record: SysDictType) => {
    setEditingType(record);
+    typeForm.resetFields();
    typeForm.setFieldsValue(record);
    setTypeDrawerVisible(true);
  };

+  const closeTypeDrawer = () => {
+    setTypeDrawerVisible(false);
+    setEditingType(null);
+    typeForm.resetFields();
+  };
+
  const handleDeleteType = async (id: number) => {
    await deleteDictType(id);
    message.success(t("common.success"));
@ -97,7 +103,7 @@ export default function Dictionaries() {
      await createDictType(values);
    }
    message.success(t("common.success"));
-    setTypeDrawerVisible(false);
+    closeTypeDrawer();
    loadTypes();
  };

@ -127,10 +133,17 @@ export default function Dictionaries() {

  const handleEditItem = (record: SysDictItem) => {
    setEditingItem(record);
+    itemForm.resetFields();
    itemForm.setFieldsValue(record);
    setItemDrawerVisible(true);
  };

+  const closeItemDrawer = () => {
+    setItemDrawerVisible(false);
+    setEditingItem(null);
+    itemForm.resetFields();
+  };
+
  const handleDeleteItem = async (id: number) => {
    await deleteDictItem(id);
    message.success(t("common.success"));
@ -145,18 +158,28 @@ export default function Dictionaries() {
      await createDictItem(values);
    }
    message.success(t("common.success"));
-    setItemDrawerVisible(false);
+    closeItemDrawer();
    if (selectedType) loadItems(selectedType.typeCode);
  };

  return (
-    <PageContainer
-      title={t("dicts.title")}
-      subtitle={t("dicts.subtitle")}
-    >
-      <Row gutter={24} style={{ flex: 1, minHeight: 0, overflow: "hidden" }}>
+    <div className="app-page dictionaries-page">
+      <PageHeader title={t("dicts.title")} subtitle={t("dicts.subtitle")}/>
+
+      <Row gutter={24} className="flex-1 min-h-0 overflow-hidden">
        <Col span={8} className="h-full flex flex-col overflow-hidden">
-          <Card title={<Space><BookOutlined aria-hidden="true" /><span>{t("dicts.dictType")}</span></Space>} className="app-page__panel-card flex-1 flex flex-col overflow-hidden" styles={{ body: { padding: "12px", flex: 1, display: "flex", flexDirection: "column", overflow: "hidden",maxHeight:"calc(100vh - 360px)" } }} extra={can("sys_dict:type:create") && <Button type="primary" size="small" icon={<PlusOutlined aria-hidden="true" />} onClick={handleAddType}>{t("common.create")}</Button>}>
+          <Card title={<Space><BookOutlined aria-hidden="true"/><span>{t("dicts.dictType")}</span></Space>}
+                className="app-page__panel-card flex-1 flex flex-col overflow-hidden" styles={{
+            body: {
+              padding: "12px",
+              flex: 1,
+              display: "flex",
+              flexDirection: "column",
+              overflow: "hidden"
+            }
+          }} extra={can("sys_dict:type:create") &&
+            <Button type="primary" size="small" icon={<PlusOutlined aria-hidden="true"/>}
+                    onClick={handleAddType}>{t("common.create")}</Button>}>
            <div style={{ marginBottom: 12 }} className="flex-shrink-0">
              <Space.Compact style={{ width: "100%" }}>
                <Input
@ -251,7 +274,13 @@ export default function Dictionaries() {
        </Col>
      </Row>

-      <Drawer title={<Space><BookOutlined aria-hidden="true" /><span>{editingType ? t("dicts.drawerTitleTypeEdit") : t("dicts.drawerTitleTypeCreate")}</span></Space>} open={typeDrawerVisible} onClose={() => setTypeDrawerVisible(false)} width={400} destroyOnClose footer={<div className="app-page__drawer-footer"><Button onClick={() => setTypeDrawerVisible(false)}>{t("common.cancel")}</Button><Button type="primary" onClick={handleTypeSubmit}>{t("common.save")}</Button></div>}>
+      <Drawer title={<Space><BookOutlined
+        aria-hidden="true"/><span>{editingType ? t("dicts.drawerTitleTypeEdit") : t("dicts.drawerTitleTypeCreate")}</span></Space>}
+              open={typeDrawerVisible} onClose={closeTypeDrawer} width={400} destroyOnClose
+              footer={<div className="app-page__drawer-footer"><Button
+                onClick={closeTypeDrawer}>{t("common.cancel")}</Button><Button type="primary"
+                                                                               onClick={handleTypeSubmit}>{t("common.save")}</Button>
+              </div>}>
        <Form form={typeForm} layout="vertical">
          <Form.Item label={t("dicts.typeCode")} name="typeCode" rules={[{ required: true, message: t("dicts.typeCode") }]}>
            <Input disabled={!!editingType} placeholder={t("dictsExt.typeCodePlaceholder")} />
@ -265,7 +294,13 @@ export default function Dictionaries() {
        </Form>
      </Drawer>

-      <Drawer title={<Space><ProfileOutlined aria-hidden="true" /><span>{editingItem ? t("dicts.drawerTitleItemEdit") : t("dicts.drawerTitleItemCreate")}</span></Space>} open={itemDrawerVisible} onClose={() => setItemDrawerVisible(false)} width={400} destroyOnClose footer={<div className="app-page__drawer-footer"><Button onClick={() => setItemDrawerVisible(false)}>{t("common.cancel")}</Button><Button type="primary" onClick={handleItemSubmit}>{t("common.save")}</Button></div>}>
+      <Drawer title={<Space><ProfileOutlined
+        aria-hidden="true"/><span>{editingItem ? t("dicts.drawerTitleItemEdit") : t("dicts.drawerTitleItemCreate")}</span></Space>}
+              open={itemDrawerVisible} onClose={closeItemDrawer} width={400} destroyOnClose
+              footer={<div className="app-page__drawer-footer"><Button
+                onClick={closeItemDrawer}>{t("common.cancel")}</Button><Button type="primary"
+                                                                               onClick={handleItemSubmit}>{t("common.save")}</Button>
+              </div>}>
        <Form form={itemForm} layout="vertical">
          <Form.Item label={t("dicts.typeCode")} name="typeCode"><Input disabled className="tabular-nums" /></Form.Item>
          <Form.Item label={t("dicts.itemLabel")} name="itemLabel" rules={[{ required: true, message: t("dicts.itemLabel") }]}><Input placeholder={t("dictsExt.itemLabelPlaceholder")} /></Form.Item>
@ -275,6 +310,6 @@ export default function Dictionaries() {
          <Form.Item label={t("common.remark")} name="remark"><Input.TextArea placeholder={t("dictsExt.itemRemarkPlaceholder")} rows={3} /></Form.Item>
        </Form>
      </Drawer>
-    </PageContainer>
+    </div>
  );
 }
--- a/frontend/src/pages/system/sys-params/index.tsx
+++ b/frontend/src/pages/system/sys-params/index.tsx
@ -26,7 +26,8 @@ export default function SysParams() {
  const [queryParams, setQueryParams] = useState<SysParamQuery>({ pageNum: 1, pageSize: 10 });
  const [drawerOpen, setDrawerOpen] = useState(false);
  const [editing, setEditing] = useState<SysParamVO | null>(null);
-  const [form] = Form.useForm();
+  const [searchForm] = Form.useForm<SysParamQuery>();
+  const [drawerForm] = Form.useForm<SysParamVO>();

  const loadData = useCallback(async (query = queryParams) => {
    setLoading(true);
@ -44,11 +45,12 @@ export default function SysParams() {
  }, [loadData]);

  const handleSearch = (values: any) => {
+
    setQueryParams({ ...queryParams, ...values, pageNum: 1 });
  };

  const handleReset = () => {
-    form.resetFields();
+    searchForm.resetFields();
    setQueryParams({ pageNum: 1, pageSize: 10 });
  };

@ -58,17 +60,22 @@ export default function SysParams() {

  const openCreate = () => {
    setEditing(null);
-    form.resetFields();
-    form.setFieldsValue({ isSystem: 0, status: 1 });
+    drawerForm.resetFields();
+    drawerForm.setFieldsValue({isSystem: 0, status: 1});
    setDrawerOpen(true);
  };

  const openEdit = (record: SysParamVO) => {
    setEditing(record);
-    form.setFieldsValue(record);
+    drawerForm.resetFields();
+    drawerForm.setFieldsValue(record);
    setDrawerOpen(true);
  };
-
+  const closeDrawer = () => {
+    setDrawerOpen(false);
+    setEditing(null);
+    drawerForm.resetFields();
+  };
  const handleDelete = async (id: number) => {
    try {
      await deleteParam(id);
@ -80,7 +87,7 @@ export default function SysParams() {

  const submit = async () => {
    try {
-      const values = await form.validateFields();
+      const values = await drawerForm.validateFields();
      setSaving(true);
      if (editing) {
        await updateParam(editing.paramId, values);
@ -88,7 +95,7 @@ export default function SysParams() {
        await createParam(values);
      }
      message.success(t("common.success"));
-      setDrawerOpen(false);
+      closeDrawer();
      loadData();
    } finally {
      setSaving(false);
@ -175,7 +182,7 @@ export default function SysParams() {
        </Button>
      }
      toolbar={
-        <Form layout="inline" onFinish={handleSearch}>
+        <Form form={searchForm} layout="inline" onFinish={handleSearch}>
          <Form.Item name="paramKey">
            <Input placeholder={t("sysParams.paramKey")} prefix={<SearchOutlined />} allowClear style={{ width: 200 }} />
          </Form.Item>
@ -216,12 +223,14 @@ export default function SysParams() {
      <Drawer
        title={<span><SettingOutlined className="mr-2" />{editing ? t("sysParams.drawerTitleEdit") : t("sysParams.drawerTitleCreate")}</span>}
        open={drawerOpen}
-        onClose={() => setDrawerOpen(false)}
+        onClose={closeDrawer}
        width={500}
        destroyOnClose
-        footer={<div className="app-page__drawer-footer"><Button onClick={() => setDrawerOpen(false)}>{t("common.cancel")}</Button><Button type="primary" loading={saving} onClick={submit}>{t("common.save")}</Button></div>}
+        footer={<div className="app-page__drawer-footer"><Button
+          onClick={closeDrawer}>{t("common.cancel")}</Button><Button type="primary" loading={saving}
+                                                                     onClick={submit}>{t("common.save")}</Button></div>}
      >
-        <Form form={form} layout="vertical">
+        <Form form={drawerForm} layout="vertical">
          <Form.Item label={t("sysParams.paramKey")} name="paramKey" rules={[{ required: true, message: t("sysParams.paramKey") }]}>
            <Input placeholder={t("sysParamsExt.paramKeyPlaceholder")} disabled={!!editing} />
          </Form.Item>