From 8caea9e6bddc92a3e29deebb63d9af93ac86bcf6 Mon Sep 17 00:00:00 2001 From: company-desktop-computer Date: Thu, 22 Oct 2026 09:03:11 +0800 Subject: [PATCH] =?UTF-8?q?fix:=20=E6=B0=B4=E6=B1=A0=E6=88=AA=E9=9D=A2?= =?UTF-8?q?=E4=B8=AD=E8=BD=AC=E6=96=87=E4=BB=B6=20JMSJ.sat=20=E6=8C=89?= =?UTF-8?q?=E8=BF=9B=E7=A8=8B=E9=9A=94=E7=A6=BB=EF=BC=8C=E8=A1=A5=E4=B8=B4?= =?UTF-8?q?=E6=97=B6=E6=96=87=E4=BB=B6=E5=91=BD=E5=90=8D=E6=96=87=E6=A1=A3?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit JMDBDlg、JMSJDlg、HZQTDlg 共 7 处共用固定的 JMSJ.sat，两个 CAD 实例 同时运行时会互相覆盖，acisIn 可能读到另一个实例写的截面。 新增 Pool3dCmd/PoolSectionTemp.h 的 GetPoolSectionSatPath()，文件名改为 JMSJ_.sat。同一进程内所有调用点路径一致，acisOut 写、ShowLastSection 读的隐式链路不受影响；进程之间用 PID 隔离。顺带去掉无意义的 MakeLower。 同时新增 temp-file-naming.md，记录临时文件命名规则、全库现状，以及 DGJM.sat 已废弃、不再改造的结论。 --- SourceCode/Code2026/Pool3dCmd/HZQTDlg.cpp | 9 +-- SourceCode/Code2026/Pool3dCmd/JMDBDlg.cpp | 7 +- SourceCode/Code2026/Pool3dCmd/JMSJDlg.cpp | 11 ++-- .../Code2026/Pool3dCmd/PoolSectionTemp.h | 11 ++++ temp-file-naming.md | 65 +++++++++++++++++++ 5 files changed, 86 insertions(+), 17 deletions(-) create mode 100644 SourceCode/Code2026/Pool3dCmd/PoolSectionTemp.h create mode 100644 temp-file-naming.md diff --git a/SourceCode/Code2026/Pool3dCmd/HZQTDlg.cpp b/SourceCode/Code2026/Pool3dCmd/HZQTDlg.cpp index 98e2c3c..6219c97 100644 --- a/SourceCode/Code2026/Pool3dCmd/HZQTDlg.cpp +++ b/SourceCode/Code2026/Pool3dCmd/HZQTDlg.cpp @@ -4,6 +4,7 @@ #include "JMSJDlg.h" #include "GLTKDlg.h" #include "KTSetting.h" +#include "PoolSectionTemp.h" DLLIMPEXP CString IDCreate(); @@ -270,9 +271,7 @@ void CHZQTDlg::OnBnClickedCheck1() CString sScr; if (dlg.GetCrossSectionScr(sScr)) { - CString sFullPath = FilePath::GetSysDir(); - sFullPath.MakeLower(); - sFullPath += _T("\\JMSJ.sat"); + CString sFullPath = GetPoolSectionSatPath(FilePath::GetSysDir()); CStdioFile file; if (file.Open(sFullPath,CFile::modeCreate |CFile::modeWrite )) @@ -394,9 +393,7 @@ void CHZQTDlg::SetFromWall(AcDbObjectId& id) m_bUserDefSec = TRUE; pWall->GetSpecialtyData(AcDb::kForRead)->GetAt(KEY_SECTIONSCR,m_sRegScr); - CString sFullPath = FilePath::GetSysDir(); - sFullPath.MakeLower(); - sFullPath += _T("\\JMSJ.sat"); + CString sFullPath = GetPoolSectionSatPath(FilePath::GetSysDir()); CStdioFile file; if (file.Open(sFullPath,CFile::modeCreate |CFile::modeWrite )) diff --git a/SourceCode/Code2026/Pool3dCmd/JMDBDlg.cpp b/SourceCode/Code2026/Pool3dCmd/JMDBDlg.cpp index e504960..07dea9b 100644 --- a/SourceCode/Code2026/Pool3dCmd/JMDBDlg.cpp +++ b/SourceCode/Code2026/Pool3dCmd/JMDBDlg.cpp @@ -3,6 +3,7 @@ #include "JMDBDlg.h" #include "GLTKDlg.h" #include "CoreTools.h" +#include "PoolSectionTemp.h" IMPLEMENT_DYNAMIC(CJMDBDlg, CDialog) @@ -87,9 +88,7 @@ void CJMDBDlg::OnBnClickedButton1() CString sScr; if (dlg.GetCrossSectionScr(sScr)) { - CString sFullPath = FilePath::GetSysDir(); - sFullPath.MakeLower(); - sFullPath += _T("\\JMSJ.sat"); + CString sFullPath = GetPoolSectionSatPath(FilePath::GetSysDir()); CStdioFile file; if (file.Open(sFullPath,CFile::modeCreate |CFile::modeWrite )) @@ -173,7 +172,7 @@ void CJMDBDlg::OnBnClickedButton2() if (m_pSectionReg != NULL && bSuccess) { - CString sFullPath = FilePath::getSystemTemporaryFolder() + _T("\\JMSJ.sat"); + CString sFullPath = GetPoolSectionSatPath(FilePath::getSystemTemporaryFolder()); CString sRegionScrp = _T(""); CString sLine; diff --git a/SourceCode/Code2026/Pool3dCmd/JMSJDlg.cpp b/SourceCode/Code2026/Pool3dCmd/JMSJDlg.cpp index d6e8cb3..5795b75 100644 --- a/SourceCode/Code2026/Pool3dCmd/JMSJDlg.cpp +++ b/SourceCode/Code2026/Pool3dCmd/JMSJDlg.cpp @@ -1,6 +1,7 @@ #include "stdafx.h" #include "JMSJDlg.h" #include "CoreTools.h" +#include "PoolSectionTemp.h" #include "JUXCDlg.h" #include "GLTKDlg.h" #include "BYCDlg.h" @@ -78,9 +79,7 @@ BOOL CJMSJDlg::OnInitDialog() void CJMSJDlg::ShowLastSection() { //1.截面 - CString sFullPath = FilePath::getSystemTemporaryFolder(); - sFullPath.MakeLower(); - sFullPath += _T("\\JMSJ.sat"); + CString sFullPath = GetPoolSectionSatPath(FilePath::getSystemTemporaryFolder()); AcDbVoidPtrArray arBodyPtrs; if ( Acad::eOk == AcDbBody::acisIn(sFullPath,arBodyPtrs) ) @@ -139,7 +138,7 @@ void CJMSJDlg::OnBnClickedBtaddtodb() UpdateData(TRUE); if (!m_curSection.pReg) return; - CString sFullPath = FilePath::getSystemTemporaryFolder() + _T("\\JMSJ.sat"); + CString sFullPath = GetPoolSectionSatPath(FilePath::getSystemTemporaryFolder()); CString sRegionScrp = _T(""); CString sLine; @@ -425,9 +424,7 @@ void CJMSJDlg::OnBnClickedBtselfromdb() CString sScr,sScrT; if (dlg.GetCrossSectionScr(sScr)) { - CString sFullPath = FilePath::getSystemTemporaryFolder(); - sFullPath.MakeLower(); - sFullPath += _T("\\JMSJ.sat"); + CString sFullPath = GetPoolSectionSatPath(FilePath::getSystemTemporaryFolder()); CStdioFile file; if (file.Open(sFullPath,CFile::modeCreate |CFile::modeWrite )) diff --git a/SourceCode/Code2026/Pool3dCmd/PoolSectionTemp.h b/SourceCode/Code2026/Pool3dCmd/PoolSectionTemp.h new file mode 100644 index 0000000..b4dc544 --- /dev/null +++ b/SourceCode/Code2026/Pool3dCmd/PoolSectionTemp.h @@ -0,0 +1,11 @@ +#pragma once + +// 水池截面 SAT 中转文件路径。 +// 同一个 CAD 进程内所有调用点返回同一路径；不同进程用 PID 区分， +// 避免两个实例同时写同一个 JMSJ.sat 互相覆盖。 +inline CString GetPoolSectionSatPath(LPCTSTR lpszDir) +{ + CString sPath; + sPath.Format(_T("%s\\JMSJ_%lu.sat"), lpszDir, (unsigned long)::GetCurrentProcessId()); + return sPath; +} diff --git a/temp-file-naming.md b/temp-file-naming.md new file mode 100644 index 0000000..5df2001 --- /dev/null +++ b/temp-file-naming.md @@ -0,0 +1,65 @@ +# 临时文件命名与多实例安全 + +> 记录全库临时文件（ACIS SAT、DWG 中转等）的命名规则、现状与遗留项。 +> 最后更新：2026-10-22 + +## 规则 + +1. 文件名必须带 `GetCurrentProcessId()`。同一进程内需要多次独立使用、彼此不能共享的，再加原子递增序号。 +2. 用完即删（`::DeleteFile`）。写读双方跨函数隐式传递的文件除外，见下文 `JMSJ.sat`。 +3. 路径统一用 `FilePath::getSystemTemporaryFolder()`，不要落到安装目录。 +4. 从 DWG 里读回来的文件名不可信，必须只取纯文件名，拒绝 `..\`、盘符等路径成分。 +5. 从文件读进缓冲区再当字符串用时：分配 `nLen + 1`、清零，按 `Read` 实际返回的长度构造 `CString`，不要直接 `CString = pszTxt`。 + +## 现状 + +| 位置 | 改前 | 改后 | 状态 | +| --- | --- | --- | --- | +| `CoreCommand/CoreTools.cpp` `MakeTempSatPath` | — | `x3d__.sat` | 已符合，调用方负责删除 | +| `CoreLibrary/AcDbX3dSolid.cpp` `CreateFromSat` | `temp\temp.sat` | `temp__.sat` | 已修，用完删除 | +| `CoreLibrary/AecDbEwPipeBase.cpp` `RegionAcisoutDwgOutFields` | `Region_.dat` | `Region__.dat` | 已修；未入库实体无句柄时用序号 `N` | +| `Pool3dCmd/JMDBDlg.cpp` `JMSJDlg.cpp` `HZQTDlg.cpp` | `JMSJ.sat` | `JMSJ_.sat` | 已修，见下文 | +| `CoreCommand/KTEarthWormCmd.cpp` `CO_RGCSCmdFnc` | `Region_.dat` | 未改 | 测试命令，不在产品链路，暂不处理 | +| `DGJM.sat` 20 处 | `DGJM.sat` | 未改 | 代码已废弃，见下文 | +| `PipelineCmd` `temp_$_$_.equ`、`tmp_dwg_$$%d.dwg` | 固定名 | 未改 | 界面导出路径，多实例会冲突，低优先级 | + +## DGJM.sat：废弃代码，不再处理 + +分布：`AcDbX3dSolid.cpp`(2)、`AecDbEarthWormFitting.cpp`(2)、`AcDbX3dGroup.cpp`(3)、`EarthWormCmd/*`(13)。 + +这是写方 `acisOut` 和读方 `acisIn` 之间的隐式约定：A 函数把 region 写到固定文件，B 函数再从同一个文件读回来。加 PID 或序号会让读方找不到文件，必须先把接口改成「写方返回路径、读方接收路径」才能动。 + +该链路已整体废弃，**文件名保持原样，不再改造**。只在 PR #31 中修了读取越界：`new char[n]` 后直接赋给 `CString` 会把堆里的随机字节当字符串读，现改为 `new char[n+1]` + 清零 + 按实际长度构造。涉及 `AcDbX3dSolid`、`AecDbEarthWormFitting`、`DGJMDlg`、`GJYXJMDlg`、`JMDBDlg`、`JMSJDlg` 共 6 处。 + +若将来这段代码被复用，先改接口，再按本文规则命名。 + +## JMSJ.sat：水池截面中转 + +7 处，分布在 3 个文件： + +| 文件 | 用途 | +| --- | --- | +| `JMDBDlg.cpp` | 从标准库选截面（写）→ `acisIn`；`acisOut` 导出截面字符串（读） | +| `JMSJDlg.cpp` | `ShowLastSection` 读上次截面；入库时 `acisOut` 导出；从标准库选截面（写）→ `acisIn` | +| `HZQTDlg.cpp` | 从标准库选截面（写）→ `acisIn`，两处 | + +目录原本分两套：`JMDBDlg` / `JMSJDlg` 用 `FilePath::getSystemTemporaryFolder()`，`HZQTDlg` 用 `FilePath::GetSysDir()`（安装目录下的 `sys`）。两套互不相干，本次只加 PID 后缀，目录未动。 + +统一走 `Pool3dCmd/PoolSectionTemp.h`： + +```cpp +inline CString GetPoolSectionSatPath(LPCTSTR lpszDir) +{ + CString sPath; + sPath.Format(_T("%s\\JMSJ_%lu.sat"), lpszDir, (unsigned long)::GetCurrentProcessId()); + return sPath; +} +``` + +同一进程内所有调用点返回同一路径，所以 `acisOut` 写完、`ShowLastSection` 再读这条隐式链路仍然成立；进程之间用 PID 隔离，两个 CAD 实例不会再互相覆盖。 + +**已知取舍**：`CJMSJDlg::ShowLastSection` 在 `OnInitDialog` 里读这个文件，本意是恢复上一次用的截面。加 PID 后只能读到本进程内上一次写的，跨进程/跨会话恢复失效。这个能力原本也不可靠——temp 目录会被清理，多实例还会互相覆盖。如果确实需要跨会话记忆，应改存注册表、ini 或文档数据库，而不是靠临时文件。 + +## 待处理 + +- `PipelineCmd` 的 `temp_$_$_.equ`、`tmp_dwg_$$%d.dwg`：界面导出路径，多实例会冲突，暂未改。