[1팀 이은지] Chapter 2-1. 클린코드와 리팩토링

.cursor/rules/assignment.mdc

@@ -0,0 +1,38 @@
+---
+alwaysApply: true
+---
+
+# 절대 반드시 지켜야만 하는 절대적 원칙 (안지키면 다시해야함)
+
+- 코드의 동작이나 구현이 바뀌면 안되고 반드시 구조 변경(리팩토링)만 해야해야만해
+- 공통으로 쓰이는 파일만 공통 폴더에 넣어두고, 비즈니스 로직이 담긴 경우, 관심사끼리 묶어 폴더로 관리해야해
+- src/basic/tests/basic.test.js, src/advanced/tests/advanced.test.js 테스트 코드가 모두 하나도 빠짐없이 통과해야해 (테스트는 npx vitest run 으로 watch 가 발생하지 않도록 해)
+- 테스트 코드 검증 여부는 main.basic.js 를 기준으로 검사해야만해. origin 파일은 의미없어.
+
+-> 결과파일은 main.basic.js 에 적용해줘
+-> 작업 후 마지막으로 절대 원칙이 지켜졌는지 한번 더 컴토 후 올바르게 고치고 알려줘
+
+# 체크리스트
+
+아래 체크리스트를 준수했는지 체크해줘.
+
+- 코드가 Prettier를 통해 일관된 포맷팅이 적용되어 있는가?
+- 적절한 줄바꿈과 주석을 사용하여 코드의 논리적 단위를 명확히 구분했는가?
+- 변수명과 함수명이 그 역할을 명확히 나타내며, 일관된 네이밍 규칙을 따르는가?
+- 매직 넘버와 문자열을 의미 있는 상수로 추출했는가?
+- 중복 코드를 제거하고 재사용 가능한 형태로 리팩토링했는가?
+- 함수가 단일 책임 원칙을 따르며, 한 가지 작업만 수행하는가?
+- 조건문과 반복문이 간결하고 명확한가? 복잡한 조건을 함수로 추출했는가?
+- 코드의 배치가 의존성과 실행 흐름에 따라 논리적으로 구성되어 있는가?
+- 연관된 코드를 의미 있는 함수나 모듈로 그룹화했는가?
+- ES6+ 문법을 활용하여 코드를 더 간결하고 명확하게 작성했는가?
+- 전역 상태와 부수 효과(side effects)를 최소화했는가?
+- 에러 처리와 예외 상황을 명확히 고려하고 처리했는가?
+- 코드 자체가 자기 문서화되어 있어, 주석 없이도 의도를 파악할 수 있는가?
+- 비즈니스 로직과 UI 로직이 적절히 분리되어 있는가?
+- 객체지향 또는 함수형 프로그래밍 원칙을 적절히 적용했는가?
+- 코드의 각 부분이 테스트 가능하도록 구조화되어 있는가?
+- 성능 개선을 위해 불필요한 연산이나 렌더링을 제거했는가?
+- 새로운 기능 추가나 변경이 기존 코드에 미치는 영향을 최소화했는가?
+- 리팩토링 시 기존 기능을 그대로 유지하면서 점진적으로 개선했는가?
+- 코드 리뷰를 통해 다른 개발자들의 피드백을 반영하고 개선했는가?

.cursor/rules/clean-code.mdc

@@ -0,0 +1,96 @@
+---
+alwaysApply: true
+---
+
+# Clean Code Writing Rules
+
+## MANDATORY CODE WRITING RULES
+
+You MUST follow these rules when writing any code:
+
+### CORE DESIGN PRINCIPLES
+
+- **DRY**: NEVER repeat the same code
+- **KISS**: Write code as simply as possible
+- **YAGNI**: Do NOT write unnecessary code
+- **Single Responsibility**: Functions MUST be under 20 lines and have ONE clear responsibility
+
+### CODE ORGANIZATION RULES
+
+Apply these 4 organization principles:
+
+- **Proximity**: Group related elements with blank lines
+- **Commonality**: Group related functionality into functions
+- **Similarity**: Use similar names and positions for similar roles
+- **Continuity**: Arrange code in dependency order
+
+### NAMING REQUIREMENTS
+
+#### Naming Principles (ALL MUST BE FOLLOWED)
+
+1. **Predictable**: Name must allow prediction of value, type, and return value
+2. **Contextual**: Add descriptive adjectives or nouns for context
+3. **Clear**: Remove unnecessary words while maintaining clear meaning
+4. **Concise**: Brief yet clearly convey role and purpose
+5. **Consistent**: Use identical terms for identical intentions across entire codebase
+
+#### REQUIRED Naming Patterns
+
+**Action Functions - USE THESE PATTERNS:**
+
+```
+// Creation: create~(), add~(), push~(), insert~(), new~(), append~(), spawn~(), make~(), build~(), generate~()
+// Retrieval: get~(), fetch~(), query~()
+// Transformation: parse~(), split~(), transform~(), serialize~()
+// Modification: update~(), mutation~()
+// Deletion: delete~(), remove~()
+// Communication: put~(), send~(), dispatch~(), receive~()
+// Validation: validate~(), check~()
+// Calculation: calc~(), compute~()
+// Control: init~(), configure~(), start~(), stop~()
+// Storage: save~(), store~()
+// Logging: log~(), record~()
+```
+
+**Data Variables - USE THESE PATTERNS:**
+
+```
+// Quantities: count~, sum~, num~, min~, max~, total
+// State: is~, has~, current~, selected~
+// Progressive/Past: ~ing, ~ed
+// Information: ~name, ~title, ~desc, ~text, ~data
+// Identifiers: ~ID, ~code, ~index, ~key
+// Time: ~at, ~date
+// Type: ~type
+// Collections: ~s
+// Others: item, temp, params, error
+// Conversion: from(), of()
+```
+
+### ABSTRACTION RULES
+
+- **Data Abstraction**: Simplify data structure and processing methods
+- **Process Abstraction**: Encapsulate complex logic into simple interfaces
+- **Appropriate Level**: Do NOT over-abstract or under-abstract
+
+### MANDATORY CHECKLIST
+
+Before finalizing ANY code, you MUST verify:
+
+1. ✅ **Applied standard naming patterns** from above
+2. ✅ **Organized code using 4 organization principles**
+3. ✅ **Split complex logic into small functions**
+4. ✅ **Code expresses intent without comments** (comments only when absolutely necessary)
+5. ✅ **Maintained consistent formatting**
+
+### FORBIDDEN PRACTICES
+
+- ❌ Do NOT mix similar terms (`display` vs `show`)
+- ❌ Do NOT write functions longer than 20 lines
+- ❌ Do NOT repeat code patterns
+- ❌ Do NOT use unclear or ambiguous names
+- ❌ Do NOT violate naming consistency across codebase
+
+## COMPLIANCE REQUIREMENT
+
+ALL code output MUST comply with these rules. No exceptions.