diff --git a/README.md b/README.md index 697a0cd..b123c5c 100644 --- a/README.md +++ b/README.md @@ -56,3 +56,21 @@ You must follow the **SDLC process** to refactor the code: - Once you complete the refactorization, push your changes to your forked repository. - Create a pull request (PR) to the original repository, explaining the improvements you made and how you followed the **SDLC** approach. +Documentación: +Quiz realizado por: +Sergio Diaz - 192357 +Luis Lemus - 192239 + +Se hizo refactorizacion al codigo storage añadiendo: + -2 arrays para guardar tanto los valores de precio como los valores de la cantidad de objetos vendidos + -Se añadio un for que funcionara para calcular el total de la venta + +Se hizo refactorizacion al codigo AgeValidation añadiendo: + -constantes para definir limites de edad + -scanner para que el usuario pueda ingresar su edad + -un if adicional para los casos en que se digiten valores menores a 0 + +Se hizo refactorizacion al codigo TaxCalculation añadiendo: + -Los nombres para TAX_RATE_HIGH, TAX_RATE_LOW, TAX_THRESHOLD añadiendo sus respectivos valores + -Se creó un array que almacenara el valor de los productos + -Se reemplazaron todos los valores magicos \ No newline at end of file diff --git a/src/AgeValidation.java b/src/AgeValidation.java index 40e0198..01fac02 100644 --- a/src/AgeValidation.java +++ b/src/AgeValidation.java @@ -1,13 +1,23 @@ +/* importamos el scanner al codigo */ +import java.util.Scanner; public class AgeValidation { public static void main(String[] args) { - int age = 18; - - if (age >= 18) { - System.out.println("Access granted"); - } - - if (age < 18) { - System.out.println("Access denied"); + /* Asignamos una constante para el limite de edad */ + final int LIMIT_AGE = 18 ; + try ( /* Declaramos la variable scanner */ Scanner scanner = new Scanner( System.in )) { + int age ; + /* Le preguntamos al usuario cual es su edad */ + System.out.println("What is your age?"); + /* usamos el scanner definido anteriormente para guardar el dato que digite el usuario */ + age = scanner.nextInt(); + /* nos deshacemos de la redundancia de los dos if usando un if-else */ + if (age > LIMIT_AGE ) { + System.out.println("Access granted"); + } else if (age <= LIMIT_AGE & age>0) { + System.out.println("Access denied"); + } else { + System.out.println("Unexisting age");/* Aqui limitamos los valores para que si se llega a ingresar un valor menor a cero aparezca el mensaje de edad inexistente */ + } } } } diff --git a/src/Store.java b/src/Store.java index 2a4729a..78028cf 100644 --- a/src/Store.java +++ b/src/Store.java @@ -1,19 +1,25 @@ public class Store { public static void main(String[] args) { - int p1 = 15; - int p2 = 10; - int p3 = 5; + + /*Se crea un array para el producto */ + int[] PrecioProductos = {13,3,5}; - int total1 = p1 * 2; - int total2 = p2 * 3; - int total3 = p3 * 4; + /*Se crea el array cantidad de productos + para verificar cuantos productos se venden*/ - int totalSales = total1 + total2 + total3; + int[] CantidadProductos = {2,3,4}; - if (totalSales > 50) { + int totalVentas=0; + + /*Se hace el calculo para verificar la cantidad de productos vendidos */ + for(int i=0; i< PrecioProductos.length;i++){ + totalVentas += PrecioProductos[i]*CantidadProductos[i]; + } + + if (totalVentas > 50) { System.out.println("Good sales performance"); } else { System.out.println("Low sales performance"); } } -} +} \ No newline at end of file diff --git a/src/TaxCalculation.java b/src/TaxCalculation.java index 3417866..e4eb62d 100644 --- a/src/TaxCalculation.java +++ b/src/TaxCalculation.java @@ -1,12 +1,17 @@ public class TaxCalculation { public static void main(String[] args) { - double productPrice1 = 100; - double productPrice2 = 200; - double tax1 = productPrice1 * 0.15; - double tax2 = productPrice2 * 0.10; + /* asignamos las constantes que vamos a utilizar */ + final double TAX_RATE_HIGH = 0.15; + final double TAX_RATE_LOW = 0.10; + final double TAX_THRESHOLD = 50; + /* utilizamos un array para almacenar los precios de los productos */ + double[] productPrice = {100, 200}; + /* reemplazamos los valores magicos por las constantes */ + double tax1 = productPrice[0] * TAX_RATE_HIGH; + double tax2 = productPrice[1] * TAX_RATE_LOW; double totalTax = tax1 + tax2; - if (totalTax > 50) { + if (totalTax > TAX_THRESHOLD) { System.out.println("High total tax: " + totalTax); } else { System.out.println("Low tax");