Consolidar código duplicado preservando las reglas de negocio

Dos métodos similares no representan necesariamente una sola regla. Los procesadores de pedidos y devoluciones comparten mecanismos de validación, pero usan distintos prefijos, umbrales de envío y sentidos de actualización del inventario. Eliminar esas diferencias sería un cambio de comportamiento.

Resumen del laboratorio

[!NOTE] Un código similar no significa la misma política de envío o inventario.

Conceptos · Primera tarea · Lista de evidencias · Restablecer

Objetivos de aprendizaje

• Identifica duplicación exacta frente a semántica usando código real.
• Captura las salidas y los efectos secundarios actuales antes de extraer.
• Elige una pequeña abstracción compartida sin generalizar en exceso las políticas.
• Detecta una refactorización que cambie accidentalmente un límite.

Antes de empezar

Prepara 07-duplication con la configuración común. La fuente es el fixture ECommerceOrderAndReturn incluido. Usa su destino de SDK de .NET y un solo proceso de compilación.

Conceptos y casos de uso

Los procesadores capturan y registran excepciones. Por tanto, un código de salida cero de la demostración no demuestra que se haya lanzado una excepción esperada. Este es un ejercicio de caracterización, no una prueba de seguridad de producción.

Escenario del ejercicio

Debes reducir la duplicación de mantenimiento sin cambiar los totales, el inventario, los mensajes de validación ni el orden de eventos. Un cambio de negocio separado necesitaría sus propios criterios de aceptación y revisión.

Tarea 1 - Capturar una línea base real

1. Lee OrderProcessor.cs, ReturnProcessor.cs, Configuration/AppConfig.cs y los servicios a los que llaman.

2. Compila el proyecto copiado:

   dotnet build ECommerceOrderAndReturn.csproj -m:1 -p:UseSharedCompilation=false
   dotnet run --no-build --project ECommerceOrderAndReturn.csproj
   

3. Registra los totales actuales de envío de pedidos y devoluciones, el inventario antes y después y los IDs rechazados. Trata EXPECTED_OUTPUT.md como una ilustración histórica, no como la salida de hoy.

4. Identifica las marcas de tiempo o los identificadores generados antes de comparar registros. No elimines valores de negocio solo para hacer que un diff parezca igual.

Tarea 2 - Analizar la duplicación con Ask

Compare OrderProcessor.Validate and ReturnProcessor.Validate. Cite the shared
mechanics and the policy differences. Do the same for shipping and inventory.
Do not edit. Flag swallowed exceptions separately from the refactoring scope.

Comprueba la respuesta frente a estas reglas concretas de envío:

No sustituyas «más de» por «al menos». Prueba valores inmediatamente por debajo, iguales y por encima de los umbrales antes de mover código.

Tarea 3 - Planificar una extracción

1. En Plan, elige validación o envío para el primer cambio.
2. Exige una tabla de reglas preservadas, parámetros propuestos, llamadores existentes, casos de regresión y reversión.
3. Explica por qué un auxiliar compartido es más sencillo que una nueva jerarquía de clases.
4. Conserva los métodos públicos originales de procesamiento y el orden de efectos secundarios.
5. Define un punto de detención tras una extracción que preserve el comportamiento.

Tarea 4 - Implementar y cuestionar el resultado

1. Pide a Agent que implemente únicamente esa extracción.
2. Inspecciona todos los llamadores; los auxiliares sin uso no constituyen consolidación.
3. Añade aserciones sobre valores deterministas o un harness de caracterización de alcance reducido. No rebautices una transcripción de consola como «pruebas unitarias».
4. Vuelve a ejecutar la misma compilación y demostración y las aserciones añadidas.
5. Invierte temporalmente un umbral en la copia desechable. Confirma que falle la aserción pertinente y después restáuralo.
6. Considera una segunda extracción solo después de verificar la primera.

Verifica tu trabajo

• Los puntos de entrada públicos y las reglas observables permanecen sin cambios.
• Se prueban los límites de igualdad de peso y valor.
• Reservar y restituir siguen teniendo efectos opuestos sobre el inventario.
• Las nuevas aserciones rechazan un umbral intencionalmente incorrecto.
• El diff contiene reutilización real y no una abstracción sin uso.

Solución de problemas

Práctica independiente

Refactoriza un auxiliar de notificaciones conservando los métodos y mensajes públicos. Explica por qué cambiar el comportamiento de reintentos sería una función separada.

Restablecimiento

Detén la demostración, guarda la línea base y la evidencia de comparación y restaura únicamente los archivos nombrados del ejercicio en la copia desechable. No sobrescribas el fixture fuente.

Referencias oficiales

• Planificar con agentes
• Buenas prácticas para agentes
• Compatibilidad con pruebas de C#