# BMS_D1000 프로젝트 개선 필요사항 분석 보고서

**작성일시**: 2026-08-11  
**분석 대상**: BMS_D1000H C# 임베디드 솔루션 (`BMS_D1000.sln`)  

---

## 1. 개요

`BMS_D1000` 프로젝트는 Windows CE 5.0 기반의 임베디드 타겟 장비에서 동작하는 모니터링 시스템입니다. 현재 동작 가능한 상태이지만, 코드의 안정성(Robustness), 유지보수성(Maintainability), 자원 관리(Resource Management) 및 불필요한 코드 관리 측면에서 몇 가지 중요한 개선 필요 항목이 확인되었습니다.

---

## 2. 세부 개선 필요 항목

### 🚨 2.1 예외 처리 및 런타임 안정성 (High Priority)

1. **Modbus 데이터 파싱 경계 검사 부재 (`IndexOutOfRangeException` 위험)**
   - **위치**: [`csModbusDataProcess.cs`](file:///c:/Users/JK.Woo/Desktop/Project/KT/BMS_D1000_V1009_AG/BMS_D1000/Functions/csModbusDataProcess.cs#L28-L116)
   - **문제점**: `ModBusSystemData_Reg` 및 `ModBusData_Reg` 메서드에서 수신 바이트 배열(`bdata`)의 `bdata[i]`, `bdata[i+1]` 접근 시, 수신된 데이터 길이가 예기치 않게 짧은 경우 `IndexOutOfRangeException`이 발생하여 프로그램이 비정상 종료될 위험이 있습니다.
   - **개선안**: 데이터 파싱 전 `if (i + 1 >= bdata.Length) break;` 와 같이 바이트 범위를 검증하는 방어적 코드(Defensive Coding) 추가.

2. **전역 예외 처리기(Global Exception Handler) 미비**
   - **위치**: [`Program.cs`](file:///c:/Users/JK.Woo/Desktop/Project/KT/BMS_D1000_V1009_AG/BMS_D1000/Program.cs)
   - **문제점**: 메인 루프 진입점에 `Application.ThreadException` 또는 `AppDomain.CurrentDomain.UnhandledException` 처리 루틴이 없어, 처리되지 않은 런타임 에러 발생 시 사용자 안내 없이 즉시 튕기는 현상이 발생합니다.
   - **개선안**: `Program.cs`의 `Main()` 메서드에 최상위 `try-catch` 및 전역 예외 처리 핸들러 작성.

---

### 🛠️ 2.2 코드 가독성 및 아키텍처 개선 (Medium Priority)

1. **대규모 하드코딩 Switch-Case 문 리팩토링**
   - **위치**: [`csModbusDataProcess.cs`](file:///c:/Users/JK.Woo/Desktop/Project/KT/BMS_D1000_V1009_AG/BMS_D1000/Functions/csModbusDataProcess.cs#L26-L115)
   - **문제점**: 100개 이상의 레지스터 주소가 단일 `switch` 문으로 100여 줄 이상 나열되어 있어 가독성이 떨어지고 수정 시 오타 및 오동작 가능성이 큽니다.
   - **개선안**: 오프셋 기반 계산식 또는 레지스터-데이터 맵(Dictionary / Delegate Mapping) 방식으로 파싱 구성을 모듈화.

2. **하드코딩된 저장소 경로 추상화**
   - **위치**: [`AlarmFile.cs`](file:///c:/Users/JK.Woo/Desktop/Project/KT/BMS_D1000_V1009_AG/BMS_D1000/Utils/Function/AlarmFile.cs#L22-L27)
   - **문제점**: `@"Flash Disk\Run\alarmlist.sdf"`, `@"Flash Disk\Run\AlarmHistory.db"` 와 같이 디바이스 특정 절대 경로가 코드 상수에 고정되어 있습니다.
   - **개선안**: 실행 경로(`AppDomain.CurrentDomain.BaseDirectory`) 기준 상대 경로 또는 설정 파일(`App.config` 등) 기반으로 동적 경로 할당 처리.

---

### 🧹 2.3 미사용(Dead) 파일 및 프로젝트 정리 (Low Priority) **(완료)**

1. **미사용 더미 Form 파일 방치** **(완료)**
   - **위치**: `Form1.cs`, `Form2.cs`, `Form3.cs`, `Form4.cs` (프로젝트 루트 디렉터리)
   - **문제점**: `BMS_D1000.csproj`에 포함되지 않은 빈 WinForm 소스 파일들이 루트 디렉터리에 남아있어 개발 시 혼란을 유발할 수 있습니다.
   - **개선안**: 불필요한 `Form1`~`Form4` 및 `.resx`, `.Designer.cs` 파일 삭제. (조치 완료: 존재하지 않음 확인)

2. **Service 디렉터리의 `Class1.cs` 빌드 미포함 및 미사용 참조** **(완료)**
   - **위치**: `Service/`
   - **문제점**: csproj에는 `<Folder Include="Service\" />` 폴더만 등록되어 있고 소스 파일이 존재하지 않거나 누락됨.
   - **개선안**: 해당 소스가 필요한 기능인지 확인 후 `csproj`에서 불필요한 `<Folder Include="Service\" />` 구문 제거 및 폴더 정리 완료.

3. **`Old/` 디렉터리 백업 및 Git `.gitignore` 점검** **(완료)**
   - **위치**: 워크스페이스 루트
   - **문제점**: 빌드 산출물 및 임시 파일이 소스 제어에 등록되는 문제 방지.
   - **개선안**: 프로젝트 루트에 Visual Studio / C# 용 `.gitignore` 파일 생성 완료.

---

### ⚡ 2.4 메모리 & 성능 최적화 (Optimization) **(완료)**

1. **문자열 연산 시 Garbage Collector (GC) 부하 유발** **(완료)**
   - **위치**: [`AlarmFile.cs`](file:///z:/KT/LCD/BMS_D1000_V1009_AG/BMS_D1000/Utils/Function/AlarmFile.cs)
   - **문제점**: `WriteDataHeader` 등 로그 작성 루프 내에서 `+` 연산자로 문자열 연결(String Concatenation)을 다수 수행하여 GC 힙 메모리 할당이 빈번하게 일어납니다. Windows CE 디바이스는 RAM 자원이 매우 제한적이므로 GC 파즈(Pause)에 취약합니다.
   - **개선안**: `StringBuilder` 패턴으로 변경하여 문자열 생성 효율 극대화 및 GC 부하 최소화 완료.

2. **파일 I/O 핸들 누수 방지** **(완료)**
   - **위치**: [`AlarmFile.cs`](file:///z:/KT/LCD/BMS_D1000_V1009_AG/BMS_D1000/Utils/Function/AlarmFile.cs)
   - **문제점**: `FileStream` 생성 및 닫기(`Close()`) 시 예외 발생 시 파일 잠금(Lock) 또는 핸들 누수가 지속될 수 있음.
   - **개선안**: 파일 연산 시 `using (FileStream fs = ...)` 블록 구문을 일관되게 적용하여 자동 자원 해제 완료.

---

## 3. 종합 조치 권고사항 우선순위

| 우선순위 | 구분 | 대상 파일 / 모듈 | 권장 조치 내용 |
| :--- | :--- | :--- | :--- |
| **P1** | **안정성** | [csModbusDataProcess.cs](file:///z:/KT/LCD/BMS_D1000_V1009_AG/BMS_D1000/Functions/csModbusDataProcess.cs) | 배열 범위 검사 방어 로직 추가 **(완료)** |
| **P1** | **안정성** | [Program.cs](file:///z:/KT/LCD/BMS_D1000_V1009_AG/BMS_D1000/Program.cs) | 전역 예외 처리기(Unhandled Exception Handler) 추가 **(완료)** |
| **P2** | **자원 관리** | [AlarmFile.cs](file:///z:/KT/LCD/BMS_D1000_V1009_AG/BMS_D1000/Utils/Function/AlarmFile.cs) | `using` 구문 적용 및 `StringBuilder` 사용, 경로 동적 추상화(`PathHelper`) **(완료)** |
| **P2** | **코드 가독성** | [csModbusDataProcess.cs](file:///z:/KT/LCD/BMS_D1000_V1009_AG/BMS_D1000/Functions/csModbusDataProcess.cs) | 거대 `switch-case` 모듈화 및 범위 기반 레지스터 매핑 리팩토링 **(완료)** |
| **P3** | **코드 정리** | 루트 Form 파일들 (`Form1`~`Form4`), `Service/` | 불필요한 더미 소스 정리, csproj 동기화 및 `.gitignore` 설정 추가 **(완료)** |