GPT-4Code quality
دليل إعادة هيكلة الكود القديم
لماذا تهم هذه المطالبة
Refactoring without a plan leads to half-finished migrations. Ordering by risk and ensuring each step is deployable reduces the chance of a blocked release.
فيم نستخدمها
Code quality
المطالبة
Analyze this legacy code and create a refactoring plan. For each refactoring step: 1) What pattern to apply (extract method, introduce parameter object, etc.), 2) The before/after code, 3) How to verify the refactoring didn't break anything, 4) Estimated risk level. Order steps from lowest to highest risk. Ensure each step is independently deployable.
النتيجة
## Refactoring Plan (5 Steps)
### Step 1: Extract Validation Logic (Risk: Low) **Pattern**: Extract Method **Before**: 45-line function with inline validation **After**: `validateOrder()` + `processOrder()` **Verify**: Existing tests should pass unchanged
### Step 2: Replace Magic Numbers (Risk: Low) **Pattern**: Replace Magic Number with Named Constant ```js // Before if (order.total > 10000) { ... } // After const FREE_SHIPPING_THRESHOLD = 10000; if (order.total > FREE_SHIPPING_THRESHOLD) { ... } ```
تؤدي إعادة الهيكلة دون تخطيط إلى عمليات ترحيل غير مكتملة. إن ترتيب الأولويات بناءً على مستوى المخاطر والتأكد من قابلية نشر كل خطوة يحد من احتمالية توقف الإصدار.
يوضح هذا التوجيه نهجاً منظمًا للتعامل مع جودة الكود، مُنتجاً نتائج متسقة وعالية الجودة يمكن البناء عليها وتطويرها بشكل تكراري.