Ajouter des objets à un arraylist

Question

public ArrayList<Vehicle> getVehiclesSoldRecently() 
{ 

    ArrayList<Vehicle> a=new ArrayList<>(); 
    ListIterator<Vehicle> it= list.listIterator(); 

    while(it.hasNext()) 
    { 

     Vehicle b=it.next(); 

     if(b.getAgeSolding()<=14) 
     { 
      a.add(b); 
     } 
    } 
    return a; 

} 

J'ai créé cette méthode afin de créer une ArrayList des véhicules vendus au cours des 14 derniers jours, et j'ai un problème. La méthode getAgeSolding fonctionne parfaitement, et la condition dans le if est vérifiée aussi, dans le cas où j'applique cette méthode.

Mais pourquoi le b n'est-il pas ajouté à ArrayList si la condition est vérifiée? J'obtiens une ArrayList vide dans tous les cas. Pourquoi?

la classe de véhicule

package model; 
import java.time.*; 
import java.time.temporal.ChronoUnit; 


    public class Vehicle 
    { 
private String manufacturer, model, VIN; 
private LocalDate datemanuf, datesold; 
private Customer cust; 
private boolean sold; 
private final char taxband; 
private final int price; 

public Vehicle(String manufacturer, String model, String VIN, LocalDate datemanuf, char taxband, int price) 
{ 
    this.manufacturer = manufacturer; 
    this.model = model; 
    this.VIN = VIN; 
    this.datemanuf = datemanuf; 
    this.taxband = taxband; 
    this.price = price; 
    this.cust=null; 
    this.datesold=null; 
    this.sold=false; 
} 

public String getManufacturer() 
{ 
    return manufacturer; 
} 

public String getModel() 
{ 
    return model; 
} 

public Customer getCust() 
{ 
    return cust; 
} 

public String getVIN() 
{ 
    return VIN; 
} 

public LocalDate getDatemanuf() 
{ 
    return datemanuf; 
} 

public LocalDate getDatesold() 
{ 
    return datesold; 
} 

public boolean isSold() 
{ 
    return sold; 
} 

public char getTaxband() 
{ 
    return taxband; 
} 

public int getPrice() 
{ 
    return price; 
} 

public void buy(Customer cust, LocalDate datesold) 
{ 
    this.cust=cust; 
    this.datesold=datesold; 
    this.sold =true; 
} 

public long getAgeOfTheVehicle() 
{ 
    LocalDate Now=LocalDate.now(); 

    long a=datemanuf.until(Now,ChronoUnit.WEEKS); 

    return a; 
} 

public long getAgeSolding() 
{ 
    LocalDate Now=LocalDate.now(); 

    long a=datesold.until(Now,ChronoUnit.DAYS); 

    return a; 
} 

@Override 
public String toString() 
{ 
    String str1=""; 
    String str2; 

    if(sold==true)// TODO code application logic here 
    { 
     str1="Vehicle owned by "+cust.getName()+" since "+datesold; 
    } 

    switch(taxband) 
    { 
     case 'A': 
      str2="0-100"; 
      break; 
     case 'B': 
      str2="101-110"; 
      break; 
     case 'C': 
      str2="111-120"; 
      break; 
     case 'D': 
      str2="121-130"; 
      break; 
     case 'E': 
      str2="131-140"; 
      break; 
     case 'F': 
      str2="141-150"; 
      break; 
     case 'G': 
      str2="151-160"; 
      break; 
     default: 
      str2=""; 
    } 


    return "Manufacturer: "+manufacturer+"\n"+"Model: "+model+"\n"+"VIN: "+VIN+"\n"+"Date of manufacture: "+datemanuf+"\n"+"Price :"+price+" £\n"+"Tax Band: "+str2+"\n"+"Age of Vehicle: "+this.getAgeOfTheVehicle()+" weeks.\n"+str1+"\n"; 
} 

}

la classe Salle d'exposition

public class Showroom 
{ 
    private ArrayList<Vehicle> list; 
    private int position; 

public Showroom() 
{ 
    this.list =new ArrayList<>(); 
    this.position=1; 
} 

public int getPosition() 
{ 
    return position; 
} 



public ArrayList<Vehicle> getList() 
{ 
    return list; 
} 

public boolean add(Vehicle v) 
{ 
    list.add(v); 

    return true; 
} 

public Vehicle find(String VIN) 
{ 
    ListIterator<Vehicle> it= list.listIterator(); 
    int n=1; 

    while(it.hasNext()) 
    { 
     Vehicle a=it.next(); 
     if(a.getVIN().equalsIgnoreCase(VIN)) 
     { 
      this.position=n; 
      return a; 
     } 
     n++; 

    } 
    return null; 
} 

public Vehicle next() 
{ 
    int n=this.position; 
    ListIterator<Vehicle> it= list.listIterator(n); 

    Vehicle a=it.next(); 
    position++; 
    return a; 
} 

public Vehicle previous() 
{ 
    int n=this.position; 
    ListIterator<Vehicle> it= list.listIterator(n-1); 

    Vehicle a=it.previous(); 
    position--; 
    return a; 
} 

public Vehicle current() 
{ 
    int n=this.position; 
    ListIterator<Vehicle> it= list.listIterator(n); 

    Vehicle a=it.previous(); 
    return a; 
} 

public ArrayList<Vehicle> getVehiclesSoldRecently() 
{ 

    ArrayList<Vehicle> a=new ArrayList<>(); 
    ListIterator<Vehicle> it= list.listIterator(); 

    while(it.hasNext()) 
    { 

    Vehicle b=it.next(); 

    if(b.getAgeSolding()<=14) 

     { 

      a.add(b); 
     } 


    return a; 

} 

}

Answer 1

Vous devez utiliser une boucle foreach au lieu d'une boucle while. Les boucles Foreach sont parfaites pour regarder chaque élément d'une liste.

for(Vehicle vehicle : list){ 
    if(vehicle.getAgeSolding()<=14){ 
     a.add(vehicle); 
    } 
} 

La façon dont vous l'avez fait devrait théoriquement fonctionner aussi bien. Mais essayez de le convertir en une boucle foreach et voyez si vous pouvez dire pourquoi ça ne marche pas, puisque c'est une façon plus naturelle de le faire.

Si vous êtes sûr que list a des véhicules à l'intérieur de celui-ci, la seule réponse est qu'aucun des véhicules d'un retour int inférieur ou égal à 14.

Answer 2

Votre code fonctionne bien:

import java.util.ArrayList; 
import java.util.List; 
import java.util.ListIterator; 

public class Main { 

    static List<Vehicle> list; 

    public static void main(String[] args) { 
     list=new ArrayList<>(); 
     list.add(new Vehicle(10)); 
     list.add(new Vehicle(20)); 
     System.out.println(getVehiclesSoldRecently().get(0).getAgeSolding()); 
    } 

    public static ArrayList<Vehicle> getVehiclesSoldRecently() 
    { 

     ArrayList<Vehicle> a=new ArrayList<>(); 
     ListIterator<Vehicle> it= list.listIterator(); 

     while(it.hasNext()) 
     { 

      Vehicle b=it.next(); 

      if(b.getAgeSolding()<=14) 

      { 

       a.add(b); 
      } 
     } 
     return a; 

    } 

} 

Le problème doit être ailleurs.

Mais vous pouvez le faire avec élégance dans Java8:

public static List<Vehicle> getVehiclesSoldRecently2(){ 
    return list.stream() 
      .filter(x->x.getAgeSolding()<=14) 
      .collect(Collectors.toList()); 
}