Subiendo correcciones

Author: fvenegasn

ejercicio2-balanza/src/main/java/ar/edu/unlp/info/oo1/ejercicio2_balanza/Balanza.java

@@ -5,7 +5,7 @@ public class Balanza {
 	private double precioTotal;
 	private double pesoTotal;
 
-	public Balanza() { // consultar
+	public Balanza() {
 		ponerEnCero();
 	}
 
@@ -18,13 +18,14 @@ public void ponerEnCero() {
 	public void agregarProducto(Producto producto) {
 		this.cantidadDeProductos++;
 		this.pesoTotal = this.pesoTotal+ producto.getPeso();
-		this.precioTotal = this.precioTotal + (producto.getPrecio()); // no correspondería realizarlo en el producto 
-														   			  //ya que es la balanza quien obtiene este calculo, 
-														   			  // un producto no puede calcular su precio total
+		this.precioTotal = this.precioTotal + (producto.getPrecio());
 	}
 
 	public Ticket emitirTicket() {
-		Ticket ticket = new Ticket(this.getCantidadDeProductos(), this.getPesoTotal(), this.getPrecioTotal());
+		int qProductos = this.getCantidadDeProductos();
+		double pesoTotal = this.getPesoTotal();
+		double precioTotal = this.getPrecioTotal(); // se agregan variables que obtienen los valores de los atributos para no enviar como parámetro la referencia de mi atributo. sería como "pasar por valor" los atributos
+		Ticket ticket = new Ticket(qProductos, pesoTotal, precioTotal);
 		return ticket;
 	}
 

ejercicio2-balanza/target/classes/META-INF/maven/ar.edu.unlp.info.oo1/ejercicio2-balanza/pom.properties

@@ -1,5 +1,5 @@
 #Generated by Maven Integration for Eclipse
-#Sat Sep 02 20:02:24 ART 2023
+#Thu Sep 28 00:29:33 ART 2023
 artifactId=ejercicio2-balanza
 groupId=ar.edu.unlp.info.oo1
 m2e.projectLocation=C\:\\Users\\User\\eclipse-workspace\\ejercicio2-balanza

ejercicio2-balanza/target/classes/ar/edu/unlp/info/oo1/ejercicio2_balanza/Balanza.class

@@ -6,7 +6,7 @@
 public class Balanza {
 	private List<Producto> productos;
 
-	public Balanza() { // consultar
+	public Balanza() {
 		ponerEnCero();
 	}
 
@@ -19,7 +19,8 @@ public void agregarProducto(Producto producto) {
 	}
 
 	public Ticket emitirTicket() {
-		Ticket ticket = new Ticket(this.getProductos());
+		List<Producto> productosEnTicket = this.getProductos();
+		Ticket ticket = new Ticket(productosEnTicket);
 		return ticket;
 	}
 

ejercicio3bis-balanzaMejorada/src/main/java/ar/edu/unlp/info/oo1/ejercicio2_balanza/Balanza.java

@@ -26,7 +26,7 @@ public LocalDate getFecha() {
 		return fecha;
 	}
 
-	// no los estoy calculando dos veces de la misma forma en dos distintas clases?
+	// no los estoy calculando dos veces de la misma forma en dos distintas clases? -> es casualidad y está bien que sea así
 	public int getCantidadDeProductos() {
 		return this.productos.size();
 	}
@@ -39,6 +39,4 @@ public double getPrecioTotal() {
 		return this.productos.stream().mapToDouble(producto -> producto.getPrecio()).sum();
 	}
 
-	
-	
 }

ejercicio3bis-balanzaMejorada/src/main/java/ar/edu/unlp/info/oo1/ejercicio2_balanza/Ticket.java

@@ -1,5 +1,5 @@
 #Generated by Maven Integration for Eclipse
-#Tue Sep 05 11:45:04 ART 2023
+#Thu Sep 28 20:40:08 ART 2023
 artifactId=ejercicio2-balanza
 groupId=ar.edu.unlp.info.oo1
 m2e.projectLocation=C\:\\Users\\User\\eclipse-workspace\\ejercicio3bis-balanzaMejorada

ejercicio3bis-balanzaMejorada/target/classes/META-INF/maven/ar.edu.unlp.info.oo1/ejercicio2-balanza/pom.properties

@@ -13,22 +13,22 @@ protected boolean puedeExtraer(double monto) {
 		return false;
 	}
 
+	@Override
 	public void depositar(double monto) {
-		super.depositar(monto);
-		this.extraerSinControlar(this.impuesto(monto));
+		super.depositar(monto-this.impuesto(monto));
 	}
 
-	public boolean extraer(double monto) {
-		if (super.extraer(monto)) {
-			this.extraerSinControlar(this.impuesto(monto));
+	@Override
+	public boolean extraer(double monto) { // no lo puedo convertir a sentencia única porque sino no puedo devolver true o false en base a la operación, sino que me queda void
+		if (super.extraer(monto+this.impuesto(monto))) {
 			return true;
 		}
 		return false;
 	}
 
-	public boolean transferirACuenta(double monto, Cuenta cuentaDestino) {
-		if (super.transferirACuenta(monto, cuentaDestino)) {
-			this.extraerSinControlar(this.impuesto(monto));
+	@Override
+	public boolean transferirACuenta(double monto, Cuenta cuentaDestino) { // idem
+		if (super.transferirACuenta(monto+this.impuesto(monto), cuentaDestino)) {
 			return true;
 		}
 		return false;

ejercicio3bis-balanzaMejorada/target/classes/ar/edu/unlp/info/oo1/ejercicio2_balanza/Balanza.class

@@ -8,6 +8,7 @@
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
+// los tests se pueden hacer más robustos agregando más verificaciones en cada uno de los casos de las operaciones
 
 public class CajaDeAhorroTest {
 

ejercicio9-cuentaConGanchos/src/main/java/ar/edu/unlp/info/oo1/ejercicio9_cuentaConGanchos/CajaDeAhorro.java

@@ -1,5 +1,5 @@
 #Generated by Maven Integration for Eclipse
-#Sun Sep 24 09:45:17 ART 2023
+#Thu Sep 28 00:29:35 ART 2023
 artifactId=ejercicio9-cuentaConGanchos
 groupId=ar.edu.unlp.info.oo1
 m2e.projectLocation=C\:\\Users\\User\\eclipse-workspace\\ejercicio9-cuentaConGanchos