Actualmente, estaba creando una vista de tabla usando JavaFX y me encontré con este problema en el que simplemente agregaría el último elemento de la matriz (ya que todos los demás elementos se sobrescriben ...)
public void companyTable() { for(CompanyData s: companydataList()){ companyDataTableView.getItems().setAll(s); } } Donde companyDataList es:
private List<CompanyData> companydataList(){ CompanyData company = new CompanyData("test",9,1); for(String i : sim.getCompanyNames()) { company.setPFCompanyName(i); } for(int j : sim.getCompanyValues()) { company.setPFShareValues(j); } List<CompanyData> companydata = new ArrayList<>(); companydata.add(company); return companydata; }Los datos se agregan a esto (Establecedores y captadores de cadenas)
private final StringProperty PFCompanyName; private final IntegerProperty PFShareValues; public CompanyData(String CompanyName, int ShareValue, int ClosingPence) { this.PFCompanyName = new SimpleStringProperty(CompanyName); this.PFShareValues = new SimpleIntegerProperty(ShareValue); } public String getPFCompanyName() { return PFCompanyName.get(); } public StringProperty PFCompanyNameProperty() { return PFCompanyName; } public void setPFCompanyName(String PFCompanyName) { this.PFCompanyName.set(PFCompanyName); } public int getPFShareValues(int j) { return PFShareValues.get(); } public IntegerProperty PFShareValuesProperty() { return PFShareValues; } public void setPFShareValues(int PFShareValues) { this.PFShareValues.set(PFShareValues); }Actualmente la salida es:
CompanyName CompanyValue Samsung 1093Pero lo que deseo es:
CompanyName CompanyValue Nokia 3 Apple 1 HTC 9 Samsung 1093El método setAll(...) reemplaza todos los elementos actualmente en la lista con los que usted proporciona (los "establece todos"). Entonces, cada vez que itera a través de su ciclo, reemplaza todos los elementos con el actual. Al final, solo tendrá un elemento en la tabla.
Una ObservableList es un subtipo de java.util.List estándar, por lo que puede llamar a cualquiera de los métodos de lista estándar. Por ejemplo, puede agregar cada elemento en su lugar:
public void companyTable() { for(CompanyData s: companydataList()){ companyDataTableView.getItems().add(s); } }Por supuesto, no es necesario que escribas el ciclo tú mismo, simplemente puedes agregarlos todos:
public void companyTable() { companyDataTableView.getItems().addAll(companydataList()); }o, si es lo que necesitas, configúralos todos:
public void companyTable() { companyDataTableView.getItems().setAll(companydataList()); } Además, su método companydataList() solo crea una instancia de CompanyData y luego la cambia constantemente. Aquí está su implementación actual, con comentarios que explican qué hace cada línea que escribió:
private List<CompanyData> companydataList(){ // create a single instance: CompanyData company = new CompanyData("test",9,1); // repeatedly change the name of that instance: for(String i : sim.getCompanyNames()) { company.setPFCompanyName(i); } // repeatedly change the value of that instance: for(int j : sim.getCompanyValues()) { company.setPFShareValues(j); } // create an empty list: List<CompanyData> companydata = new ArrayList<>(); // add one object to the list companydata.add(company); // return the list containing the single object: return companydata; } Debe crear una instancia de CompanyData para cada uno de los pares de name / value y agregar cada instancia a la lista. Suponiendo sim.getCompanyNames() y sim.getCompanyValues() devuelvan listas (o matrices; asumiré que son listas) de la misma longitud, debe hacer algo como
private List<CompanyData> companydataList(){ List<String> companyNames = sim.getCompanyNames(); List<Integer> companyValues = sim.getCompanyValues(); List<CompanyData> companydata = new ArrayList<>(); for (int i = 0 ; i < companyNames.size(); i++) { String name = companyNames.get(i); int value = companyValues.get(i); CompanyData company = new CompanyData(); company.setPFCompanyName(name); company.setPFShareValues(value); companydata.add(company); } return companydata; } Obviamente, sería mucho más sensato hacer que sim , que supongo que es algún tipo de acceso a datos, devuelva un List<CompanyData> directamente en primer lugar, en lugar de dos listas diferentes para las diferentes propiedades.