Tengo un metodo que recibe un ArrayList de productos, cada producto tiene su codigo, importe, descripcion y stock. Lo que deberia hacer el metodo es devolver el stock del producto si este esta en la lista (para eso se inserta el codigo de producto en los parametros), o devuelve 0 si el codigo de producto no esta en la lista. El problema es que siempre devuelve 0 el metodo aunque SI EXISTA el producto con el codigo correspondiente. aqui esta el metodo:
public static Integer devolverStock(ArrayList<Producto> e, Integer code) {
for(Integer i=0;i<e.size();i++) {
if(e.get(i).getCodigo()==code) {
return e.get(i).getStock();
}
}
return 0;
}
esto es lo que hago en el metodo main:
System.out.println("Stock del producto 10234879: " + Producto.devolverStock(lista,10234879));
el problema es que SIEMPRE devuelve 0, da igual que el codigo exista en la lista, siempre me devuelve 0, y el producto existe porque lo comprobe yo mismo imprimiento la lista usando toString()
esta es la clase Producto:
package lista_productos;
import java.util.ArrayList;
public class Producto implements Comparable<Producto>{
private Integer codigo;
private String descripcion;
private Double importe;
private Integer stock;
public Producto(Integer codigo, String descripcion, Double importe, Integer stock) {
this.codigo = codigo;
this.descripcion = descripcion;
this.importe = importe;
this.stock = stock;
}
public String toString() {
return "Codigo: " + this.getCodigo() + " | Descripcion: " + this.getDescripcion() + " | Importe: " + this.getImporte() + " | Stock: " + this.getStock() + "\n";
}
public int compareTo(Producto p) {
if(this.getCodigo()>p.getCodigo()) return 1;
if(this.getCodigo()<p.getCodigo()) return -1;
return 0;
}
public void incrementar10() {
setImporte(getImporte() + (getImporte()*10/100));
}
public static ArrayList<Producto> menos50Stock(ArrayList<Producto> e){
ArrayList<Producto> menos50 = new ArrayList<Producto>();
for(Integer i=0;i<e.size();i++) {
if(e.get(i).getStock()<50) {
menos50.add(e.get(i));
}
}
return menos50;
}
public static ArrayList<Producto> mas50Stock(ArrayList<Producto> e){
ArrayList<Producto> mas50 = new ArrayList<Producto>();
for(Integer i=0;i<e.size();i++) {
if(e.get(i).getStock()>=50) {
mas50.add(e.get(i));
}
}
return mas50;
}
public static Integer devolverStock(ArrayList<Producto> e, Integer code) {
for(Integer i=0;i<e.size();i++) {
if(e.get(i).getCodigo()==code) {
return e.get(i).getStock();
}
}
return 0;
}
public Integer getCodigo() {
return codigo;
}
public void setCodigo(Integer codigo) {
this.codigo = codigo;
}
public String getDescripcion() {
return descripcion;
}
public void setDescripcion(String descripcion) {
this.descripcion = descripcion;
}
public Double getImporte() {
return importe;
}
public void setImporte(Double importe) {
this.importe = importe;
}
public Integer getStock() {
return stock;
}
public void setStock(Integer stock) {
this.stock = stock;
}
}
**el metodo main completo:**
package lista_productos;
import java.util.ArrayList;
import java.util.Collections;
public class Pruebas {
public static void main(String[] args) {
ArrayList<Producto> lista = new ArrayList<Producto>();
ArrayList<Producto> menos50;
ArrayList<Producto> mas50;
Producto yerba = new Producto(12209999,"Yerba_Mate_Marolio_x1Kg",25.50,75);
Producto vino = new Producto(00015470,"Vino_Balbo_x1Lt",60.00,100);
Producto galletitas = new Producto(10002450,"Surtido_Bagley_x500g",40.00,100);
Producto azucar = new Producto(10520123,"Azucar_Ledesma_x1Kg",50.00,25);
Producto pan = new Producto(10234879,"Pan_xKg",35.50,12);
lista.add(yerba);
lista.add(vino);
lista.add(galletitas);
lista.add(azucar);
lista.add(pan);
Collections.sort(lista);
lista.get(0).incrementar10();
menos50 = Producto.menos50Stock(lista);
mas50 = Producto.mas50Stock(lista);
System.out.println(lista.toString());
System.out.println();
System.out.println(menos50.toString());
System.out.println();
System.out.println(mas50.toString());
System.out.println("Stock del producto 10234879: " + Producto.devolverStock(lista,10234879));
}
}
getCodigo()
y cómo está organizado tu ArrayList?Integer
, noint
, lo cual quiere decir queInteger
es una clase, no un primitivo, debes usar por esoequals
en la comparación:if(e.get(i).getCodigo().equals(code))
... ¿Por qué? Porque cada código de producto es una instancia de la claseInteger
y debes comparar las instancias de los objetos, no los valores.!
, por ejemplo:if ( !valor.equals(otrovalor) ){
... ahí estarías preguntando si esfalse